Bug 944537 - No need to check for showProgress in BrowserToolbar (r=mfinkle)
authorLucas Rocha <lucasr@mozilla.com>
Tue, 03 Dec 2013 16:20:31 +0000
changeset 174128 918596edaa773a76b337c93f3370d706b85032cd
parent 174127 a1445386e051b28068f11bb6663a9be12d5a7187
child 174129 8f06133698b40e4f899aa134c39b095e6899f0bd
push id445
push userffxbld
push dateMon, 10 Mar 2014 22:05:19 +0000
treeherdermozilla-release@dc38b741b04e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmfinkle
bugs944537
milestone28.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 944537 - No need to check for showProgress in BrowserToolbar (r=mfinkle)
mobile/android/base/toolbar/BrowserToolbar.java
--- a/mobile/android/base/toolbar/BrowserToolbar.java
+++ b/mobile/android/base/toolbar/BrowserToolbar.java
@@ -561,18 +561,17 @@ public class BrowserToolbar extends Geck
             switch (msg) {
                 case TITLE:
                     updateTitle();
                     break;
 
                 case START:
                     updateBackButton(canDoBack(tab));
                     updateForwardButton(canDoForward(tab));
-                    Boolean showProgress = (Boolean)data;
-                    if (showProgress && tab.getState() == Tab.STATE_LOADING) {
+                    if (tab.getState() == Tab.STATE_LOADING) {
                         setProgressVisibility(true);
                     }
                     setSecurityMode(tab.getSecurityMode());
                     setPageActionVisibility(mStop.getVisibility() == View.VISIBLE);
                     break;
 
                 case STOP:
                     updateBackButton(canDoBack(tab));