Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1558)

Unified Diff: chrome/common/extensions/api/_permission_features.json

Issue 1184353005: Add extension IDs for all versions of remoting apps to permissions list of metricsPrivate API. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/common/extensions/api/_permission_features.json
diff --git a/chrome/common/extensions/api/_permission_features.json b/chrome/common/extensions/api/_permission_features.json
index bf904785e5fe52781860db29b1d809d6e925061f..8fed7657783bf71d1eb9627dab7e7459ed4d76ea 100644
--- a/chrome/common/extensions/api/_permission_features.json
+++ b/chrome/common/extensions/api/_permission_features.json
@@ -688,7 +688,14 @@
"896B85CC7E913E11C34892C1425A093C0701D386", // http://crbug.com/456214
"11A01C82EF355E674E4F9728A801F5C3CB40D83F", // http://crbug.com/456214
"F410C88469990EE7947450311D24B8AF2ADB2595", // http://crbug.com/456214
- "9E527CDA9D7C50844E8A5DB964A54A640AE48F98" // Chrome Remote Desktop
+ // Chrome Remote Desktop (CRD) stable:
not at google - send to devlin 2015/06/16 18:25:59 "stable"?
anandc 2015/06/16 18:57:35 There are different versions of the CRD extension.
not at google - send to devlin 2015/06/16 19:02:34 Yes "release" would be more accurate. Or just remo
anandc 2015/06/16 19:08:28 Makes sense. Done.
+ "9E527CDA9D7C50844E8A5DB964A54A640AE48F98",
+ "DF52618D0B040D8A054D8348D2E84DDEEE5974E7", // CRD QA
+ "269D721F163E587BC53C6F83553BF9CE2BB143CD", // CRD QA Backup
+ "6B55A5329E3F1F30F6032BDB20B2EB4378DBF767", // CRD QA v2
+ "C449A798C495E6CF7D6AF10162113D564E67AD12", // CRD Dogfood
+ "01E9FFA9A8F3C18271FE91BEE46207F3B81755CC", // Pre-release CRD
+ "97B23E01B2AA064E8332EE43A7A85C628AADC3F2" // Sample app_remoting app.
not at google - send to devlin 2015/06/16 18:25:59 sample app?
anandc 2015/06/16 18:57:35 Yes. See here: https://code.google.com/p/chromium/
not at google - send to devlin 2015/06/16 19:02:34 Mm ok my obtuse comment was more "if this is a sam
anandc 2015/06/16 19:08:28 I see. The sample app ID is used extensively for t
]
},
"mdns": [{
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698