Title: [123629] trunk/Source/WebKit/blackberry
Revision
123629
Author
[email protected]
Date
2012-07-25 10:21:57 -0700 (Wed, 25 Jul 2012)

Log Message

[BlackBerry] clock_gettime() in BackingStore.cpp should use CLOCK_MONOTONIC
https://bugs.webkit.org/show_bug.cgi?id=91898

Reviewed by Yong Li.
Patch by Jacky Jiang  <[email protected]>

Let pthread condition variable in BackingStore.cpp use monotonic clock
to fix a regression caused by r123266.

* Api/BackingStore.cpp:
(BlackBerry::WebKit::BackingStorePrivate::BackingStorePrivate):

Modified Paths

Diff

Modified: trunk/Source/WebKit/blackberry/Api/BackingStore.cpp (123628 => 123629)


--- trunk/Source/WebKit/blackberry/Api/BackingStore.cpp	2012-07-25 17:07:29 UTC (rev 123628)
+++ trunk/Source/WebKit/blackberry/Api/BackingStore.cpp	2012-07-25 17:21:57 UTC (rev 123629)
@@ -228,7 +228,12 @@
     pthread_mutexattr_destroy(&attr);
 
     pthread_mutex_init(&m_blitGenerationLock, 0);
-    pthread_cond_init(&m_blitGenerationCond, 0);
+
+    pthread_condattr_t condattr;
+    pthread_condattr_init(&condattr);
+    pthread_condattr_setclock(&condattr, CLOCK_MONOTONIC);
+    pthread_cond_init(&m_blitGenerationCond, &condattr);
+    pthread_condattr_destroy(&condattr);
 }
 
 BackingStorePrivate::~BackingStorePrivate()

Modified: trunk/Source/WebKit/blackberry/ChangeLog (123628 => 123629)


--- trunk/Source/WebKit/blackberry/ChangeLog	2012-07-25 17:07:29 UTC (rev 123628)
+++ trunk/Source/WebKit/blackberry/ChangeLog	2012-07-25 17:21:57 UTC (rev 123629)
@@ -1,3 +1,16 @@
+2012-07-25  Jacky Jiang  <[email protected]>
+
+        [BlackBerry] clock_gettime() in BackingStore.cpp should use CLOCK_MONOTONIC
+        https://bugs.webkit.org/show_bug.cgi?id=91898
+
+        Reviewed by Yong Li.
+
+        Let pthread condition variable in BackingStore.cpp use monotonic clock
+        to fix a regression caused by r123266.
+
+        * Api/BackingStore.cpp:
+        (BlackBerry::WebKit::BackingStorePrivate::BackingStorePrivate):
+
 2012-07-25  Andy Chen  <[email protected]>
 
         [BlackBerry] Add "SelectAll" to WebPage
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to