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

Issue 615209 link

Starred by 8 users

Issue metadata

Status: Duplicate
Merged: issue 609848
Owner: ----
Closed: May 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: Bug



Sign in to add a comment

Problems with Chrome 51 stable breaking some JavaScript apps

Project Member Reported by paulir...@chromium.org, May 26 2016

Issue description

There are some reports from the comments in the releases blog: http://googlechromereleases.blogspot.com/2016/05/stable-channel-update_25.html?showComment=1464285891280#c8670593883128351373

> This update completely broke my ability to work on developing my Javascript app. Check out at this page: http://www.ccpnc.ac.uk/magresview/magresview/magres_view.html, for example at times the developer console returns a ReferenceError when I ask for variables like "Jmol" which is fundamental to the code and is surely defined if the page works at all. Then it goes back to working regularly. Then it slows down when I use it from a local web server, or it returns "null" when I use jQuery to fetch a reference to an element. Any clue as to WTH is going on?

and 

> This release is causing havoc with our production application. Every user that upgrades from 50 to 51 is having serious problems.
> It may be related to Simone's issues above. We just realized that our problems are being caused by this release of Chrome.
 
@simone and blackat

My guess is you'll have some javascript exceptions in the console, which should start to indicate what is broken. 
Please share those and probably also share the line of code that is triggering the error?

It's possible we introduced a new javascript property/object that conflicts with something you were doing.
Additionally, if you can find a case where I can reproduce a page failing consistently, that'd be helpful.
Could this be the same issue that is breaking our Proxy script?  https://bugs.chromium.org/p/chromium/issues/detail?id=615084 
Potentially, if so then ` --disable-out-of-process-pac  ` should fix the problem.

Comment 4 by phi...@heede.dk, May 27 2016

@paul there is an open issue,  bug 609848 , for a known regression that is causing massive problems for users of older versions of the 'xstyle' JS library for working with CSS; the symptom is similar to what's described here (shows up as a "ReferenceError" in the javascript console) 

the change that introduced the bug in Chrome 51 is already reverted in canary builds but is pending review for merge into to current stable channel, anything you can do to speed that along would be very much appreciated by the many engineers otherwise working on patches to both SaaS both also software deployments that are affected
Thanks philip. I bet you're right and its 609848

I'll wait a little bit before I dupe into that, but that'll likely be the resolution. 

Comment 6 by gov...@chromium.org, May 27 2016

Cc: pbomm...@chromium.org sshruthi@chromium.org
Labels: ReleaseBlock-Stable M-51
Marking as M51 Stable blocker as this issue is reported by outside users. Please feel free to remove "ReleaseBlock-Stable" if you think otherwise. 

We're cutting M51 Stable RC early next week on Tuesday afternoon. pls make sure to land the fix and get it merged before then.

Comment 7 by gov...@chromium.org, May 27 2016

Labels: -Pri-2 Pri-1
> My guess is you'll have some javascript exceptions in the console, which should start to indicate what is broken. 
> Please share those and probably also share the line of code that is triggering the error?

The only exception I've observed is the ReferenceError that signals the problem itself when I try to access a variable, but clearly something must have gone wrong before that. Other than that, it's hard to identify a specific line of code where things have gone wrong - I assume it's somewhere in the huge minified libraries that I'm using but haven't written.
Hi, I'm having a similar problem.

Console is printing out the following:

    undefined:1 Uncaught (in promise)

For me this is happening when trying to export an image generated from "plotly.js". https://plot.ly/javascript/static-image-export/

I have attached a file which will reproduce the issue.

Hope this helps.
chrome-51.0.2704.63.html
1.4 KB View Download
Labels: Needs-Feedback
paulirish@, Would you mind providing an update on this.
Per comment #5, this is a dupe of 609848 which is fixed and merged to M51. Hence, we're moving forward with M51 release.
 
paulirish@, please let us know ASAP if there is anything else is still pending here. Thank you.
Mergedinto: 609848
Status: Duplicate (was: Unconfirmed)
AFAIK, this is a dupe of 609848
Hi,
We are also facing same JS broken issue; We are using the JQuery ("webkitMatchSelector" and "querySelectAll"; these functions are breaking after upgrade the latest version of chrome 51.0.2704.63 ).

please let us know the update for this. Thank you.
As per c#11 and c#12, this should be fixed in the next release of M51. Please update if that does not seem to be the case. 
This patch is now pushing out to stable channel in version 51.0.2704.79 for Desktop (Win,Mac & Linux).
>  We are using the JQuery ("webkitMatchSelector" and "querySelectAll"; these functions are breaking after upgrade the latest version of chrome 51.0.2704.63 ).

This is a different bug. However I believe that if you update your jQuery version, it is fixed there.

Sign in to add a comment