commit 29ccf8a32bf65a5512191d3633353132bb2cd88b
author Michael Kolb <kolby@google.com> Wed Feb 23 16:13:24 2011 -0800
committer Michael Kolb <kolby@google.com> Wed Feb 23 17:03:38 2011 -0800
tree 87a20aded77d8c2c3b4be6ce25c73f9f6f91302f
parent 17e78fd06905d090cfab4c4f7313bd4b3eec5473

use title height without progress bar

    Bug: 3473996
    let the titlebar decide how high it is
    work around View.getHeight being final
    depends on I6e9a85d88eba176886e53b260d02082d26b410d8

Change-Id: I87a1acbddbb71637979491c40858a3ee22a230b5

src/com/android/browser/ScrollWebView.java

diff --git a/src/com/android/browser/ScrollWebView.java b/src/com/android/browser/ScrollWebView.java
index 1d7f23a..2bf07e1 100644
--- a/src/com/android/browser/ScrollWebView.java
+++ b/src/com/android/browser/ScrollWebView.java
@@ -33,6 +33,7 @@
     private boolean mIsCancelled;
     private boolean mBackgroundRemoved = false;
     private boolean mUserInitiated = false;
+    private TitleBarBase mTitleBar;
 
     /**
      * @param context
@@ -70,6 +71,11 @@
         super(context);
     }
 
+    @Override
+    protected int getTitleHeight() {
+        return (mTitleBar != null) ? mTitleBar.getEmbeddedHeight() : 0;
+    }
+
     // scroll runnable implementation
     public void run() {
         if (!mIsCancelled && (mScrollListener != null)) {
@@ -84,6 +90,7 @@
     @Override
     public void setEmbeddedTitleBar(final View title) {
         super.setEmbeddedTitleBar(title);
+        mTitleBar = (TitleBarBase) title;
         if (title != null && mScrollListener != null) {
             // allow the scroll listener to initialize its state
             post(this);