Title: [198361] trunk/Source/WebCore
Revision
198361
Author
[email protected]
Date
2016-03-17 17:42:51 -0700 (Thu, 17 Mar 2016)

Log Message

Some media tests are flaky.
https://bugs.webkit.org/show_bug.cgi?id=155614

Reviewed by Eric Carlson.

* html/track/TextTrack.cpp:
(WebCore::TextTrack::~TextTrack):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (198360 => 198361)


--- trunk/Source/WebCore/ChangeLog	2016-03-17 23:55:09 UTC (rev 198360)
+++ trunk/Source/WebCore/ChangeLog	2016-03-18 00:42:51 UTC (rev 198361)
@@ -1,3 +1,13 @@
+2016-03-17  Brent Fulgham  <[email protected]>
+
+        Some media tests are flaky.
+        https://bugs.webkit.org/show_bug.cgi?id=155614
+
+        Reviewed by Eric Carlson.
+
+        * html/track/TextTrack.cpp:
+        (WebCore::TextTrack::~TextTrack):
+
 2016-03-17  Brady Eidson  <[email protected]>
 
         Don't try to restore deleted MemoryIndexes if their owning object store is not restored.

Modified: trunk/Source/WebCore/html/track/TextTrack.cpp (198360 => 198361)


--- trunk/Source/WebCore/html/track/TextTrack.cpp	2016-03-17 23:55:09 UTC (rev 198360)
+++ trunk/Source/WebCore/html/track/TextTrack.cpp	2016-03-18 00:42:51 UTC (rev 198361)
@@ -136,12 +136,12 @@
             m_client->textTrackRemoveCues(this, m_cues.get());
 
         for (size_t i = 0; i < m_cues->length(); ++i)
-            m_cues->item(i)->setTrack(0);
-        if (m_regions) {
-            for (size_t i = 0; i < m_regions->length(); ++i)
-                m_regions->item(i)->setTrack(0);
-        }
+            m_cues->item(i)->setTrack(nullptr);
     }
+    if (m_regions) {
+        for (size_t i = 0; i < m_regions->length(); ++i)
+            m_regions->item(i)->setTrack(nullptr);
+    }
     clearClient();
 }
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to