Adding <keygen> Content Setting (Android UI)

chrome/android/java/src/org/chromium/chrome/browser/preferences/website/ContentSettingsResources.java

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.chrome.browser.preferences.website;
 
 import org.chromium.chrome.R;
 import org.chromium.chrome.browser.ContentSettingsType;
 
 /**
  * A class with utility functions that get the appropriate string and icon resources for the
  * Android UI that allows managing content settings.
  */
 public class ContentSettingsResources {
     /**
      * Returns the resource id of the icon for a content type.
      */
     public static int getIcon(int contentType) {
         switch (contentType) {
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_COOKIES:
                 return R.drawable.permission_cookie;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_FULLSCREEN:
                 return R.drawable.permission_fullscreen;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_GEOLOCATION:
                 return R.drawable.permission_location;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_JAVASCRIPT:
                 return R.drawable.permission_javascript;
+            case ContentSettingsType.CONTENT_SETTINGS_TYPE_KEYGEN:
+                return R.drawable.permission_javascript;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA:
                 return R.drawable.permission_camera;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC:
                 return R.drawable.permission_mic;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_MIDI_SYSEX:
                 return R.drawable.permission_midi;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_NOTIFICATIONS:
                 return R.drawable.permission_push_notification;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_POPUPS:
                 return R.drawable.permission_popups;
@@ skipping 11 matching lines @@
     public static int getTitle(int contentType) {
         switch (contentType) {
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_COOKIES:
                 return R.string.cookies_title;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_FULLSCREEN:
                 return R.string.website_settings_fullscreen;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_GEOLOCATION:
                 return R.string.website_settings_device_location;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_JAVASCRIPT:
                 return R.string.javascript_permission_title;
+            case ContentSettingsType.CONTENT_SETTINGS_TYPE_KEYGEN:
+                return R.string.keygen_permission_title;

[Finnur, 2015/11/11 10:25:45]

This is not required by MIDI, so it should not be required of Keygen.

[svaldez, 2015/11/19 17:50:25]

Done.

             case ContentSettingsType.CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA:
                 return R.string.website_settings_use_camera;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC:
                 return R.string.website_settings_use_mic;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_NOTIFICATIONS:
                 return R.string.push_notifications_permission_title;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_POPUPS:
                 return R.string.popup_permission_title;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_PROTECTED_MEDIA_IDENTIFIER:
                 return org.chromium.chrome.R.string.protected_content;
             default:
                 return 0;
         }
     }
 
     /**
      * Returns the resource id of the title explanation, shown on the Website Details page for
      * a content type.
      */
     public static int getExplanation(int contentType) {
         switch (contentType) {
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_COOKIES:
                 return R.string.cookies_title;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_FULLSCREEN:
                 return R.string.fullscreen_permission_title;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_GEOLOCATION:
                 return R.string.geolocation_permission_title;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_JAVASCRIPT:
                 return R.string.javascript_permission_title;
+            case ContentSettingsType.CONTENT_SETTINGS_TYPE_KEYGEN:
+                return R.string.keygen_permission_title;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA:
                 return R.string.camera_permission_title;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC:
                 return R.string.mic_permission_title;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_MIDI_SYSEX:
                 return R.string.midi_sysex_permission_title;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_NOTIFICATIONS:
                 return R.string.push_notifications_permission_title;
             case ContentSettingsType.CONTENT_SETTINGS_TYPE_POPUPS:
                 return R.string.popup_permission_title;
@@ skipping 132 matching lines @@
     }
 
     /**
      * Returns the summary for Cookie content settings when it is allowed
      * except for those from third party sources.
      */
     public static int getCookieAllowedExceptThirdPartySummary() {
         return R.string.website_settings_category_allowed_except_third_party;
     }
 }