Bug 810342 - Show toolbar shadow on about:blank (r=lucasr)
authorShriram Kunchanapalli <kshriram18@gmail.com>
Thu, 10 Jan 2013 01:34:13 +0530
changeset 118409 d70c9eb65c754602fef64c8e64af95597e3ecb66
parent 118408 927268cb78116fd88bc8a8ad9b72304d509f4f4c
child 118410 9f1d90aba921ee9dd7aab50faf299f319070b135
push id24166
push userMs2ger@gmail.com
push dateFri, 11 Jan 2013 13:57:41 +0000
treeherdermozilla-central@63c4b0f66a0c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerslucasr
bugs810342
milestone21.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 810342 - Show toolbar shadow on about:blank (r=lucasr)
mobile/android/base/BrowserToolbar.java
--- a/mobile/android/base/BrowserToolbar.java
+++ b/mobile/android/base/BrowserToolbar.java
@@ -922,18 +922,19 @@ public class BrowserToolbar implements V
     public void setShadowVisibility(boolean visible) {
         Tab tab = Tabs.getInstance().getSelectedTab();
         if (tab == null) {
             return;
         }
 
         String url = tab.getURL();
 
-        // Only set shadow to visible when not on about screens.
-        visible &= !(url == null || url.startsWith("about:"));
+        // Only set shadow to visible when not on about screens except about:blank.
+        visible &= !(url == null || (url.startsWith("about:") && 
+                     !url.equals("about:blank")));
 
         if ((mShadow.getVisibility() == View.VISIBLE) != visible) {
             mShadow.setVisibility(visible ? View.VISIBLE : View.GONE);
         }
     }
 
     public void setTitle(CharSequence title) {
         Tab tab = Tabs.getInstance().getSelectedTab();