MSE: Implement TrackDefault object

Source/core/html/track/AudioTrack.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "core/html/track/AudioTrack.h"
 
 #include "core/html/HTMLMediaElement.h"
 
 namespace blink {
@@ skipping 49 matching lines @@
     DEFINE_STATIC_LOCAL(const AtomicString, keyword, ("translation", AtomicString::ConstructFromLiteral));
     return keyword;
 }
 
 const AtomicString& AudioTrack::commentaryKeyword()
 {
     DEFINE_STATIC_LOCAL(const AtomicString, keyword, ("commentary", AtomicString::ConstructFromLiteral));
     return keyword;
 }
 
-bool AudioTrack::isValidKind(const AtomicString& kind) const
+bool AudioTrack::isValidKindKeyword(const AtomicString& kind)
 {
     return (kind == alternativeKeyword())
         || (kind == descriptionsKeyword())
         || (kind == mainKeyword())
         || (kind == mainDescriptionsKeyword())
         || (kind == translationKeyword())
-        || (kind == commentaryKeyword());
+        || (kind == commentaryKeyword())
+        || (kind == emptyAtom);

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

Can you add to the commit message the reason why this is safe? (Because it only
affects TrackBase::setKind, and defaultKind() is the empty string.)

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

Done.

+}
+
+bool AudioTrack::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.

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

Done.

+{
+    return (kind == alternativeKeyword())
+        || (kind == descriptionsKeyword())
+        || (kind == mainKeyword())
+        || (kind == mainDescriptionsKeyword())
+        || (kind == translationKeyword())
+        || (kind == commentaryKeyword())
+        || (kind == emptyAtom);
 }
 
 AtomicString AudioTrack::defaultKind() const
 {
     return emptyAtom;
 }
 
 }