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

Unified Diff: third_party/WebKit/LayoutTests/imagecapture/MediaStreamTrack-getSettings.html

Issue 2795923003: Image Capture: remove MediaSettingsRange.current (Closed)
Patch Set: bugfix: getMediaTrackSettings() should merge dictionaries, not override the input parameter; Layout… Created 3 years, 8 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/MediaStreamTrack-getSettings.html
diff --git a/third_party/WebKit/LayoutTests/imagecapture/MediaStreamTrack-getSettings.html b/third_party/WebKit/LayoutTests/imagecapture/MediaStreamTrack-getSettings.html
index 3ae9df20d7b87d9ea0854d5caa4fb927def74a30..3c1ed9d0603b6c1ed35eb29934f6fdfc86bb1d12 100644
--- a/third_party/WebKit/LayoutTests/imagecapture/MediaStreamTrack-getSettings.html
+++ b/third_party/WebKit/LayoutTests/imagecapture/MediaStreamTrack-getSettings.html
@@ -45,6 +45,18 @@ async_test(function(t) {
assert_equals(settings.focusMode,
meteringModeNames[mock_settings.focus_mode], 'focusMode');
+ assert_equals(settings.pointsOfInterest.length,
+ mock_settings.points_of_interest.length,
+ 'pointsOfInterest');
+ for (i = 0; i < settings.pointsOfInterest.length; ++i) {
+ assert_approx_equals(settings.pointsOfInterest[i].x,
+ mock_settings.points_of_interest[i].x, 0.01,
+ 'pointsOfInterest x');
+ assert_approx_equals(settings.pointsOfInterest[i].y,
+ mock_settings.points_of_interest[i].y, 0.01,
+ 'pointsOfInterest y');
+ }
+
assert_equals(settings.exposureCompensation,
mock_settings.exposure_compensation.current);
assert_equals(settings.colorTemperature,
@@ -58,7 +70,8 @@ async_test(function(t) {
assert_equals(settings.zoom, mock_settings.zoom.current);
- assert_equals(settings.torch, mock_settings.torch);
+ // TODO(mcasas): do |torch| when the mojom interface is updated,
+ // https://crbug.com/700607.
t.done();
}, 100);

Powered by Google App Engine
This is Rietveld 408576698