New issue
Advanced search Search tips

Issue 645098 link

Starred by 2 users
Status: Fixed
Owner:
sunyunjia@chromium.org
Closed: Sep 2016
Cc:
dtapu...@chromium.org
bokan@chromium.org
majidvp@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Use monotonic time instead of current time in autoscroll controller

Project Member Reported by dtapu...@chromium.org, Sep 8 2016 
Use of a wall clock to drive an animation is dangerous. We should be using the monotonic time.

Basically implement this FIXME:

https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/page/Au
toscrollController.cpp?dr&q=AutoscrollController.cpp&sq=package:chromium&l=236
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 8 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7d69521b65cd8d03eabe16411c7198dd1520d382

commit 7d69521b65cd8d03eabe16411c7198dd1520d382
Author: sunyunjia <sunyunjia@chromium.org>
Date: Thu Sep 08 19:39:42 2016

Use monotonic time instead of current time in autoscroll controller
Use of a wall clock to drive an animation is dangerous. We should be using the monotinic time.

BUG= 645098 

Review-Url: https://codereview.chromium.org/2326643002
Cr-Commit-Position: refs/heads/master@{#417365}

[modify] https://crrev.com/7d69521b65cd8d03eabe16411c7198dd1520d382/third_party/WebKit/Source/core/page/AutoscrollController.cpp

Comment 2 by sunyunjia@chromium.org, Sep 8 2016

Status: Fixed (was: Assigned)
►

Sign in to add a comment