Issue 2606 - Firefox logo shown in background on empty Adblock Browser New Tab and Search pages

mobile/android/base/home/PanelAuthLayout.java

 /* -*- Mode: Java; c-basic-offset: 4; tab-width: 20; indent-tabs-mode: nil; -*-
  * This Source Code Form is subject to the terms of the Mozilla Public
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 package org.mozilla.gecko.home;
 
 import org.mozilla.gecko.GeckoAppShell;
 import org.mozilla.gecko.GeckoEvent;
 import org.mozilla.gecko.R;
@@ skipping 35 matching lines @@
             @Override
             public void onClick(View v) {
                 GeckoAppShell.sendEventToGecko(GeckoEvent.createBroadcastEvent("HomePanels:Authenticate", panelId));
             }
         });
 
         final ImageView imageView = (ImageView) findViewById(R.id.image);
         final String imageUrl = authConfig.getImageUrl();
 
         if (TextUtils.isEmpty(imageUrl)) {
+            // Use a default image if an image URL isn't specified.
             // Changed icon. See https://issues.adblockplus.org/ticket/2606

[Felix Dahlke, 2016/11/17 18:38:51]

Nit: Makes more sense to me if you'd move this one line down, i.e. after the
upstream comment. But not important.

-            // Use a default image if an image URL isn't specified.
             imageView.setImageResource(R.drawable.abb_icon_home_empty);
         } else {
             ImageLoader.with(getContext())
                        .load(imageUrl)
                        .into(imageView);
         }
     }
 }