DescriptionRevert of Remove burst mode from VSyncMonitor. (https://codereview.chromium.org/346813002/)
Reason for revert:
Four of the introduced tests time out on Android.
VSyncMonitorTest#testVSyncActivationFromIdleAllowJBVSync
VSyncMonitorTest#testVSyncActivationFromIdleDisallowJBVSync
VSyncMonitorTest#testVSyncPeriodAllowJBVSync
VSyncMonitorTest#testVSyncPeriodDisallowJBVSync
The log output can be seen here:
http://build.chromium.org/p/chromium.linux/builders/Android%20Tests%20%28dbg%29/builds/21188/steps/contentshell_instrumentation_tests/logs/stdio
This failure could be observed in the try-bot results as part of the patch. Please do not use NOTRY unless you're absolutely certain that the failures are unrelated.
Original issue's description:
> Remove burst mode from VSyncMonitor.
>
> Extra vsyncs after each rendering cause 1% more power consumption.
>
> NOTRY=true
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=279997
TBR=tedchoc@chromium.org,sievers@chromium.org,aruslan@chromium.org,brianderson@chromium.org,skyostil@google.com,skyostil@chromium.org,bulach@chromium.org,miguelg@chromium.org,yfriedman@chromium.org,newt@chromium.org,egor.starkov@samsung.com
NOTREECHECKS=true
NOTRY=true
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=280012
Patch Set 1 #
Created: 6 years, 6 months ago
(Patch set is too large to download)
Messages
Total messages: 3 (0 generated)
|