fix anchor link url bar updates

    Bug: 5621026

Allow onPageFinished to reach Controller for links with anchor tags
Note that the bookmark thumbnail update won't overload the
async task anymore, since only one update message is active at any given
time (see bug: 4517571)

Change-Id: Iaeb9950ef15446aa9c824c7196f26a748591c944

2 files changed, 2 insertions, 7 deletions

diff --git a/src/com/android/browser/Controller.java b/src/com/android/browser/Controller.java
index 01e090b92..67be22e9c 100644
--- a/src/com/android/browser/Controller.java
+++ b/src/com/android/browser/Controller.java
@@ -842,7 +842,8 @@ public class Controller
         mUi.onTabDataChanged(tab);
         if (!tab.isPrivateBrowsingEnabled()
                 && !TextUtils.isEmpty(tab.getUrl())
-                && !tab.isSnapshot()) {
+                && !tab.isSnapshot()
+                && !tab.inPageLoad()) {
             // Only update the bookmark screenshot if the user did not
             // cancel the load early and there is not already
             // a pending update for the tab.
@@ -2011,7 +2012,6 @@ public class Controller
         }
         final String url = tab.getUrl();
         final String originalUrl = view.getOriginalUrl();
-
         if (TextUtils.isEmpty(url)) {
             return;
         }
diff --git a/src/com/android/browser/Tab.java b/src/com/android/browser/Tab.java
index 0abc86bdf..a7598c2c6 100644
--- a/src/com/android/browser/Tab.java
+++ b/src/com/android/browser/Tab.java
@@ -610,11 +610,6 @@ class Tab implements PictureListener {
 
         @Override
         public void onPageFinished(WebView view, String url) {
-            if (!mInPageLoad) {
-                // In page navigation links (www.something.com#footer) will
-                // trigger an onPageFinished which we don't care about.
-                return;
-            }
             if (!isPrivateBrowsingEnabled()) {
                 LogTag.logPageFinishedLoading(
                         url, SystemClock.uptimeMillis() - mLoadStartTime);