Whitelist MR extensions for settingsPrivate API.

chrome/common/extensions/api/_permission_features.json

 // Copyright (c) 2012 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.
 
 // This features file defines permissions for extension APIs implemented
 // under src/chrome.
 // See extensions/common/features/* to understand this file, in particular
 // feature.h, simple_feature.h, and base_feature_provider.h.
 
 // To add a new whitelisted ID, SHA-1 it and force it to uppercase. In Bash:
@@ skipping 906 matching lines @@
     "location": "component"
   },
   "screensaver": {
     "channel": "stable",
     "extension_types": ["legacy_packaged_app", "hosted_app", "platform_app"]
   },
   "sessions": {
     "channel": "stable",
     "extension_types": ["extension", "legacy_packaged_app"]
   },
-  "settingsPrivate": {
-    "channel": "trunk",
-    "extension_types": ["extension", "platform_app"],
-    "location": "component"
-  },
+  "settingsPrivate": [
+    {
+      "channel": "trunk",
+      "extension_types": ["extension", "platform_app"],
+      "location": "component"
+    },
+    {
+      "channel": "trunk",

[Devlin, 2016/01/28 19:04:11]

Assuming you want to use this anywhere other than trunk, this will need to be
updated (if it's your intent to only use it on trunk for now, then this is
fine).

[amp, 2016/01/28 19:38:35]

Yes, we will want to have this released to stable for M50.  Is it ok to set this
to stable here since it's now limited by a whitelist, or do we need further
approval to do that?  I've set it to stable for now.

+      "extension_types": ["extension", "platform_app"],

[amp, 2016/01/28 17:59:11]

I tried adding a 
"location": "external_component",
line to the second entry here, but it gave me an error when I tried it to use
the API so I have left it with just the whitelist for now.

[Devlin, 2016/01/28 19:04:11]

Huh.  Interesting.  With a whitelist, it doesn't really matter, but I still
would have expected it to work.  Are you sure that on your dev build MR is added
as an external component (don't sweat it if it's a pain to debug).

All that said, please do remove "platform_app" here.

[amp, 2016/01/28 19:38:35]

Right, thanks for the clarification. My dev build is just a regular extension so
it would fail.  The release extension is an external_component but that would be
harder to test with this limitation.

And platform_app should be removed in latest patch.

+      "whitelist": [
+        "63ED55E43214C211F82122ED56407FF1A807F2A3",  // Media Router Dev
+        "226CF815E39A363090A1E547D53063472B8279FA"   // Media Router Release
+      ]
+    }
+  ],
   "signedInDevices": {
     "channel": "dev",
     "extension_types": ["extension", "legacy_packaged_app", "platform_app"]
   },
   "streamsPrivate": {
     "channel": "stable",
     "extension_types": ["extension", "legacy_packaged_app", "platform_app"],
     "whitelist": [
       "787000072C6FBB934AF5A42275CDE73FC977D995",  // browser_tests
       "2FC374607C2DF285634B67C64A2E356C607091C3",  // QuickOffice
@@ skipping 118 matching lines @@
     "channel": "stable",
     "extension_types": ["platform_app"],
     "platforms": ["chromeos"],
     "whitelist": [
       "A948368FC53BE437A55FEB414106E207925482F5",  // Files app
       "8C726564C6DBE7380BAB388DE6CC7EC93EB44B06",  // Video player
       "5065C83E84321221675D20FBE4DC43C655CC8C6F"   // Files app tests
     ]
   }
 }