From d7660ccb7b23930e3603fa42aae7b40c204e6c29 Mon Sep 17 00:00:00 2001 From: Grace Kloba Date: Mon, 17 Aug 2009 17:13:01 -0700 Subject: [PATCH] Fix 2058938. As zoom control is uesd in overview mode to switch to reading mode, we also want to hide it after double tap is done. --- core/java/android/webkit/WebView.java | 25 +++++++++++++------------ 1 file changed, 13 insertions(+), 12 deletions(-) diff --git a/core/java/android/webkit/WebView.java b/core/java/android/webkit/WebView.java index 7468aef5df759..f49aab1309ca9 100644 --- a/core/java/android/webkit/WebView.java +++ b/core/java/android/webkit/WebView.java @@ -4689,19 +4689,20 @@ public class WebView extends AbsoluteLayout mZoomCenterY = mLastTouchY; mInZoomOverview = !mInZoomOverview; mCallbackProxy.uiOnChangeViewingMode(mInZoomOverview); - if (mInZoomOverview) { - if (getSettings().getBuiltInZoomControls()) { - if (mZoomButtonsController.isVisible()) { - mZoomButtonsController.setVisible(false); - } - } else { - if (mZoomControlRunnable != null) { - mPrivateHandler.removeCallbacks(mZoomControlRunnable); - } - if (mZoomControls != null) { - mZoomControls.hide(); - } + // remove the zoom control after double tap + if (getSettings().getBuiltInZoomControls()) { + if (mZoomButtonsController.isVisible()) { + mZoomButtonsController.setVisible(false); } + } else { + if (mZoomControlRunnable != null) { + mPrivateHandler.removeCallbacks(mZoomControlRunnable); + } + if (mZoomControls != null) { + mZoomControls.hide(); + } + } + if (mInZoomOverview) { zoomWithPreview((float) getViewWidth() / mZoomOverviewWidth); } else { // mLastTouchX and mLastTouchY are the point in the current viewport