Implement MediaStreamTrack contentHint skeleton.

third_party/WebKit/Source/modules/mediastream/MediaStreamTrack.cpp

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  * Copyright (C) 2011 Ericsson AB. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 89 matching lines @@
 
   if (!ended())
     MediaStreamCenter::instance().didSetMediaStreamTrackEnabled(
         m_component.get());
 }
 
 bool MediaStreamTrack::muted() const {
   return m_component->muted();
 }
 
+String MediaStreamTrack::contentHint() const {
+  DEFINE_STATIC_LOCAL(String, none, (""));
+  DEFINE_STATIC_LOCAL(String, speech, ("speech"));

[Guido Urdaneta, 2016/12/08 10:35:36]

Do you really need these? You are only using them once.
Maybe it would be better to define them as char[] and reuse them also in
setContentHint.

[pbos, 2016/12/08 16:58:32]

Done, do you prefer this over inlining the strings?

+  DEFINE_STATIC_LOCAL(String, music, ("music"));
+  DEFINE_STATIC_LOCAL(String, fluid, ("fluid"));
+  DEFINE_STATIC_LOCAL(String, detailed, ("detailed"));
+  WebMediaStreamTrack::ContentHint hint = m_component->contentHint();
+  switch (hint) {
+    case WebMediaStreamTrack::ContentHint::None:
+      return none;
+    case WebMediaStreamTrack::ContentHint::AudioSpeech:
+      return speech;
+    case WebMediaStreamTrack::ContentHint::AudioMusic:
+      return music;
+    case WebMediaStreamTrack::ContentHint::VideoFluid:
+      return fluid;
+    case WebMediaStreamTrack::ContentHint::VideoDetailed:
+      return detailed;
+  }
+
+  NOTREACHED();
+  return String();
+}
+
+void MediaStreamTrack::setContentHint(const String& hint) {
+  WebMediaStreamTrack::ContentHint translatedHint =
+      WebMediaStreamTrack::ContentHint::None;
+  switch (m_component->source()->type()) {
+    case MediaStreamSource::TypeAudio:
+      if (hint == "") {
+        translatedHint = WebMediaStreamTrack::ContentHint::None;
+      } else if (hint == "speech") {
+        translatedHint = WebMediaStreamTrack::ContentHint::AudioSpeech;
+      } else if (hint == "music") {
+        translatedHint = WebMediaStreamTrack::ContentHint::AudioMusic;
+      } else {
+        // TODO(pbos): Log warning?
+        return;
+      }
+      break;
+    case MediaStreamSource::TypeVideo:
+      if (hint == "") {
+        translatedHint = WebMediaStreamTrack::ContentHint::None;
+      } else if (hint == "fluid") {
+        translatedHint = WebMediaStreamTrack::ContentHint::VideoFluid;
+      } else if (hint == "detailed") {
+        translatedHint = WebMediaStreamTrack::ContentHint::VideoDetailed;
+      } else {
+        // TODO(pbos): Log warning?
+        return;
+      }
+  }
+
+  m_component->setContentHint(translatedHint);
+}
+
 bool MediaStreamTrack::remote() const {
   return m_component->source()->remote();
 }
 
 String MediaStreamTrack::readyState() const {
   if (ended())
     return "ended";
 
   switch (m_readyState) {
     case MediaStreamSource::ReadyStateLive:
@@ skipping 134 matching lines @@
 }
 
 DEFINE_TRACE(MediaStreamTrack) {
   visitor->trace(m_registeredMediaStreams);
   visitor->trace(m_component);
   EventTargetWithInlineData::trace(visitor);
   ActiveDOMObject::trace(visitor);
 }
 
 }  // namespace blink