Ticket #2532096 (closed defect)

Reporter


Dav Glass
Opened: 04/2/12
Last modified: 04/17/12
Status: closed
Type: defect
Resolution: fixed

Owner


Ryan Grove
Target Release: 3.5.0
Priority: P1 (critical)
Summary: Chrome on Android hangs after loading CSS
Description:

When trying to load anything on Chrome for Android Y.Get hangs on the CSS (similar to the way it did on Opera).

I've tracked this down to Get's env.cssLoad being set to true when it needs to be set to false.

When I force it false, CSS starts loading again however it throws 2 test failures:

test: single css, failure
test: multiple css, failure

They both report "onSuccess shouldn't have been called"

Type: defect Observed in Version: development master
Component: Get Utility Severity: S1 (critical)
Assigned To: Ryan Grove Target Release: 3.5.0
Location: Library Code Priority: P1 (critical)
Tags: Relates To:
Browsers: N/A
URL: http://stage.yuilibrary.com/yui/docs/tabview/tabview-basic.html
Test Information:

Just install Chrome for Android and visit any example on stage that requires loading CSS.

Change History

Dav Glass

Posted: 04/2/12

After debugging this a little more, it seems that Chrome on Android reports Webkit 535.7 and Y.Get is forking for >= 535.24 (it assumes that Chrome 19 === 5.3.5.24).

First, I added proper Y.UA detection for Chrome on Android:
https://github.com/davglass/yui3/commit/118b2b253667c51a1cff0367ecde38cb69f7b2eb

Then I updated Y.Get to set it's env.cssLoad to false for this browser:
https://github.com/davglass/yui3/commit/42a6544f291365e9f5791d3b076030fd5f3804bc

I also updated the test to make sure they are flagged as cssFail is not supported on this browser.

Jenny Donnelly

YUI Developer

  • Username: jenny
  • GitHub: jenny
Posted: 04/2/12

Ok for 3.5.0.

Dav Glass

Posted: 04/2/12
  • resolution changed to fixed
  • status changed from new to checkedin

Fixes #2532096 - Force Chrome on Android to poll for dynamic CSS
View Commit: 64098e1f6c9d15a0cdd615f6544053871c58364e

Dav Glass

Posted: 04/10/12

Fixes #2532096 - Force Chrome on Android to poll for dynamic CSS
View Commit: 64098e1f6c9d15a0cdd615f6544053871c58364e

Jenny Donnelly

YUI Developer

  • Username: jenny
  • GitHub: jenny
Posted: 04/17/12
  • status changed from checkedin to closed

Closing out bugs checked in to 3.5.0.