commit | 518c2112614d1c6b375e3667a150459378925bcc | [log] [tgz] |
---|---|---|
author | ksakamoto@chromium.org <ksakamoto@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | Tue Jul 22 20:32:40 2014 +0900 |
committer | Qijiang Fan <fqj@google.com> | Thu Jun 04 17:10:40 2020 +0900 |
tree | 352ec58f897faca7beb430108f9347f15aaa86cc | |
parent | 27c6d9211824595f81119ecc4f012435ee8b2c28 [diff] |
Revert of High resolution timer fix for Windows (https://codereview.chromium.org/395913006/) Reason for revert: This patch seems to make following browser_tests flakey PPAPINaClNewlibTest.Graphics2D_FlushOffscreenUpdate NetInternalsTest.netInternalsHSTSViewAddOverwrite NetInternalsTest.netInternalsHSTSViewAddDelete NetInternalsTest.netInternalsHSTSViewAddTwice http://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%282%29/builds/34734 http://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%281%29/builds/32086 http://build.chromium.org/p/chromium.win/builders/Vista%20Tests%20%281%29/builds/47545 Original issue's description: > This is jamesr@ code I am landing. > > On Windows the message pump code tried to manage the systemwide timer resolution to fire delayed tasks with better than 15ms resolution but it was buggy: > > 1- A short task that was not followed by any other task will leave the systemwide timer pegged to 1ms > > 2- After we decided to crank up the timer we would 'lease' the timer for 1 second, for no good reason. > > Both issues are detrimental to battery power. > > The source of both problems is that we tried to decide with incomplete information. This patch solves that by having 1 bit for each pending task that requires a high resolution timer and a sum of the number of tasks that require high res timers. > > BUG=153139 > TEST=included here, also see the bug for manual testing. > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=284625 TBR=jamesr@chromium.org,darin@chromium.org,cpu@chromium.org NOTREECHECKS=true NOTRY=true BUG=153139 Review URL: https://codereview.chromium.org/407073004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@284664 0039d316-1c4b-4281-b951-d872f2087c98 CrOS-Libchrome-Original-Commit: 318717747560914c8052a2d072d20d0542c16fb7