Chromium Code Reviews Chromium Code Reviews
Issue 2255613003:
Introduce session type parameter to extension features (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| OLD | NEW |
|---|---|
| 1 # Extension Features Files | 1 # Extension Features Files |
| 2 | 2 |
| 3 [TOC] | 3 [TOC] |
| 4 | 4 |
| 5 ## Summary | 5 ## Summary |
| 6 | 6 |
| 7 The Extension features files specify the different requirements for extension | 7 The Extension features files specify the different requirements for extension |
| 8 feature availability. | 8 feature availability. |
| 9 | 9 |
| 10 An **extension feature** can be any component of extension capabilities. Most | 10 An **extension feature** can be any component of extension capabilities. Most |
| (...skipping 235 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 246 The only accepted value is the bool `true`. | 246 The only accepted value is the bool `true`. |
| 247 | 247 |
| 248 ### platforms | 248 ### platforms |
| 249 | 249 |
| 250 The `platforms` property specifies the properties the feature should be | 250 The `platforms` property specifies the properties the feature should be |
| 251 available on. | 251 available on. |
| 252 | 252 |
| 253 The accepted values are lists of strings from `chromeos`, `mac`, `linux`, and | 253 The accepted values are lists of strings from `chromeos`, `mac`, `linux`, and |
| 254 `win`. | 254 `win`. |
| 255 | 255 |
| 256 ### session\_types | |
| 257 | |
| 258 The `session_types` property specifies in which types of sessions a feature | |
| 259 should be available. The session type describes the type of user that is | |
| 260 logged in the current session. | |
| 261 | |
| 262 The accepted value is a list of strings from `regular`, `kiosk`. Empty list is | |
|
Devlin
2016/09/12 22:33:08
nit: The accepted values are lists of strings from
tbarzic
2016/09/12 23:05:10
Done.
| |
| 263 ignored. | |
| 264 | |
| 256 ### whitelist | 265 ### whitelist |
| 257 | 266 |
| 258 The `whitelist` property specifies a list of ID hashes for extensions that | 267 The `whitelist` property specifies a list of ID hashes for extensions that |
| 259 are the only extensions allowed to access a feature. | 268 are the only extensions allowed to access a feature. |
| 260 | 269 |
| 261 Accepted values are lists of id hashes. | 270 Accepted values are lists of id hashes. |
| 262 | 271 |
| 263 ## ID Hashes | 272 ## ID Hashes |
| 264 | 273 |
| 265 Instead of listing the ID directly in the whitelist or blacklist section, we | 274 Instead of listing the ID directly in the whitelist or blacklist section, we |
| (...skipping 104 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 370 at runtime, this has the added benefit of allowing us to validate at compile | 379 at runtime, this has the added benefit of allowing us to validate at compile |
| 371 time rather than needing a unittest (or allowing incorrect features). | 380 time rather than needing a unittest (or allowing incorrect features). |
| 372 | 381 |
| 373 In theory, invalid features should result in a compilation failure; in practice, | 382 In theory, invalid features should result in a compilation failure; in practice, |
| 374 the compiler is probably missing some cases. | 383 the compiler is probably missing some cases. |
| 375 | 384 |
| 376 ## Still to come | 385 ## Still to come |
| 377 | 386 |
| 378 TODO(devlin): Add documentation for extension types. Probably also more on | 387 TODO(devlin): Add documentation for extension types. Probably also more on |
| 379 requirements for individual features. | 388 requirements for individual features. |
| OLD | NEW |
