commit 60eb980eb398c5459fbc0e72d141a58161e7d3be
author Vivek Sekhar <vsekhar@codeaurora.org> Mon Jul 21 19:13:33 2014 -0700
committer Vivek Sekhar <vsekhar@codeaurora.org> Wed Jul 30 12:18:35 2014 -0700
tree 91151ecec515d7e74eb59067e0d2ccf765215cfb
parent 4ddafb1795f2199b6e3e16ff2d706433db533cca

Fix issues in Autohide TitleBar

- Fixes issue when longpress of TitleBar
  shows copy/paste menu on top of titleBar.
- Fixes issue in landscape mode where the
  setTranslationY value is reused from portrait
  mode.

Change-Id: I521b4dd7a5385c621c941487f77894a3dcef42d3

src/com/android/browser/BaseUi.java

diff --git a/src/com/android/browser/BaseUi.java b/src/com/android/browser/BaseUi.java
index 4648de7..c6238f4 100644
--- a/src/com/android/browser/BaseUi.java
+++ b/src/com/android/browser/BaseUi.java
@@ -855,7 +855,7 @@
     }
 
     public void transalateTitleBar(float topControlsOffsetYPix) {
-        if (mTitleBar != null) {
+        if (mTitleBar != null && !mInActionMode) {
             mTitleBar.bringToFront();
             if (topControlsOffsetYPix != 0.0) {
                 mTitleBar.setEnabled(false);
@@ -863,7 +863,6 @@
                 mTitleBar.setEnabled(true);
             }
             mTitleBar.setTranslationY(topControlsOffsetYPix);
-
         }
     }
 
@@ -968,8 +967,11 @@
         return mUiController;
     }
 
+    boolean mInActionMode = false;
+
     @Override
     public void onActionModeStarted(ActionMode mode) {
+        mInActionMode = true;
         boolean hide_title_on_scroll =
             mActivity.getResources().getBoolean(R.bool.hide_title_on_scroll);
         if (!hide_title_on_scroll) {
@@ -981,6 +983,7 @@
 
     @Override
     public void onActionModeFinished(boolean inLoad) {
+        mInActionMode = false;
         boolean hide_title_on_scroll =
             mActivity.getResources().getBoolean(R.bool.hide_title_on_scroll);
         if (!hide_title_on_scroll) {