Title: [126065] trunk/Source/WebCore
Revision
126065
Author
[email protected]
Date
2012-08-20 14:09:18 -0700 (Mon, 20 Aug 2012)

Log Message

Remove improper ASSERT in AudioParamTimeline::valuesForTimeRangeImpl()
https://bugs.webkit.org/show_bug.cgi?id=94504

Reviewed by Kenneth Russell.

AudioParamTimeline::valuesForTimeRangeImpl() is able to handle the case where there are no timeline events.
So don't ASSERT for that case.

* Modules/webaudio/AudioParamTimeline.cpp:
(WebCore::AudioParamTimeline::valuesForTimeRangeImpl):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (126064 => 126065)


--- trunk/Source/WebCore/ChangeLog	2012-08-20 21:03:07 UTC (rev 126064)
+++ trunk/Source/WebCore/ChangeLog	2012-08-20 21:09:18 UTC (rev 126065)
@@ -1,3 +1,16 @@
+2012-08-20  Chris Rogers  <[email protected]>
+
+        Remove improper ASSERT in AudioParamTimeline::valuesForTimeRangeImpl()
+        https://bugs.webkit.org/show_bug.cgi?id=94504
+
+        Reviewed by Kenneth Russell.
+
+        AudioParamTimeline::valuesForTimeRangeImpl() is able to handle the case where there are no timeline events.
+        So don't ASSERT for that case.
+
+        * Modules/webaudio/AudioParamTimeline.cpp:
+        (WebCore::AudioParamTimeline::valuesForTimeRangeImpl):
+
 2012-08-20  Ken Buchanan  <[email protected]>
         Line boxes not being dirtied correctly during inline removal
         https://bugs.webkit.org/show_bug.cgi?id=93156

Modified: trunk/Source/WebCore/Modules/webaudio/AudioParamTimeline.cpp (126064 => 126065)


--- trunk/Source/WebCore/Modules/webaudio/AudioParamTimeline.cpp	2012-08-20 21:03:07 UTC (rev 126064)
+++ trunk/Source/WebCore/Modules/webaudio/AudioParamTimeline.cpp	2012-08-20 21:09:18 UTC (rev 126065)
@@ -170,7 +170,6 @@
         return defaultValue;
 
     // Return default value if there are no events matching the desired time range.
-    ASSERT(m_events.size());
     if (!m_events.size() || endTime <= m_events[0].time()) {
         for (unsigned i = 0; i < numberOfValues; ++i)
             values[i] = defaultValue;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to