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

Unified Diff: Source/core/html/track/LoadableTextTrack.cpp

Issue 351563005: Oilpan: Change LoadableTextTrack::m_trackElement a strong Member (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 6 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
« no previous file with comments | « Source/core/html/track/LoadableTextTrack.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/html/track/LoadableTextTrack.cpp
diff --git a/Source/core/html/track/LoadableTextTrack.cpp b/Source/core/html/track/LoadableTextTrack.cpp
index 9544b18b9fbe35d59bb317713321506508bd7b16..bd52aa4b1655740e2284bb8df55bbdd2c6ee2819 100644
--- a/Source/core/html/track/LoadableTextTrack.cpp
+++ b/Source/core/html/track/LoadableTextTrack.cpp
@@ -61,8 +61,10 @@ void LoadableTextTrack::clearTrackElement()
void LoadableTextTrack::setMode(const AtomicString& mode)
{
TextTrack::setMode(mode);
+#if !ENABLE(OILPAN)
if (!m_trackElement)
return;
+#endif
if (m_trackElement->readyState() == HTMLTrackElement::NONE)
m_trackElement->scheduleLoad();
@@ -98,8 +100,10 @@ void LoadableTextTrack::loadTimerFired(Timer<LoadableTextTrack>*)
if (m_loader)
m_loader->cancelLoad();
+#if !ENABLE(OILPAN)
if (!m_trackElement)
return;
+#endif
// 4.8.10.12.3 Sourcing out-of-band text tracks (continued)
@@ -134,8 +138,10 @@ void LoadableTextTrack::cueLoadingCompleted(TextTrackLoader* loader, bool loadin
{
ASSERT_UNUSED(loader, m_loader == loader);
+#if !ENABLE(OILPAN)
if (!m_trackElement)
return;
+#endif
m_trackElement->didCompleteLoad(loadingFailed ? HTMLTrackElement::Failure : HTMLTrackElement::Success);
}
« no previous file with comments | « Source/core/html/track/LoadableTextTrack.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698