Issue metadata
Sign in to add a comment
|
Omnibox flashes white briefly when switching to a different tab via tab switcher |
||||||||||||||||||||||
Issue descriptionRequires NTPMaterialDesign feature flag to be enabled. 1) Open 2 new tabs (make sure they are on the NTP) 2) Tap on the tab switcher to switch tabs 3) Switch to a different tab from the foreground tab Observe: a brief white flash where the omnibox is, before going back to gray. This happens each time the foreground tab is switched.
,
May 17 2016
How awesome is this. Let me guess: Technically this is not a regression, i.e. the white flashing existed before - but now it has become visible as we changed the background color. Yay! We'll treat this as a regression for UX reasons.
,
May 17 2016
Issue 612082 has been merged into this issue.
,
May 18 2016
I can reproduce this on yesterday evening's Dev release 52.0.2739.0
,
May 19 2016
Still reproducible on yesterday evening's Dev release 52.0.2739.3 (which is less than 24h fresher).
,
May 19 2016
Woo! I can't reproduce this in May's custom APK anymore!
,
May 19 2016
,
May 19 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/4648bfe49691cfcedcb719c349c1c524aac7bd3b commit 4648bfe49691cfcedcb719c349c1c524aac7bd3b Author: maybelle <maybelle@chromium.org> Date: Thu May 19 18:06:20 2016 Fix omnibox flashing on phones when switching between foreground/background NTPs BUG= 612114 Review-Url: https://codereview.chromium.org/1993253002 Cr-Commit-Position: refs/heads/master@{#394812} [modify] https://crrev.com/4648bfe49691cfcedcb719c349c1c524aac7bd3b/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManagerDocument.java [modify] https://crrev.com/4648bfe49691cfcedcb719c349c1c524aac7bd3b/chrome/android/java/src/org/chromium/chrome/browser/util/ColorUtils.java
,
May 20 2016
Should we mark this regression bug as fixed and file a separate one for the general improvements to fix this for good?
,
May 25 2016
Issue 614673 has been merged into this issue.
,
May 25 2016
Marking as fixed and filed Issue 614690 to track the remaining omnibox flashing issues (which are also technically not a regression cause by the MD changes, since I can reproduce those on stable)
,
Jun 7 2016
Issue 614673 has been merged into this issue.
,
Jun 7 2016
,
Jul 1 2016
,
Jul 1 2016
,
Jul 1 2016
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by bauerb@chromium.org
, May 16 2016