Bug 91898 - [BlackBerry] clock_gettime() in BackingStore.cpp should use CLOCK_MONOTONIC
Summary: [BlackBerry] clock_gettime() in BackingStore.cpp should use CLOCK_MONOTONIC
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit BlackBerry (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jacky Jiang
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-20 13:24 PDT by Jacky Jiang
Modified: 2012-07-25 10:23 PDT (History)
6 users (show)

See Also:

Attachments
Patch (1.60 KB, patch)
2012-07-20 14:21 PDT, Jacky Jiang 		no flags Details | Formatted Diff | Diff
Patch (1.63 KB, patch)
2012-07-25 10:03 PDT, Jacky Jiang 		yong.li.webkit: review+
Details | Formatted Diff | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jacky Jiang 2012-07-20 13:24:21 PDT 
clock_gettime() in BackingStore.cpp should use CLOCK_MONOTONIC.
Comment 1 Jacky Jiang 2012-07-20 14:21:54 PDT 
Created attachment 153600 [details]
Patch
Comment 2 WebKit Review Bot 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>
Comment 3 WebKit Review Bot 2012-07-20 14:45:39 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 4 Jacky Jiang 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.
Comment 5 Jacky Jiang 2012-07-23 08:46:01 PDT 
reopen.
Comment 6 Jacky Jiang 2012-07-25 10:03:31 PDT 
Created attachment 154379 [details]
Patch
Comment 7 Yong Li 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+
Comment 8 Jacky Jiang 2012-07-25 10:23:30 PDT 
Comment on attachment 154379 [details]
Patch

Committed, http://trac.webkit.org/changeset/123629.