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

Unified Diff: chrome/test/data/policy/policy_test_cases.json

Issue 1210173012: Split the Media settings UI into separate microphone and camera sections. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Reintroduced the label, but as hidden. Created 5 years, 5 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 | « chrome/browser/ui/webui/options/content_settings_handler.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/policy/policy_test_cases.json
diff --git a/chrome/test/data/policy/policy_test_cases.json b/chrome/test/data/policy/policy_test_cases.json
index 6bcdec0ba0750df49a2adec47947f94a0cb81d92..4e5fd5799dda639289239cf30e4e6a6e98ebee85 100644
--- a/chrome/test/data/policy/policy_test_cases.json
+++ b/chrome/test/data/policy/policy_test_cases.json
@@ -1065,7 +1065,7 @@
"test_policy": { "DefaultMediaStreamSetting": 2 },
"pref_mappings": [
{ "pref": "profile.managed_default_content_settings.media_stream",
- "indicator_selector": "[content-setting=media-stream]",
+ "indicator_selector": "[content-setting=media-stream-mic],[content-setting=media-stream-camera]",
"indicator_tests": [
{ "policy": { "DefaultMediaStreamSetting": 2 },
"value": "block"
@@ -1079,15 +1079,65 @@
"note": "TODO(bartfab): Flag this with can_be_recommended when http://crbug.com/106682 is fixed."
},
+
+ "DefaultMediaStreamSetting.OverriddenByAudioCaptureAllowed": {
+ "os": ["win", "linux", "mac", "chromeos"],
+ "test_policy": { "DefaultMediaStreamSetting": 3,
+ "AudioCaptureAllowed": false },
+ "pref_mappings": [
+ { "pref": "profile.managed_default_content_settings.media_stream",
+ "indicator_selector": "[content-setting=media-stream-mic]",
+ "indicator_tests": [
+ { "policy": { "AudioCaptureAllowed": false },
+ "value": "block"
+ },
+ { "policy": { "AudioCaptureAllowed": true,
+ "DefaultMediaStreamSetting": 2 },
+ "value": "block"
+ },
+ { "policy": { "AudioCaptureAllowed": true,
+ "DefaultMediaStreamSetting": 3 },
+ "value": "ask"
+ }
+ ]
+ }
+ ]
+ },
+
+ "DefaultMediaStreamSetting.OverriddenByVideoCaptureAllowed": {
+ "os": ["win", "linux", "mac", "chromeos"],
+ "test_policy": { "DefaultMediaStreamSetting": 3,
+ "VideoCaptureAllowed": false },
+ "pref_mappings": [
+ { "pref": "profile.managed_default_content_settings.media_stream",
+ "indicator_selector": "[content-setting=media-stream-camera]",
+ "indicator_tests": [
+ { "policy": { "VideoCaptureAllowed": false },
+ "value": "block"
+ },
+ { "policy": { "VideoCaptureAllowed": true,
+ "DefaultMediaStreamSetting": 2 },
+ "value": "block"
+ },
+ { "policy": { "VideoCaptureAllowed": true,
+ "DefaultMediaStreamSetting": 3 },
+ "value": "ask"
+ }
+ ]
+ }
+ ]
+ },
"AudioCaptureAllowed": {
"os": ["win", "linux", "mac", "chromeos"],
"test_policy": { "AudioCaptureAllowed": false },
"pref_mappings": [
{ "pref": "hardware.audio_capture_enabled",
- "indicator_selector": "#media-indicator",
+ "indicator_selector": "[content-setting=media-stream-mic][value=block]",
"indicator_tests": [
- { "policy": { "AudioCaptureAllowed": false } }
+ { "policy": { "AudioCaptureAllowed": false },
+ "value": "block"
+ }
]
}
]
@@ -1108,9 +1158,11 @@
"test_policy": { "VideoCaptureAllowed": false },
"pref_mappings": [
{ "pref": "hardware.video_capture_enabled",
- "indicator_selector": "#media-indicator",
+ "indicator_selector": "[content-setting=media-stream-camera][value=block]",
"indicator_tests": [
- { "policy": { "VideoCaptureAllowed": false } }
+ { "policy": { "VideoCaptureAllowed": false },
+ "value": "block"
+ }
]
}
]
« no previous file with comments | « chrome/browser/ui/webui/options/content_settings_handler.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698