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

Unified Diff: Source/core/html/track/TrackListBase.h

Issue 284513003: Implement AudioTrack, AudioTrackList, VideoTrack, and VideoTrackList (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: complete the transition to WebTrackId Created 6 years, 7 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: Source/core/html/track/TrackListBase.h
diff --git a/Source/core/html/track/TrackListBase.h b/Source/core/html/track/TrackListBase.h
new file mode 100644
index 0000000000000000000000000000000000000000..5f9b883626d2935379f114687db0d6aafe760f6e
--- /dev/null
+++ b/Source/core/html/track/TrackListBase.h
@@ -0,0 +1,124 @@
+// 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.
+
+#ifndef TrackListBase_h
+#define TrackListBase_h
+
+#include "core/events/EventTarget.h"
+
+#include "core/html/HTMLMediaElement.h"
+#include "core/html/track/TrackEvent.h"
+
+namespace WebCore {
+
+template<class T>
+class TrackListBase : public RefCounted<TrackListBase<T> >, public EventTargetWithInlineData {
+ REFCOUNTED_EVENT_TARGET(TrackListBase<T>);
+
+public:
+ explicit TrackListBase(HTMLMediaElement* mediaElement)
+ : m_mediaElement(mediaElement)
+ {
+ }
+
+ virtual ~TrackListBase()
+ {
+ ASSERT(m_tracks.isEmpty());
+ ASSERT(!m_mediaElement);
+ }
+
+ unsigned length() const { return m_tracks.size(); }
+ PassRefPtr<T> anonymousIndexedGetter(unsigned index) const
+ {
+ if (index >= m_tracks.size())
+ return nullptr;
+ return m_tracks[index];
+ }
+
+ PassRefPtr<T> getTrackById(const String& id) const
+ {
+ for (unsigned i = 0; i < m_tracks.size(); ++i) {
+ if (m_tracks[i]->id() == id)
+ return m_tracks[i];
+ }
+
+ return nullptr;
+ }
+
+ DEFINE_ATTRIBUTE_EVENT_LISTENER(change);
+ DEFINE_ATTRIBUTE_EVENT_LISTENER(addtrack);
+ DEFINE_ATTRIBUTE_EVENT_LISTENER(removetrack);
+
+ // EventTarget interface
+ virtual ExecutionContext* executionContext() const OVERRIDE
+ {
+ if (m_mediaElement)
+ return m_mediaElement->executionContext();
+ return 0;
+ }
+
+ void shutdown()
+ {
+ removeAll();
+ m_mediaElement = 0;
+ }
+
+ void add(PassRefPtr<T> track)
+ {
+ ASSERT(track);
acolwell GONE FROM CHROMIUM 2014/05/16 19:47:56 nit: No need for this assert since the code would
+ track->setMediaElement(m_mediaElement);
+ m_tracks.append(track.get());
+ scheduleTrackEvent(EventTypeNames::addtrack, track);
+ }
+
+ void remove(blink::WebTrackId webId)
+ {
+ for (unsigned i = 0; i < m_tracks.size(); ++i) {
+ if (m_tracks[i]->webId() != webId)
+ continue;
+
+ m_tracks[i]->setMediaElement(0);
+ scheduleTrackEvent(EventTypeNames::removetrack, m_tracks[i]);
+ m_tracks.remove(i);
+ return;
+ }
+ }
+
+ void removeAll()
+ {
+ for (unsigned i = 0; i < m_tracks.size(); ++i)
+ m_tracks[i]->setMediaElement(0);
+
+ m_tracks.clear();
+ }
+
+ void scheduleChangeEvent()
+ {
+ EventInit initializer;
+ initializer.bubbles = false;
+ initializer.cancelable = false;
+ RefPtr<Event> event = Event::create(EventTypeNames::change, initializer);
+ event->setTarget(this);
+ m_mediaElement->scheduleEvent(event);
+ }
+
+private:
+ void scheduleTrackEvent(const AtomicString& eventName, PassRefPtr<T> track)
+ {
+ TrackEventInit initializer;
+ initializer.track = track;
+ initializer.bubbles = false;
+ initializer.cancelable = false;
+ RefPtr<Event> event = TrackEvent::create(eventName, initializer);
+ event->setTarget(this);
+ m_mediaElement->scheduleEvent(event);
+ }
+
+ Vector<RefPtr<T> > m_tracks;
acolwell GONE FROM CHROMIUM 2014/05/16 19:47:56 I think this needs to be WillBeHeapVector<RefPtrWi
+ HTMLMediaElement* m_mediaElement;
acolwell GONE FROM CHROMIUM 2014/05/16 19:47:56 I think this should be RawPtrWillBeMember like in
+};
+
+}
+
+#endif

Powered by Google App Engine
This is Rietveld 408576698