Chromium Code Reviews| Index: Source/core/html/track/TextTrack.cpp |
| diff --git a/Source/core/html/track/TextTrack.cpp b/Source/core/html/track/TextTrack.cpp |
| index 460b90a7108581db694a8b6d428e0c18a2c90674..9f1098ef5c8249b9d4c994c9b2310280b9b1b3a8 100644 |
| --- a/Source/core/html/track/TextTrack.cpp |
| +++ b/Source/core/html/track/TextTrack.cpp |
| @@ -97,14 +97,10 @@ const AtomicString& TextTrack::showingKeyword() |
| } |
| TextTrack::TextTrack(Document& document, TextTrackClient* client, const AtomicString& kind, const AtomicString& label, const AtomicString& language, const AtomicString& id, TextTrackType type) |
| - : TrackBase(TrackBase::TextTrack) |
| + : TrackBase(TrackBase::TextTrack, label, language, id) |
| , m_cues(0) |
| , m_regions(0) |
| , m_document(&document) |
| - , m_mediaElement(0) |
| - , m_label(label) |
| - , m_language(language) |
| - , m_id(id) |
| , m_mode(disabledKeyword()) |
| , m_client(client) |
| , m_trackType(type) |
| @@ -150,16 +146,12 @@ bool TextTrack::isValidKindKeyword(const AtomicString& value) |
| return false; |
| } |
| -void TextTrack::setKind(const AtomicString& kind) |
| +void TextTrack::setKind(const AtomicString& newKind) |
| { |
| - String oldKind = m_kind; |
| + String oldKind = kind(); |
|
acolwell GONE FROM CHROMIUM
2013/11/28 03:02:17
nit: s/String/AtomicString?
|
| + TrackBase::setKind(newKind); |
| - if (isValidKindKeyword(kind)) |
| - m_kind = kind; |
| - else |
| - m_kind = subtitlesKeyword(); |
| - |
| - if (m_client && oldKind != m_kind) |
| + if (m_client && oldKind != kind()) |
| m_client->textTrackKindChanged(this); |
| } |
| @@ -377,10 +369,10 @@ void TextTrack::cueDidChange(TextTrackCue* cue) |
| int TextTrack::trackIndex() |
| { |
| - ASSERT(m_mediaElement); |
| + ASSERT(mediaElement()); |
| if (m_trackIndex == invalidTrackIndex) |
| - m_trackIndex = m_mediaElement->textTracks()->getTrackIndex(this); |
| + m_trackIndex = mediaElement()->textTracks()->getTrackIndex(this); |
| return m_trackIndex; |
| } |
| @@ -393,10 +385,10 @@ void TextTrack::invalidateTrackIndex() |
| bool TextTrack::isRendered() |
| { |
| - if (m_kind != captionsKeyword() && m_kind != subtitlesKeyword()) |
| + if (kind() != captionsKeyword() && kind() != subtitlesKeyword()) |
| return false; |
| - if (m_mode != showingKeyword()) |
| + if (kind() != showingKeyword()) |
| return false; |
| return true; |
| @@ -412,10 +404,10 @@ TextTrackCueList* TextTrack::ensureTextTrackCueList() |
| int TextTrack::trackIndexRelativeToRenderedTracks() |
| { |
| - ASSERT(m_mediaElement); |
| + ASSERT(mediaElement()); |
| if (m_renderedTrackIndex == invalidTrackIndex) |
| - m_renderedTrackIndex = m_mediaElement->textTracks()->getTrackIndexRelativeToRenderedTracks(this); |
| + m_renderedTrackIndex = mediaElement()->textTracks()->getTrackIndexRelativeToRenderedTracks(this); |
| return m_renderedTrackIndex; |
| } |