MSE: Implement TrackDefault object

Source/core/html/track/VideoTrack.cpp

Index: Source/core/html/track/VideoTrack.cpp
diff --git a/Source/core/html/track/VideoTrack.cpp b/Source/core/html/track/VideoTrack.cpp
index 86d26b57cf077973b8ab39a406cdc856d6914768..18abe424f28fa2a6bbf30bf5813214f969d44536 100644
--- a/Source/core/html/track/VideoTrack.cpp
+++ b/Source/core/html/track/VideoTrack.cpp
@@ -69,14 +69,26 @@ const AtomicString& VideoTrack::commentaryKeyword()
     return keyword;
 }
 
-bool VideoTrack::isValidKind(const AtomicString& kind) const
+bool VideoTrack::isValidKindKeyword(const AtomicString& kind)
 {
     return (kind == alternativeKeyword())
         || (kind == captionsKeyword())
         || (kind == mainKeyword())
         || (kind == signKeyword())
         || (kind == subtitlesKeyword())
-        || (kind == commentaryKeyword());
+        || (kind == commentaryKeyword())
+        || (kind == emptyAtom);
+}
+
+bool VideoTrack::isValidKindKeyword(const String& kind)

[philipj_slow, 2014/11/14 12:03:37]

AtomicString is free to convert to String (and has an implicit cast operator) so
only the String variant is needed. (I hope, I only tested the AudioTrack case.)

[wolenetz, 2014/11/17 21:51:07]

Done.

+{
+    return (kind == alternativeKeyword())
+        || (kind == captionsKeyword())
+        || (kind == mainKeyword())
+        || (kind == signKeyword())
+        || (kind == subtitlesKeyword())
+        || (kind == commentaryKeyword())
+        || (kind == emptyAtom);
 }
 
 AtomicString VideoTrack::defaultKind() const