Let Android port access properly sized images

chrome/android/java/src/org/chromium/chrome/browser/WebsiteSettingsPopup.java

Index: chrome/android/java/src/org/chromium/chrome/browser/WebsiteSettingsPopup.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/WebsiteSettingsPopup.java b/chrome/android/java/src/org/chromium/chrome/browser/WebsiteSettingsPopup.java
index 940596453861ee48a1201c40619844869de6ff51..3c7e6db019a21c0c06875a5e701f8f35ca4c0690 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/WebsiteSettingsPopup.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/WebsiteSettingsPopup.java
@@ -65,10 +65,12 @@ class WebsiteSettingsPopup implements OnClickListener {
 
     /** Adds a section, which contains an icon, a headline, and a description. */
     @CalledByNative
-    private void addSection(Bitmap icon, String headline, String description) {
+    private void addSection(int enumeratedIconID, String headline, String description) {

[Yaron, 2013/05/17 22:36:16]

Nit: "ID" should be "Id" througout

[gone, 2013/05/20 18:48:10]

Usage in the codebase seems really inconsistent :/

[Yaron, 2013/05/20 20:54:25]

Cases with ID are wrong:
http://source.android.com/source/code-style.html#treat-acronyms-as-words (also
according to internal style guide)

         View section = LayoutInflater.from(mContext).inflate(R.layout.website_settings, null);
         ImageView i = (ImageView) section.findViewById(R.id.website_settings_icon);
-        i.setImageBitmap(icon);
+        int drawableID = ResourceID.mapToDrawableID(enumeratedIconID);
+        i.setImageResource(drawableID);
+
         TextView h = (TextView) section.findViewById(R.id.website_settings_headline);
         h.setText(headline);
         if (TextUtils.isEmpty(headline)) h.setVisibility(View.GONE);