Move code that applies to audio and video tracks into TrackBase.

Source/core/html/track/TrackBase.h

 /*
  * Copyright (C) 2011 Apple Inc.  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
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 13 matching lines @@
  */
 
 #ifndef TrackBase_h
 #define TrackBase_h
 
 #include "core/events/EventTarget.h"
 #include "wtf/RefCounted.h"
 
 namespace WebCore {
 
-class TrackBase : public RefCounted<TrackBase>, public EventTargetWithInlineData {
-    REFCOUNTED_EVENT_TARGET(TrackBase);
+class HTMLMediaElement;
+
+class TrackBase : public RefCounted<TrackBase> {
 public:
-    virtual ~TrackBase() { }
+    virtual ~TrackBase();
 
     enum Type { TextTrack, AudioTrack, VideoTrack };
     Type type() const { return m_type; }
 
+    void setMediaElement(HTMLMediaElement* element) { m_mediaElement = element; }

[acolwell GONE FROM CHROMIUM, 2013/11/28 03:02:17]

Does the base really need a reference to the mediaElement? It looks like the
TextTrack code only cares about the TextTrackList. It seems like we should just
have TextTrack hold a reference to the TextTrackList instead of providing access
to the whole HTMLMediaElement.

+    HTMLMediaElement* mediaElement() { return m_mediaElement; }
+
+    AtomicString kind() const { return m_kind; }
+    virtual void setKind(const AtomicString&);
+
+    AtomicString label() const { return m_label; }
+    void setLabel(const AtomicString& label) { m_label = label; }
+
+    AtomicString language() const { return m_language; }
+    void setLanguage(const AtomicString& language) { m_language = language; }
+
+    AtomicString id() const { return m_id; }
+    void setId(const AtomicString& id) { m_id = id; }
+
 protected:
-    explicit TrackBase(Type type) : m_type(type) { }
+    explicit TrackBase(Type, const AtomicString& label, const AtomicString& language, const AtomicString& id);

[acolwell GONE FROM CHROMIUM, 2013/11/28 03:02:17]

nit: remove explicit

+
+    virtual bool isValidKind(const AtomicString&) const = 0;
+    virtual AtomicString defaultKind() const = 0;
 
 private:
     Type m_type;
+    HTMLMediaElement* m_mediaElement;
+    AtomicString m_kind;
+    AtomicString m_label;
+    AtomicString m_language;
+    AtomicString m_id;
 };
 
 } // namespace WebCore
 
 #endif // TrackBase_h