New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 627066 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Feb 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Chrome browser UI freezes up when leaving party.robinglauser.ch

Reported by ahalder@etouch.net, Jul 11 2016

Issue description

Application Version: 54.0.2793.0
Android Build Number: 6.0.1/MOB30N
Device: Spice Mi-498

Pre-condition:
1. No tabs should be open in Chrome

Steps to reproduce:
1. Launch Chrome > Go to party.robinglauser.ch
2. Long tap on the disco ball image and choose to Open it in new tab
3. Switch to the 2nd tab now > Observe no .gif file opened by Chrome and also observe the URL Omnibox
4. Tap on URL Omnibox now > Observe
5. Switch back to 1st tab now > Observe


Observed behavior:
Chrome becomes extremely slow and is unable to open .gif file in new tab from party.robinglauser.ch web page

Expected behavior:
Chrome should work fluidly and should be able to open .gif file in new tab from party.robinglauser.ch

Frequency: 
<5/5>

Additional comments:
This is a regression issue
This issue is not present on latest M52-52.0.2743.73 and M53-53.0.2785.13

Last Good Build: 54.0.2790.2
First Bad Build: 54.0.2793.0

This issue is seen on Android devices ex. Spice Mi-498 (6.0.1/MOB30D), Karbonn Sparkle V (5.1.1/LMY47V), Samsung Galaxy J2 (5.1.1/LMY47X), Samsung Galaxy S3 (4.3/JSS15J), Samsung Galaxy J7 (5.1.1/LMY48B), Samsung Galaxy S4 (5.0.1/LRX22C), Lenovo A6000 (5.0.2/LRX22G) and Moto G 2nd Gen (6.0/MPB24)
 
Please find logs and video @ http://go/chrome-androidlogs1/6/627066
Labels: M-54
Owner: tedc...@chromium.org
Status: Assigned (was: Unconfirmed)

Comment 3 by ahalder@etouch.net, Jul 13 2016

Also note: After step 3,
Chrome automatically enters tab switcher mode if u swipe 2nd tab left or right, even though the tab switcher button isn't tapped (refer the screen record for a clear view)
Owner: aelias@chromium.org

Comment 5 by aelias@chromium.org, Jan 24 2017

Cc: aelias@chromium.org
Labels: -Restrict-View-Google -Pri-3 Pri-2 Type-Bug
Owner: mdjones@chromium.org
Summary: Chrome browser UI freezes up when leaving party.robinglauser.ch (was: Chrome becomes extremely slow and is unable to open a .gif file in new tab from party.robinglauser.ch web page)
Still repro on M56.  This page has the gimmick that it continually changes the tab theme color at 60FPS.  Artificial stress test, but it probably can repro in a normal scenario with the wrong timing.  Range contains https://codereview.chromium.org/2114513002 "Fade theme color in tab switcher", I suspect that's the culprit.
Since the animation never ends, the layout never exits the tab switcher:

https://cs.chromium.org/chromium/src/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/Layout.java?rcl=0&l=882

I think the problem is that the layout tab animations shouldn't block layouts but still need to be updated. I'm looking at a fix now.
Project Member

Comment 7 by bugdroid1@chromium.org, Jan 31 2017

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

commit 27f8a728708b1ee8a3db41afcf47537a601d5abf
Author: mdjones <mdjones@chromium.org>
Date: Tue Jan 31 00:44:31 2017

LayoutTab fade animations should not block layouts

Layouts in chrome do not finish transitions until all animations are
completed. Because of this, tab driven animations can block chrome's
UI. Particularly, theme color transitions will cause color fading
effects in the tab switcher. For example, if the tab is updating the
theme color faster than the animations complete, the animations never
end. These animations should not be considered when performing layout
transitions that are critical to chrome's function.

After this change, LayoutTab animations will no longer affect a layout
animation's completion state. If the animation is not complete, it
will request an update from the renerer, but it will no longer block
the UI.

BUG= 627066 

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

[modify] https://crrev.com/27f8a728708b1ee8a3db41afcf47537a601d5abf/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/Layout.java

Comment 8 by ahalder@etouch.net, Feb 14 2017

This issue is fixed on latest M58-58.0.3012.0 Thanks!
Status: Verified (was: Assigned)

Sign in to add a comment