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

Side by Side Diff: extensions/common/api/_manifest_features.json

Issue 2618493002: Chrome app manifest support for action handlers. (Closed)
Patch Set: Address comments Created 3 years, 11 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // This features file defines manifest keys implemented under src/extensions. 5 // This features file defines manifest keys implemented under src/extensions.
6 // See chrome/common/extensions/api/_features.md to understand this file, as 6 // See chrome/common/extensions/api/_features.md to understand this file, as
7 // well as feature.h, simple_feature.h, and base_feature_provider.h. 7 // well as feature.h, simple_feature.h, and base_feature_provider.h.
8 8
9 { 9 {
10 "action_handlers": {
11 "channel": "stable",
Devlin 2017/01/17 21:42:01 Let's not launch this to stable just yet - it's us
jdufault 2017/01/20 20:31:47 How long of an incubation period? The changes here
Devlin 2017/01/30 21:19:59 Mostly, we want the consumers of this API to have
12 "extension_types": ["platform_app"]
13 },
Devlin 2017/01/17 21:42:01 should we restrict this to chromeos?
jdufault 2017/01/20 20:31:47 Done.
10 "app": { 14 "app": {
11 "channel": "stable", 15 "channel": "stable",
12 "extension_types": ["legacy_packaged_app", "hosted_app", "platform_app"] 16 "extension_types": ["legacy_packaged_app", "hosted_app", "platform_app"]
13 }, 17 },
14 // The default platform app CSP can only be overridden by whitelisted apps. 18 // The default platform app CSP can only be overridden by whitelisted apps.
15 // This is a separate key from the top-level content_security_policy one since 19 // This is a separate key from the top-level content_security_policy one since
16 // we can't combine type restrictions with whitelisted ID restrictions. If 20 // we can't combine type restrictions with whitelisted ID restrictions. If
17 // there is a need for additional whitelisted entries, the feature system 21 // there is a need for additional whitelisted entries, the feature system
18 // should instead be extended to support OR-ing of restrictions. 22 // should instead be extended to support OR-ing of restrictions.
19 "app.content_security_policy": { 23 "app.content_security_policy": {
(...skipping 302 matching lines...) Expand 10 before | Expand all | Expand 10 after
322 "extension_types": ["platform_app"], 326 "extension_types": ["platform_app"],
323 "location": "component" 327 "location": "component"
324 } 328 }
325 ], 329 ],
326 "webview": { 330 "webview": {
327 "channel": "stable", 331 "channel": "stable",
328 "extension_types": ["platform_app"], 332 "extension_types": ["platform_app"],
329 "min_manifest_version": 2 333 "min_manifest_version": 2
330 } 334 }
331 } 335 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698