diff options
| author | fmeawad <fmeawad@chromium.org> | 2014-08-26 13:10:45 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2014-08-26 20:12:51 +0000 |
| commit | 05399593156ef3b8d96933258cd32fe716f63461 (patch) | |
| tree | 78a2765a70904253a57f46a95fa03c78dafc860f /content/app/content_main_runner.cc | |
| parent | 07aef0e93387ef992facfdb9c22716a400f85c46 (diff) | |
| download | chromium_src-05399593156ef3b8d96933258cd32fe716f63461.zip chromium_src-05399593156ef3b8d96933258cd32fe716f63461.tar.gz chromium_src-05399593156ef3b8d96933258cd32fe716f63461.tar.bz2 | |
We have noticed a clock shift when QPC was deployed. It shows as a regression on the perf bots
https://chromeperf.appspot.com/report?masters=ChromiumPerf&bots=chromium-rel-win8-dual&tests=startup.warm.dirty.blank_page%2Fwindow_display_time&rev=286928
It is not a real regression, the initial_time and initial_ticks are not properly initialized when switching to HighResolution (i.e. QPC).
This CL initializes the now_function to the HighResNowWrapper instead of setting it to RolloverProtectedNow then to the HighResNowWrapper.
By doing that, we avoid getting an incorrect initial_time and initial_ticks using the RolloverProtectedNow and avoid having to reinitialize.
BUG=158234
Review URL: https://codereview.chromium.org/446203002
Cr-Commit-Position: refs/heads/master@{#291974}
Diffstat (limited to 'content/app/content_main_runner.cc')
| -rw-r--r-- | content/app/content_main_runner.cc | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/content/app/content_main_runner.cc b/content/app/content_main_runner.cc index e9aebce..5827ae2 100644 --- a/content/app/content_main_runner.cc +++ b/content/app/content_main_runner.cc @@ -661,8 +661,6 @@ class ContentMainRunnerImpl : public ContentMainRunner { MachBroker::ChildSendTaskPortToParent(); } #elif defined(OS_WIN) - base::TimeTicks::SetNowIsHighResNowIfSupported(); - bool init_device_scale_factor = true; if (command_line.HasSwitch(switches::kDeviceScaleFactor)) { std::string scale_factor_string = command_line.GetSwitchValueASCII( |