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

Unified Diff: third_party/WebKit/Source/modules/imagecapture/MediaSettingsRange.h

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/Source/modules/imagecapture/MediaSettingsRange.h
diff --git a/third_party/WebKit/Source/modules/imagecapture/MediaSettingsRange.h b/third_party/WebKit/Source/modules/imagecapture/MediaSettingsRange.h
index d27f8e2975a848b93f3d72ec120150b700276f65..2f03c0497cac89028b1718d8da97c115ca34cc99 100644
--- a/third_party/WebKit/Source/modules/imagecapture/MediaSettingsRange.h
+++ b/third_party/WebKit/Source/modules/imagecapture/MediaSettingsRange.h
@@ -17,32 +17,28 @@ class MediaSettingsRange final : public GarbageCollected<MediaSettingsRange>,
public:
static MediaSettingsRange* create(double max,
double min,
- double current,
double step) {
- return new MediaSettingsRange(max, min, current, step);
+ return new MediaSettingsRange(max, min, step);
}
static MediaSettingsRange* create(media::mojom::blink::RangePtr range) {
return MediaSettingsRange::create(*range);
}
static MediaSettingsRange* create(const media::mojom::blink::Range& range) {
- return MediaSettingsRange::create(range.max, range.min, range.current,
- range.step);
+ return MediaSettingsRange::create(range.max, range.min, range.step);
}
double max() const { return m_max; }
double min() const { return m_min; }
- double current() const { return m_current; }
double step() const { return m_step; }
DEFINE_INLINE_TRACE() {}
private:
- MediaSettingsRange(double max, double min, double current, double step)
- : m_max(max), m_min(min), m_current(current), m_step(step) {}
+ MediaSettingsRange(double max, double min, double step)
+ : m_max(max), m_min(min), m_step(step) {}
double m_max;
double m_min;
- double m_current;
double m_step;
};

Powered by Google App Engine
This is Rietveld 408576698