Remove support for the keygen tag

This removes support for <keygen> by updating it
to be an HTMLUnknownElement. As a result, it's
no longer a form-associated element and no
longer has IDL-assigned properties.

The <keygen> tag is still left in the parser,
similar to <applet>, so that it maintains the
document parse behaviours (such as self-closing),
but is otherwise a neutered element.

Tests that were relying on <keygen> having its
own browser-created shadow root (for its custom
select element) have been updated to use
progress bars, while other tests (such as
<keygen>-related crash tests) have been
fully removed.

As Blink no longer treats this tag as special,
all the related IPC infrastructure is removed,
including preferences and enterprise flags,
and all localized strings, as they're all now
unreachable.

This concludes the "Intent to Remove" thread
for <keygen> at
https://groups.google.com/a/chromium.org/d/msg/blink-dev/z_qEpmzzKh8/BH-lkwdgBAAJ

BUG=568184
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2536993002
Cr-Commit-Position: refs/heads/master@{#440245}

Author: sleevi

android_webview/browser/aw_resource_context.h

@@ -12,6 +12,8 @@
 #include "base/synchronization/lock.h"
 #include "content/public/browser/resource_context.h"
 
+class GURL;
+
 namespace net {
 class URLRequestContextGetter;
 }

chrome/android/java/res/xml/single_website_preferences.xml

@@ -47,8 +47,6 @@
         android:key="popup_permission_list" />
     <org.chromium.chrome.browser.preferences.ChromeBaseListPreference
         android:key="background_sync_permission_list" />
-    <org.chromium.chrome.browser.preferences.ChromeBaseListPreference
-        android:key="keygen_permission_list" />
     <org.chromium.chrome.browser.preferences.ChromeBaseListPreference
         android:key="protected_media_identifier_permission_list" />
     <org.chromium.chrome.browser.preferences.ChromeBaseListPreference

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

@@ -118,12 +118,6 @@ private static Map<Integer, ResourceItem> getResourceInfo() {
                                  R.string.javascript_permission_title, ContentSetting.ALLOW,
                                  ContentSetting.BLOCK,
                                  R.string.website_settings_category_javascript_allowed, 0));
