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

Unified Diff: third_party/WebKit/Source/modules/mediastream/MediaStreamTrack.cpp

Issue 2623263003: Update MediaStreamTrack content-hint values. (Closed)
Patch Set: Created 3 years, 11 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/mediastream/MediaStreamTrack.cpp
diff --git a/third_party/WebKit/Source/modules/mediastream/MediaStreamTrack.cpp b/third_party/WebKit/Source/modules/mediastream/MediaStreamTrack.cpp
index 1202c132a53f2578a24ec5e73f45811f16f88745..483b6f82384783ccdd90212a0c890a2f7689ff01 100644
--- a/third_party/WebKit/Source/modules/mediastream/MediaStreamTrack.cpp
+++ b/third_party/WebKit/Source/modules/mediastream/MediaStreamTrack.cpp
@@ -47,8 +47,8 @@ namespace {
static const char kContentHintStringNone[] = "";
static const char kContentHintStringAudioSpeech[] = "speech";
static const char kContentHintStringAudioMusic[] = "music";
-static const char kContentHintStringVideoFluid[] = "fluid";
-static const char kContentHintStringVideoDetailed[] = "detailed";
+static const char kContentHintStringVideoMotion[] = "motion";
+static const char kContentHintStringVideoDetail[] = "detail";
} // namespace
MediaStreamTrack* MediaStreamTrack::create(ExecutionContext* context,
@@ -121,10 +121,10 @@ String MediaStreamTrack::contentHint() const {
return kContentHintStringAudioSpeech;
case WebMediaStreamTrack::ContentHintType::AudioMusic:
return kContentHintStringAudioMusic;
- case WebMediaStreamTrack::ContentHintType::VideoFluid:
- return kContentHintStringVideoFluid;
- case WebMediaStreamTrack::ContentHintType::VideoDetailed:
- return kContentHintStringVideoDetailed;
+ case WebMediaStreamTrack::ContentHintType::VideoMotion:
+ return kContentHintStringVideoMotion;
+ case WebMediaStreamTrack::ContentHintType::VideoDetail:
+ return kContentHintStringVideoDetail;
}
NOTREACHED();
@@ -152,10 +152,10 @@ void MediaStreamTrack::setContentHint(const String& hint) {
case MediaStreamSource::TypeVideo:
if (hint == kContentHintStringNone) {
translatedHint = WebMediaStreamTrack::ContentHintType::None;
- } else if (hint == kContentHintStringVideoFluid) {
- translatedHint = WebMediaStreamTrack::ContentHintType::VideoFluid;
- } else if (hint == kContentHintStringVideoDetailed) {
- translatedHint = WebMediaStreamTrack::ContentHintType::VideoDetailed;
+ } else if (hint == kContentHintStringVideoMotion) {
+ translatedHint = WebMediaStreamTrack::ContentHintType::VideoMotion;
+ } else if (hint == kContentHintStringVideoDetail) {
+ translatedHint = WebMediaStreamTrack::ContentHintType::VideoDetail;
} else {
// TODO(pbos): Log warning?
// Invalid values for video are to be ignored (similar to invalid enum

Powered by Google App Engine
This is Rietveld 408576698