commit 8a66078a13cf74fe6b44380b4ee6d28426776a3f
author Vivek Sekhar <vsekhar@codeaurora.org> Mon Oct 27 17:00:53 2014 -0700
committer WebTech Code Review <code-review@localhost> Thu Oct 30 08:09:10 2014 -0700
tree 04c50425637d2a058a7e9831bc07fcb01dfc05e1
parent 0e10a20dff43af1c6111b8df606d1a83b51d51d2

Fixes to title bar using top controls

- Do not differentiate between title bar in fixed vs auto-hide mode
  using different android layouts. Use top controls to always show
  titlebar for fixed titlebar.
- Always show the titlebar when page is being loaded even if the user
  scrolls during the page load.
- Fixes the jankiness seen in fixed titlebar mode when user navigates
  between webpages.
- Reaffirm top controls when user switches between fixed and auto-hide
  titlebar mode.

Change-Id: I2531695f34e725ae444f6159645c7557dec69fab

src/com/android/browser/BaseUi.java

diff --git a/src/com/android/browser/BaseUi.java b/src/com/android/browser/BaseUi.java
index fbb5ad0..1ea1460 100644
--- a/src/com/android/browser/BaseUi.java
+++ b/src/com/android/browser/BaseUi.java
@@ -819,8 +819,8 @@
             } else {
                 mTitleBar.setEnabled(true);
             }
-
-            mTitleBar.setTranslationY(topControlsOffsetYPix);
+            if (!mTitleBar.isFixed() && !isLoading())
+                mTitleBar.setTranslationY(topControlsOffsetYPix);
         }
     }
 
@@ -898,10 +898,6 @@
 
     }
 
-    public void addFixedTitleBar(View view) {
-        mFixedTitlebarContainer.addView(view);
-    }
-
     public void setContentViewMarginTop(int margin) {
         LinearLayout.LayoutParams params =
                 (LinearLayout.LayoutParams) mContentView.getLayoutParams();