Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1977)

Unified Diff: content/public/android/java/src/org/chromium/content/browser/ZoomManager.java

Issue 10693134: Revert 146000 - Split out ContentViewCore from ContentView for embedders. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/public/android/java/src/org/chromium/content/browser/ZoomManager.java
===================================================================
--- content/public/android/java/src/org/chromium/content/browser/ZoomManager.java (revision 146004)
+++ content/public/android/java/src/org/chromium/content/browser/ZoomManager.java (working copy)
@@ -21,7 +21,7 @@
class ZoomManager {
private static final String TAG = "ContentViewZoom";
- private ContentViewCore mContentViewCore;
+ private ContentView mContentView;
private ZoomButtonsController mZoomButtonsController;
private class ScaleGestureListener implements ScaleGestureDetector.OnScaleGestureListener {
@@ -55,14 +55,14 @@
public boolean onScaleBegin(ScaleGestureDetector detector) {
if (ignoreDetectorEvents()) return false;
mPinchEventSent = false;
- mContentViewCore.setIgnoreSingleTap(true);
+ mContentView.setIgnoreSingleTap(true);
return true;
}
@Override
public void onScaleEnd(ScaleGestureDetector detector) {
- if (!mPinchEventSent || !mContentViewCore.isAlive()) return;
- mContentViewCore.pinchEnd(detector.getEventTime());
+ if (!mPinchEventSent || !mContentView.isAlive()) return;
+ mContentView.pinchEnd(detector.getEventTime());
mPinchEventSent = false;
}
@@ -76,11 +76,11 @@
// that pinchBy() is called without any pinchBegin().
// To solve this problem, we call pinchBegin() here if it is never called.
if (!mPinchEventSent) {
- mContentViewCore.pinchBegin(detector.getEventTime(),
+ mContentView.pinchBegin(detector.getEventTime(),
(int) detector.getFocusX(), (int) detector.getFocusY());
mPinchEventSent = true;
}
- mContentViewCore.pinchBy(
+ mContentView.pinchBy(
detector.getEventTime(), (int) detector.getFocusX(), (int) detector.getFocusY(),
detector.getScaleFactor());
return true;
@@ -89,15 +89,15 @@
private boolean ignoreDetectorEvents() {
return mPermanentlyIgnoreDetectorEvents ||
mTemporarilyIgnoreDetectorEvents ||
- !mContentViewCore.isAlive();
+ !mContentView.isAlive();
}
}
private ScaleGestureDetector mMultiTouchDetector;
private ScaleGestureListener mMultiTouchListener;
- ZoomManager(final Context context, ContentViewCore contentViewCore) {
- mContentViewCore = contentViewCore;
+ ZoomManager(final Context context, ContentView contentView) {
+ mContentView = contentView;
mMultiTouchListener = new ScaleGestureListener();
mMultiTouchDetector = new ScaleGestureDetector(context, mMultiTouchListener);
}
@@ -148,14 +148,13 @@
void updateMultiTouchSupport() {
mMultiTouchListener.setPermanentlyIgnoreDetectorEvents(
- !mContentViewCore.getContentSettings().supportsMultiTouchZoom());
+ !mContentView.getContentSettings().supportsMultiTouchZoom());
}
private ZoomButtonsController getZoomControls() {
if (mZoomButtonsController == null &&
- mContentViewCore.getContentSettings().shouldDisplayZoomControls()) {
- mZoomButtonsController = new ZoomButtonsController(
- mContentViewCore.getContainerView());
+ mContentView.getContentSettings().shouldDisplayZoomControls()) {
+ mZoomButtonsController = new ZoomButtonsController(mContentView);
mZoomButtonsController.setOnZoomListener(new ZoomListener());
// ZoomButtonsController positions the buttons at the bottom, but in
// the middle. Change their layout parameters so they appear on the
@@ -176,8 +175,8 @@
void updateZoomControls() {
if (mZoomButtonsController == null) return;
- boolean canZoomIn = mContentViewCore.canZoomIn();
- boolean canZoomOut = mContentViewCore.canZoomOut();
+ boolean canZoomIn = mContentView.canZoomIn();
+ boolean canZoomOut = mContentView.canZoomOut();
if (!canZoomIn && !canZoomOut) {
// Hide the zoom in and out buttons if the page cannot zoom
mZoomButtonsController.getZoomControls().setVisibility(View.GONE);
@@ -201,9 +200,9 @@
@Override
public void onZoom(boolean zoomIn) {
if (zoomIn) {
- mContentViewCore.zoomIn();
+ mContentView.zoomIn();
} else {
- mContentViewCore.zoomOut();
+ mContentView.zoomOut();
}
// ContentView will call updateZoomControls after its current page scale
// is got updated from the native code.

Powered by Google App Engine
This is Rietveld 408576698