Bug 917947: SearchEngine icons are small in the dialog. [r=mfinkle] [r=margaret]
authorSriram Ramasubramanian <sriram@mozilla.com>
Mon, 14 Oct 2013 13:52:23 -0700
changeset 164804 effaf67d0336b1587044fa3de4bdddc74ec26ac3
parent 164803 7a6e64677565d19ee58848f8f6613e6de45a2eba
child 164805 4291e6d212060c954cabc19548bafcb039202eb1
push id3066
push userakeybl@mozilla.com
push dateMon, 09 Dec 2013 19:58:46 +0000
treeherdermozilla-beta@a31a0dce83aa [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmfinkle, margaret
bugs917947
milestone27.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 917947: SearchEngine icons are small in the dialog. [r=mfinkle] [r=margaret]
mobile/android/base/preferences/SearchEnginePreference.java
--- a/mobile/android/base/preferences/SearchEnginePreference.java
+++ b/mobile/android/base/preferences/SearchEnginePreference.java
@@ -202,17 +202,19 @@ public class SearchEnginePreference exte
             }
         });
 
         // Copy the icon from this object to the prompt we produce. We lazily create the drawable,
         // as the user may not ever actually tap this object.
         if (mPromptIcon == null && mIconBitmap != null) {
             mPromptIcon = new BitmapDrawable(mFaviconView.getBitmap());
         }
-        builder.setIcon(mPromptIcon);
+
+        // Icons are hidden until Bug 926711 is fixed.
+        //builder.setIcon(mPromptIcon);
 
         // We have to construct the dialog itself on the UI thread.
         ThreadUtils.postToUiThread(new Runnable() {
             @Override
             public void run() {
                 mDialog = builder.create();
                 mDialog.setOnShowListener(new DialogInterface.OnShowListener() {
                     // Called when the dialog is shown (so we're finally able to manipulate button enabledness).