Bug 696550: Favicon should be reset in the Tab, when there is a location change [r=mfinkle]

embedding/android/GeckoApp.java

@@ -540,7 +540,7 @@ abstract public class GeckoApp
     }
 
     void handleLocationChange(final int tabId, final String uri) {
-        Tab tab = Tabs.getInstance().getTab(tabId);
+        final Tab tab = Tabs.getInstance().getTab(tabId);
         if (tab == null)
             return;
         
@@ -564,6 +564,7 @@ abstract public class GeckoApp
             public void run() {
                 mBrowserToolbar.setTitle(uri);
                 mBrowserToolbar.setFavicon(null);
+                tab.updateFavicon(null);
             }
         });
     }

embedding/android/Tab.java

@@ -139,10 +139,8 @@ public class Tab {
     }
 
     public void updateFavicon(Drawable favicon) {
-        if (favicon != null) {
-            this.favicon = favicon;
-            Log.i(LOG_FILE_NAME, "Updated favicon for tab with id: " + this.id);
-        }
+        this.favicon = favicon;
+        Log.i(LOG_FILE_NAME, "Updated favicon for tab with id: " + this.id);
     }
 
     public boolean doReload() {