Issue metadata
Sign in to add a comment
|
Regression:Weird flickering is observed on rotating cube
Reported by
adha...@etouch.net,
Jun 17 2016
|
||||||||||||||||||||||
Issue descriptionChrome Version:53.0.2769.2 (Official Build) 67c1583dc8789efc61a90031de836e361e7635e3-refs/branch-heads/2769@{#3} 32/64-bit. OS:Windows (7,8,8.1,10) Test URL: http://paulrhayes.com/experiments/cube-3d/ Steps: 1.Launch chrome,navigate to above URL. 2.Press up/down arrow keys to rotate the cube,observe. Actual: Weird flickering is observed on rotating the cube. Expected: Weird flickering should not be seen on rotating the cube. This is a regression issue, broken in M-53 series,will soon update the other info.
,
Jun 17 2016
,
Jun 17 2016
Adding RB label as this is a recent regression.
,
Jun 21 2016
@jaydasika: Could you please provide an update on this issue. Thank you.
,
Jun 21 2016
Unable to reproduce on ToT on Linux.
,
Jun 22 2016
I was able to repro on dev on Linux. Not seeing it in stable or canary.
,
Jun 22 2016
Whoops, I meant not seeing it in stable or beta.
,
Jun 22 2016
I don't see it on dev. Are you running it with some flags ?
,
Jun 22 2016
Ah, yes, good point. I forgot about that. I have experimental web platform features enabled on dev. But then I just turned off the flag and I'm still seeing the occasional jump. I'm in the process of updating my checkout and will see if I can repro on ToT.
,
Jun 22 2016
I'm seeing the same jumps with and without --enable-experimental-web-platform-features on ToT.
,
Jun 22 2016
Video attached of repro on dev on Linux, no special flags enabled.
,
Jun 24 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/94bcfedec292cc72e7b98cea38da392fbf1dc18a commit 94bcfedec292cc72e7b98cea38da392fbf1dc18a Author: jaydasika <jaydasika@chromium.org> Date: Fri Jun 24 23:29:01 2016 cc: Fix property tree animation sync bug Property trees animation sync at commit depends on the value of is_currently_animating value of a property tree node on the main thread. So, we need to do this sync before calling AnimationHost::PushProperties as AnimationHost::PushProperties can delete animations and change the is_currently_animating value. BUG= 620978 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review-Url: https://codereview.chromium.org/2090793002 Cr-Commit-Position: refs/heads/master@{#402011} [modify] https://crrev.com/94bcfedec292cc72e7b98cea38da392fbf1dc18a/cc/trees/layer_tree_host.cc [modify] https://crrev.com/94bcfedec292cc72e7b98cea38da392fbf1dc18a/cc/trees/layer_tree_host_unittest_animation.cc
,
Jun 27 2016
,
Jun 27 2016
[Automated comment] Commit may have occurred before M53 branch point (6/30/2016), needs manual review.
,
Jun 27 2016
Whoops, my bad. This doesn't need a merge
,
Jun 27 2016
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by adha...@etouch.net
, Jun 17 2016Owner: f...@opera.com
Status: Assigned (was: Unconfirmed)