Bug 91898

Summary: [BlackBerry] clock_gettime() in BackingStore.cpp should use CLOCK_MONOTONIC
Product: WebKit Reporter: Jacky Jiang <jkjiang>
Component: WebKit BlackBerryAssignee: Jacky Jiang <jkjiang>
Status: RESOLVED FIXED    
Severity: Normal CC: anilsson, mifenton, rwlbuis, tonikitoo, webkit.review.bot, yong.li.webkit
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch yong.li.webkit: review+

Jacky Jiang
Reported 2012-07-20 13:24:21 PDT
clock_gettime() in BackingStore.cpp should use CLOCK_MONOTONIC.
Attachments
Patch (1.60 KB, patch)
2012-07-20 14:21 PDT, Jacky Jiang 		no flags
DetailsFormatted DiffDiff
Patch (1.63 KB, patch)
2012-07-25 10:03 PDT, Jacky Jiang 		yong.li.webkit: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Jacky Jiang
Comment 1 2012-07-20 14:21:54 PDT
Created attachment 153600 [details] Patch
WebKit Review Bot
Comment 2 2012-07-20 14:45:36 PDT
Comment on attachment 153600 [details] Patch Clearing flags on attachment: 153600 Committed r123266: <http://trac.webkit.org/changeset/123266>
WebKit Review Bot
Comment 3 2012-07-20 14:45:39 PDT
All reviewed patches have been landed. Closing bug.
Jacky Jiang
Comment 4 2012-07-20 14:54:33 PDT
(In reply to comment #2) > (From update of attachment 153600 [details]) > Clearing flags on attachment: 153600 > > Committed r123266: <http://trac.webkit.org/changeset/123266> So weird that it mixed with the other patches. I didn't see anything wrong with my patch though.
Jacky Jiang
Comment 5 2012-07-23 08:46:01 PDT
reopen.
Jacky Jiang
Comment 6 2012-07-25 10:03:31 PDT
Created attachment 154379 [details] Patch
Yong Li
Comment 7 2012-07-25 10:13:51 PDT
Comment on attachment 154379 [details] Patch We should WTF::ThreadCondition in the future. but that won't work for now. r+
Jacky Jiang
Comment 8 2012-07-25 10:23:30 PDT
Comment on attachment 154379 [details] Patch Committed, http://trac.webkit.org/changeset/123629.
Note You need to log in before you can comment on or make changes to this bug.