diff options
author | John Reck <jreck@google.com> | 2011-11-14 16:18:50 -0800 |
---|---|---|
committer | John Reck <jreck@google.com> | 2011-11-14 16:18:50 -0800 |
commit | 576c37f1a4e3d978f489956eba4b7c2486ced324 (patch) | |
tree | 840bd63980307815afc77f5c0030846b2da8922b | |
parent | ccd78601a934cf1ffafe5854a1d465c273727f69 (diff) | |
download | packages_apps_browser-576c37f1a4e3d978f489956eba4b7c2486ced324.zip packages_apps_browser-576c37f1a4e3d978f489956eba4b7c2486ced324.tar.gz packages_apps_browser-576c37f1a4e3d978f489956eba4b7c2486ced324.tar.bz2 |
Remove old != new onScrollChanged check
Bug: 5533389
That is now meaningful in that it signifies a layer was scrolled
Change-Id: I24734eb2739c0d020b32d16394c6133f9de990c5
-rw-r--r-- | src/com/android/browser/UrlBarAutoShowManager.java | 22 |
1 files changed, 10 insertions, 12 deletions
diff --git a/src/com/android/browser/UrlBarAutoShowManager.java b/src/com/android/browser/UrlBarAutoShowManager.java index 294115e..9f7827b 100644 --- a/src/com/android/browser/UrlBarAutoShowManager.java +++ b/src/com/android/browser/UrlBarAutoShowManager.java @@ -71,19 +71,17 @@ public class UrlBarAutoShowManager implements OnTouchListener, @Override public void onScrollChanged(int l, int t, int oldl, int oldt) { mLastScrollTime = SystemClock.uptimeMillis(); - if (t != oldt) { - mIsScrolling = true; - if (t != 0) { - // If it is showing, extend it - if (mUi.isTitleBarShowing()) { - long remaining = mLastScrollTime - mTriggeredTime; - remaining = Math.max(BaseUi.HIDE_TITLEBAR_DELAY - remaining, - SCROLL_TIMEOUT_DURATION); - mUi.showTitleBarForDuration(remaining); - } - } else { - mUi.suggestHideTitleBar(); + mIsScrolling = true; + if (t != 0) { + // If it is showing, extend it + if (mUi.isTitleBarShowing()) { + long remaining = mLastScrollTime - mTriggeredTime; + remaining = Math.max(BaseUi.HIDE_TITLEBAR_DELAY - remaining, + SCROLL_TIMEOUT_DURATION); + mUi.showTitleBarForDuration(remaining); } + } else { + mUi.suggestHideTitleBar(); } } |