Title: [89544] trunk/Source/WebCore
Revision
89544
Author
[email protected]
Date
2011-06-23 01:59:10 -0700 (Thu, 23 Jun 2011)

Log Message

2011-06-23  Dmitriy Vyukov  <[email protected]>

        Reviewed by David Levin.

        Fix incorrect usage of a condition variable.
        https://bugs.webkit.org/show_bug.cgi?id=63127

        No new tests. This does not affect existing
        functionality.

        * storage/DatabaseTask.cpp:
        (WebCore::DatabaseTaskSynchronizer::waitForTaskCompletion):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (89543 => 89544)


--- trunk/Source/WebCore/ChangeLog	2011-06-23 08:24:41 UTC (rev 89543)
+++ trunk/Source/WebCore/ChangeLog	2011-06-23 08:59:10 UTC (rev 89544)
@@ -1,3 +1,16 @@
+2011-06-23  Dmitriy Vyukov  <[email protected]>
+
+        Reviewed by David Levin.
+
+        Fix incorrect usage of a condition variable.
+        https://bugs.webkit.org/show_bug.cgi?id=63127
+
+        No new tests. This does not affect existing
+        functionality.
+
+        * storage/DatabaseTask.cpp:
+        (WebCore::DatabaseTaskSynchronizer::waitForTaskCompletion):
+
 2011-06-23  David Grogan  <[email protected]>
 
         Reviewed by Eric Seidel.

Modified: trunk/Source/WebCore/storage/DatabaseTask.cpp (89543 => 89544)


--- trunk/Source/WebCore/storage/DatabaseTask.cpp	2011-06-23 08:24:41 UTC (rev 89543)
+++ trunk/Source/WebCore/storage/DatabaseTask.cpp	2011-06-23 08:59:10 UTC (rev 89544)
@@ -46,7 +46,7 @@
 void DatabaseTaskSynchronizer::waitForTaskCompletion()
 {
     m_synchronousMutex.lock();
-    if (!m_taskCompleted)
+    while (!m_taskCompleted)
         m_synchronousCondition.wait(m_synchronousMutex);
     m_synchronousMutex.unlock();
 }
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to