Chromium Code Reviews
DescriptionRevert of Enable VsyncAlignedInput to Field Trial (patchset #1 id:40001 of https://codereview.chromium.org/2752363002/ )
Reason for revert:
Reverting due to issue 704732. Will try to find the root cause before re-landing.
Original issue's description:
> Enable VsyncAlignedInput to field trial
>
> Enable Field Trial on Android, ChromeOS, Linux, Mac, Win.
>
> This patch will cause expected 1 frame delay in
> "scheduler.tough_scheduling_cases.first_gesture_scroll_update_latency".
>
> Reasons for the delay:
> The synthetic input controller sends |TouchMove| right before |BeginFrame|, which
> means the ack will come back later and the generated |GestureScrollUpdate| will falls
> into next |BeginFrame|.
>
> This is fine since the actual scroll result won't be rendered until next frame with or
> without this patch.
>
> BUG=704134
>
> Review-Url: https://codereview.chromium.org/2752363002
> Cr-Commit-Position: refs/heads/master@{#458921}
> Committed: https://chromium.googlesource.com/chromium/src/+/5a7f45dab8372cb7b80ee24dfb137a7ced2049ea
TBR=isherman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=704134
Review-Url: https://codereview.chromium.org/2791663002
Cr-Commit-Position: refs/heads/master@{#461202}
Committed: https://chromium.googlesource.com/chromium/src/+/232fbc4bc87a4c35ea8ab60e1f7ff8e4e8e14de5
Patch Set 1 #
Messages
Total messages: 6 (3 generated)
|
|||||||||||||||||||