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

Unified Diff: third_party/WebKit/LayoutTests/imagecapture/setoptions.html

Issue 2301053004: Image Capture: adding fillLightMode getting/setting (Closed)
Patch Set: Created 4 years, 3 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
Index: third_party/WebKit/LayoutTests/imagecapture/setoptions.html
diff --git a/third_party/WebKit/LayoutTests/imagecapture/setoptions.html b/third_party/WebKit/LayoutTests/imagecapture/setoptions.html
index 7f7f7223ef16fe9e59b4f00cb3946bfbc2bd06dd..6148fc47be01b00bc26d4ee2ff22848080bb4bf7 100644
--- a/third_party/WebKit/LayoutTests/imagecapture/setoptions.html
+++ b/third_party/WebKit/LayoutTests/imagecapture/setoptions.html
@@ -8,7 +8,8 @@
</body>
<script>
-const meteringModeNames = ["unavailable", "manual", "single-shot", "continuous"];
+const meteringModeNames = ["none", "manual", "single-shot", "continuous"];
+const fillLightModeNames = ["none", "off", "auto", "flash", "torch"];
// This test verifies that ImageCapture can call setOptions()s, with a mock Mojo
// interface implementation.
@@ -31,6 +32,7 @@ async_test(function(t) {
whiteBalanceMode : "manual",
iso : 120,
redEyeReduction : true,
+ fillLightMode : "flash",
};
mockImageCaptureReady
.then(mock => {
@@ -89,6 +91,10 @@ async_test(function(t) {
// TODO(mcasas): Revert to assert_equals() when yzshen@ has sorted it out.
assert_true(optionsDict.redEyeReduction == theMock.options().red_eye_reduction,
'red_eye_reduction value');
+ assert_equals(true, theMock.options().has_fill_light_mode, 'has_fill_light_mode must be true');
+ assert_equals(optionsDict.fillLightMode,
+ fillLightModeNames[theMock.options().fill_light_mode],
+ 'fillLightMode value');
t.done();
})

Powered by Google App Engine
This is Rietveld 408576698