Title: [92627] trunk/Source/WebCore
Revision
92627
Author
[email protected]
Date
2011-08-08 13:18:33 -0700 (Mon, 08 Aug 2011)

Log Message

Add detail to ASSERT message in IDBTransaction::enqueueEvent
https://bugs.webkit.org/show_bug.cgi?id=65735

I've hit this a few times but haven't been able to pin it down.  The
extra detail could be helpful.

Patch by David Grogan <[email protected]> on 2011-08-08
Reviewed by Nate Chapin.

No new tests, just an error message.

* storage/IDBTransaction.cpp:
(WebCore::IDBTransaction::enqueueEvent):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (92626 => 92627)


--- trunk/Source/WebCore/ChangeLog	2011-08-08 20:13:48 UTC (rev 92626)
+++ trunk/Source/WebCore/ChangeLog	2011-08-08 20:18:33 UTC (rev 92627)
@@ -1,3 +1,18 @@
+2011-08-08  David Grogan  <[email protected]>
+
+        Add detail to ASSERT message in IDBTransaction::enqueueEvent
+        https://bugs.webkit.org/show_bug.cgi?id=65735
+
+        I've hit this a few times but haven't been able to pin it down.  The
+        extra detail could be helpful.
+
+        Reviewed by Nate Chapin.
+
+        No new tests, just an error message.
+
+        * storage/IDBTransaction.cpp:
+        (WebCore::IDBTransaction::enqueueEvent):
+
 2011-08-08  Sheriff Bot  <[email protected]>
 
         Unreviewed, rolling out r92610.

Modified: trunk/Source/WebCore/storage/IDBTransaction.cpp (92626 => 92627)


--- trunk/Source/WebCore/storage/IDBTransaction.cpp	2011-08-08 20:13:48 UTC (rev 92626)
+++ trunk/Source/WebCore/storage/IDBTransaction.cpp	2011-08-08 20:18:33 UTC (rev 92627)
@@ -182,7 +182,7 @@
 
 void IDBTransaction::enqueueEvent(PassRefPtr<Event> event)
 {
-    ASSERT(!m_finished);
+    ASSERT_WITH_MESSAGE(!m_finished, "A finished transaction tried to enqueue an event of type %s.", event->type().string().utf8().data());
     if (!scriptExecutionContext())
         return;
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to