Title: [111941] trunk/Source/WebCore
Revision
111941
Author
eric.carl...@apple.com
Date
2012-03-23 17:34:32 -0700 (Fri, 23 Mar 2012)

Log Message

Remove unnecessary ASSERT from LoadableTextTrack::trackElementIndex
https://bugs.webkit.org/show_bug.cgi?id=82095

Reviewed by Dan Bernstein.

No new tests, removed a bogus ASSERT that was sometimes triggered by tests.

* html/track/LoadableTextTrack.cpp:
(WebCore::LoadableTextTrack::trackElementIndex): Remove ASSERT.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (111940 => 111941)


--- trunk/Source/WebCore/ChangeLog	2012-03-24 00:32:02 UTC (rev 111940)
+++ trunk/Source/WebCore/ChangeLog	2012-03-24 00:34:32 UTC (rev 111941)
@@ -1,5 +1,17 @@
 2012-03-23  Eric Carlson  <eric.carl...@apple.com>
 
+        Remove unnecessary ASSERT from LoadableTextTrack::trackElementIndex
+        https://bugs.webkit.org/show_bug.cgi?id=82095
+
+        Reviewed by Dan Bernstein.
+
+        No new tests, removed a bogus ASSERT that was sometimes triggered by tests.
+
+        * html/track/LoadableTextTrack.cpp:
+        (WebCore::LoadableTextTrack::trackElementIndex): Remove ASSERT.
+
+2012-03-23  Eric Carlson  <eric.carl...@apple.com>
+
         Layout Test media/video-source-load.html is failing
         https://bugs.webkit.org/show_bug.cgi?id=82094
 

Modified: trunk/Source/WebCore/html/track/LoadableTextTrack.cpp (111940 => 111941)


--- trunk/Source/WebCore/html/track/LoadableTextTrack.cpp	2012-03-24 00:32:02 UTC (rev 111940)
+++ trunk/Source/WebCore/html/track/LoadableTextTrack.cpp	2012-03-24 00:34:32 UTC (rev 111941)
@@ -135,7 +135,6 @@
 {
     ASSERT(m_trackElement);
     ASSERT(m_trackElement->parentNode());
-    ASSERT(m_trackElement->inDocument());
 
     size_t index = 0;
     for (Node* node = m_trackElement->parentNode()->firstChild(); node; node = node->nextSibling()) {
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to