-            localMap.put(ContentSettingsType.CONTENT_SETTINGS_TYPE_KEYGEN,
-                    new ResourceItem(R.drawable.permission_keygen,
-                                 R.string.keygen_permission_title,
-                                 R.string.keygen_permission_title, ContentSetting.ALLOW,
-                                 ContentSetting.BLOCK,
-                                 0, R.string.website_settings_category_blocked_recommended));
             localMap.put(
                     ContentSettingsType.CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA,
                     new ResourceItem(R.drawable.permission_camera,

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

@@ -73,7 +73,6 @@ public class SingleWebsitePreferences extends PreferenceFragment
     public static final String PREF_CAMERA_CAPTURE_PERMISSION = "camera_permission_list";
     public static final String PREF_COOKIES_PERMISSION = "cookies_permission_list";
     public static final String PREF_JAVASCRIPT_PERMISSION = "javascript_permission_list";
-    public static final String PREF_KEYGEN_PERMISSION = "keygen_permission_list";
     public static final String PREF_LOCATION_ACCESS = "location_access_list";
     public static final String PREF_MIC_CAPTURE_PERMISSION = "microphone_permission_list";
     public static final String PREF_MIDI_SYSEX_PERMISSION = "midi_sysex_permission_list";
@@ -90,7 +89,6 @@ public class SingleWebsitePreferences extends PreferenceFragment
             PREF_CAMERA_CAPTURE_PERMISSION,
             PREF_COOKIES_PERMISSION,
             PREF_JAVASCRIPT_PERMISSION,
-            PREF_KEYGEN_PERMISSION,
             PREF_LOCATION_ACCESS,
             PREF_MIC_CAPTURE_PERMISSION,
             PREF_MIDI_SYSEX_PERMISSION,
@@ -125,13 +123,6 @@ public void onWebsitePermissionsAvailable(Collection<Website> sites) {
             // TODO(mvanouwerkerk): Avoid modifying the outer class from this inner class.
             mSite = mergePermissionInfoForTopLevelOrigin(mSiteAddress, sites);
 
-            // Display Keygen Content Setting if Keygen is blocked.
-            if (mSite.getKeygenInfo() == null && mWebContents != null
-                    && WebsitePreferenceBridge.getKeygenBlocked(mWebContents)) {
-                String origin = mSiteAddress.getOrigin();
-                mSite.setKeygenInfo(new KeygenInfo(origin, origin, false));
-            }
-
             displaySitePermissions();
         }
     }
@@ -204,10 +195,6 @@ private static Website mergePermissionInfoForTopLevelOrigin(
                     && permissionInfoIsForTopLevelOrigin(other.getGeolocationInfo(), origin)) {
                 merged.setGeolocationInfo(other.getGeolocationInfo());
             }
-            if (merged.getKeygenInfo() == null && other.getKeygenInfo() != null
-                    && permissionInfoIsForTopLevelOrigin(other.getKeygenInfo(), origin)) {
-                merged.setKeygenInfo(other.getKeygenInfo());
-            }
             if (merged.getMidiInfo() == null && other.getMidiInfo() != null
                     && permissionInfoIsForTopLevelOrigin(other.getMidiInfo(), origin)) {
                 merged.setMidiInfo(other.getMidiInfo());
@@ -304,8 +291,6 @@ private void displaySitePermissions() {
                 setUpListPreference(preference, mSite.getCookiePermission());
             } else if (PREF_JAVASCRIPT_PERMISSION.equals(preference.getKey())) {
                 setUpListPreference(preference, mSite.getJavaScriptPermission());
-            } else if (PREF_KEYGEN_PERMISSION.equals(preference.getKey())) {
-                setUpListPreference(preference, mSite.getKeygenPermission());
             } else if (PREF_LOCATION_ACCESS.equals(preference.getKey())) {
                 setUpLocationPreference(preference);
             } else if (PREF_MIC_CAPTURE_PERMISSION.equals(preference.getKey())) {
@@ -537,8 +522,6 @@ private int getContentSettingsTypeFromPreferenceKey(String preferenceKey) {
                 return ContentSettingsType.CONTENT_SETTINGS_TYPE_COOKIES;
             case PREF_JAVASCRIPT_PERMISSION:
                 return ContentSettingsType.CONTENT_SETTINGS_TYPE_JAVASCRIPT;
-            case PREF_KEYGEN_PERMISSION:
-                return ContentSettingsType.CONTENT_SETTINGS_TYPE_KEYGEN;
             case PREF_LOCATION_ACCESS:
                 return ContentSettingsType.CONTENT_SETTINGS_TYPE_GEOLOCATION;
             case PREF_MIC_CAPTURE_PERMISSION:
@@ -599,8 +582,6 @@ public boolean onPreferenceChange(Preference preference, Object newValue) {
             mSite.setCookiePermission(permission);
         } else if (PREF_JAVASCRIPT_PERMISSION.equals(preference.getKey())) {
             mSite.setJavaScriptPermission(permission);
-        } else if (PREF_KEYGEN_PERMISSION.equals(preference.getKey())) {
-            mSite.setKeygenPermission(permission);
         } else if (PREF_LOCATION_ACCESS.equals(preference.getKey())) {
             mSite.setGeolocationPermission(permission);
         } else if (PREF_MIC_CAPTURE_PERMISSION.equals(preference.getKey())) {
@@ -679,7 +660,6 @@ protected void resetSite() {
         mSite.setCookiePermission(ContentSetting.DEFAULT);
         mSite.setGeolocationPermission(ContentSetting.DEFAULT);
         mSite.setJavaScriptPermission(ContentSetting.DEFAULT);
-        mSite.setKeygenPermission(ContentSetting.DEFAULT);
         mSite.setMicrophonePermission(ContentSetting.DEFAULT);
         mSite.setMidiPermission(ContentSetting.DEFAULT);
         mSite.setNotificationPermission(ContentSetting.DEFAULT);

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

@@ -32,7 +32,6 @@ public class Website implements Serializable {
     private ContentSettingException mCookieException;
     private GeolocationInfo mGeolocationInfo;
     private ContentSettingException mJavaScriptException;
-    private KeygenInfo mKeygenInfo;
     private LocalStorageInfo mLocalStorageInfo;
     private MicrophoneInfo mMicrophoneInfo;
     private MidiInfo mMidiInfo;
@@ -235,33 +234,6 @@ public void setJavaScriptException(ContentSettingException exception) {
         mJavaScriptException = exception;
     }
 
-    /**
-     * Sets the KeygenInfo object for this Website.
-     */
-    public void setKeygenInfo(KeygenInfo info) {
-        mKeygenInfo = info;
-    }
-
-    public KeygenInfo getKeygenInfo() {
-        return mKeygenInfo;
-    }
-
-    /**
-     * Returns what permission governs keygen access.
-     */
-    public ContentSetting getKeygenPermission() {
-        return mKeygenInfo != null ? mKeygenInfo.getContentSetting() : null;
-    }
-
-    /**
-     * Configure keygen access setting for this site.
-     */
-    public void setKeygenPermission(ContentSetting value) {
-        if (mKeygenInfo != null) {
-            mKeygenInfo.setContentSetting(value);
-        }
-    }
-
     /**
      * Sets microphone capture info class.
      */

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

@@ -55,8 +55,6 @@ public void fetchAllPreferences() {
         queue.add(new MidiInfoFetcher());
         // Cookies are stored per-host.
         queue.add(new CookieExceptionInfoFetcher());
-        // Keygen permissions are per-origin.
-        queue.add(new KeygenInfoFetcher());
         // Local storage info is per-origin.
         queue.add(new LocalStorageInfoFetcher());
         // Website storage is per-host.
@@ -269,18 +267,6 @@ public void run() {
         }
     }
 
-    private class KeygenInfoFetcher extends Task {
-        @Override
-        public void run() {
-            for (KeygenInfo info : WebsitePreferenceBridge.getKeygenInfo()) {
-                WebsiteAddress origin = WebsiteAddress.create(info.getOrigin());
-                if (origin == null) continue;
-                WebsiteAddress embedder = WebsiteAddress.create(info.getEmbedder());
-                findOrCreateSite(origin, embedder).setKeygenInfo(info);
-            }
-        }
-    }
-
     private class LocalStorageInfoFetcher extends Task {
         @Override
         public void runAsync(final TaskQueue queue) {

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

@@ -7,7 +7,6 @@
 import org.chromium.base.Callback;
 import org.chromium.base.annotations.CalledByNative;
 import org.chromium.chrome.browser.preferences.PrefServiceBridge;
-import org.chromium.content_public.browser.WebContents;
 
 import java.util.ArrayList;
 import java.util.HashMap;
@@ -27,30 +26,6 @@ public interface StorageInfoClearedCallback {
         public void onStorageInfoCleared();
     }
 
-    /**
-     * @return the list of all origins that have keygen permissions in non-incognito mode.
-     */
-    @SuppressWarnings("unchecked")
-    public static List<KeygenInfo> getKeygenInfo() {
-        ArrayList<KeygenInfo> list = new ArrayList<KeygenInfo>();
-        nativeGetKeygenOrigins(list);
-        return list;
-    }
-
-    @CalledByNative
-    private static void insertKeygenInfoIntoList(
-            ArrayList<KeygenInfo> list, String origin, String embedder) {
-        list.add(new KeygenInfo(origin, embedder, false));
-    }
-
-    /**
-     * @return whether we've blocked key generation in the current tab.
-     */
-    @SuppressWarnings("unchecked")
-    public static boolean getKeygenBlocked(WebContents webContents) {
-        return nativeGetKeygenBlocked(webContents);
-    }
-
     /**
      * @return the list of all origins that have geolocation permissions in non-incognito mode.
      */
@@ -247,12 +222,6 @@ static native int nativeGetGeolocationSettingForOrigin(
             String origin, String embedder, boolean isIncognito);
     public static native void nativeSetGeolocationSettingForOrigin(
             String origin, String embedder, int value, boolean isIncognito);
-    private static native void nativeGetKeygenOrigins(Object list);
-    static native int nativeGetKeygenSettingForOrigin(
-            String origin, String embedder, boolean isIncognito);
-    static native void nativeSetKeygenSettingForOrigin(
-            String origin, int value, boolean isIncognito);
-    private static native boolean nativeGetKeygenBlocked(Object webContents);
     private static native void nativeGetMidiOrigins(Object list);
     static native int nativeGetMidiSettingForOrigin(
             String origin, String embedder, boolean isIncognito);

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

@@ -596,9 +596,6 @@ Your Google account may have other forms of browsing history like searches and a
       <message name="IDS_JAVASCRIPT_PERMISSION_TITLE" desc="Title of the permission to run javascript [CHAR-LIMIT=32]">
         JavaScript
       </message>
-      <message name="IDS_KEYGEN_PERMISSION_TITLE" desc="Title of the permission to allow a form to perform client-side key generation [CHAR-LIMIT=32]">
-        Key generation
-      </message>
       <message name="IDS_MEDIA_PERMISSION_TITLE" desc="Title of the menu containing the media permissions [CHAR-LIMIT=32]">
         Media
       </message>

chrome/android/java/strings/android_chrome_strings.grd

@@ -830,7 +830,6 @@ chrome_java_sources = [
   "java/src/org/chromium/chrome/browser/preferences/website/ContentSettingException.java",
   "java/src/org/chromium/chrome/browser/preferences/website/ContentSettingsResources.java",
   "java/src/org/chromium/chrome/browser/preferences/website/GeolocationInfo.java",
-  "java/src/org/chromium/chrome/browser/preferences/website/KeygenInfo.java",
   "java/src/org/chromium/chrome/browser/preferences/website/LocalStorageInfo.java",
   "java/src/org/chromium/chrome/browser/preferences/website/LocationCategory.java",
   "java/src/org/chromium/chrome/browser/preferences/website/ManageSpaceActivity.java",