Issue metadata
Sign in to add a comment
|
Tab switcher graphic glitch |
||||||||||||||||||||||
Issue descriptionDevice name:s7 From "Settings > About Chrome" Application version:66.0.3358.0 Operating system:7 Steps to reproduce: (1)open new tab (2)in new tab open context menu on any of last pages (3)click "open in new tab" and during animation (less than second) click on icon showing number of opened tabs Expected result: animation and all tab switcher elements Actual result: graphic mess
,
Mar 5 2018
Tested the issue in Android and able to reproduce the issue. Similar behavior is observed since Chrome #66.0.3358.0 Steps Followed: 1. Launched the Chrome Browser with the flag #enable-custom-context-menu enabled. 2. Navigate to the URL. Eg.: www.chip.pl 3. Long tap on any of the image, to see the context menu. 4. Tap on the option "Open in new tab". 5. Immediately tap on tab switcher button. (Button with tab numbers displayed.) 6. Observed a graphical glitch on the tabs selection screen. Chrome versions tested: 66.0.3358.0, 67.0.3361.0(Canary) OS: Android 8 Android Devices: Pixel, Pixel XL This seems to be a Regression issue as same behavior is seen since M66. Unable to provide the per revision bisect as the issue is not consistently reproduced. Please navigate to below link for log's and video-- go/chrome-androidlogs/818017 Other observations: 1. Issue seems to be observed only at the initial launch. 2. Observed 3 out of 10 times. 3. Later launched issue doesn't seems to appear.
,
Mar 6 2018
,
Mar 14 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/80aa15f4573dd3661393790f8e9db3349da30106 commit 80aa15f4573dd3661393790f8e9db3349da30106 Author: Matthew Jones <mdjones@chromium.org> Date: Wed Mar 14 18:30:45 2018 Make sure all animations have ended before switching layouts It is possible to make chrome switch layouts mid-animation. This patch kills running animations in LayoutManager#startShowing to avoid starting new animations in a bad state. BUG= 818017 ,820899 Change-Id: Ie591beedbf855a2ffd31baf849a334dd5230742b Reviewed-on: https://chromium-review.googlesource.com/958333 Reviewed-by: David Trainor <dtrainor@chromium.org> Commit-Queue: Matthew Jones <mdjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#543135} [modify] https://crrev.com/80aa15f4573dd3661393790f8e9db3349da30106/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManager.java
,
Mar 14 2018
,
Mar 15 2018
Verified fix in Latest "67.0.3371.0" release on Pixel running Android O.
,
Mar 19 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ff551a3b4b969e29586995f0e78194730603883b commit ff551a3b4b969e29586995f0e78194730603883b Author: Matthew Jones <mdjones@chromium.org> Date: Mon Mar 19 22:21:12 2018 Make sure all animations have ended before switching layouts It is possible to make chrome switch layouts mid-animation. This patch kills running animations in LayoutManager#startShowing to avoid starting new animations in a bad state. BUG= 818017 ,820899 TBR=mdjones@chromium.org (cherry picked from commit 80aa15f4573dd3661393790f8e9db3349da30106) Change-Id: Ie591beedbf855a2ffd31baf849a334dd5230742b Reviewed-on: https://chromium-review.googlesource.com/958333 Reviewed-by: David Trainor <dtrainor@chromium.org> Commit-Queue: Matthew Jones <mdjones@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#543135} Reviewed-on: https://chromium-review.googlesource.com/969648 Reviewed-by: Matthew Jones <mdjones@chromium.org> Cr-Commit-Position: refs/branch-heads/3359@{#329} Cr-Branched-From: 66afc5e5d10127546cc4b98b9117aff588b5e66b-refs/heads/master@{#540276} [modify] https://crrev.com/ff551a3b4b969e29586995f0e78194730603883b/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManager.java |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by pnangunoori@chromium.org
, Mar 5 2018