Chromium Code Reviews
DescriptionFix for http://code.google.com/p/chromium/issues/detail?id=7429
The problem is that webkit's timers are all based off the WTF/CurrentTime
implementation, but the bridge was doing math between a webkit that
implementation of currentTime and Chromium's base/Time implementation.
Subtracting two different times leads to skew, which turned out to
always be negative, so the CPU was spinning for most all timers.
This bug was introduced during the merge which brought in timers. From
the outside, functionality was correct (timers fired at the right time),
but internally we could spin the CPU in while waiting for the timer to
fire.
There is some code to remove from SystemTimeChromium, which I will do,
but that is in the webkit tree, so I'll do that separately.
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=9307
Patch Set 1 #
Total comments: 1
Patch Set 2 : '' #Messages
Total messages: 2 (0 generated)
|