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

Issue 780481 link

Starred by 5 users

Issue metadata

Status: Duplicate
Merged: issue 785233
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Latest update breaks dojo

Reported by andy.for...@gmail.com, Nov 1 2017

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/62.0.3202.75 Safari/537.36

Steps to reproduce the problem:
1. View the attached HTML in Chrome 62.0.3202.75
2. Notice blank page
3. Right click, View Source
4. Notice properly rendered page

What is the expected behavior?
The page should show a very simple dojo application

What went wrong?
Previous versions of Chrome properly displayed the page. The 62.0.3202 version will not.

Did this work before? N/A 

Chrome version: 62.0.3202.75  Channel: stable
OS Version: 10.0
Flash Version:
 
test.html
1.6 KB View Download
Labels: Needs-Triage-M62
Cc: sc00335...@techmahindra.com
Labels: Triaged-ET M-64 OS-Linux OS-Mac
Status: Untriaged (was: Unconfirmed)
Able to reproduce this issue on reported verison 62.0.3202.75 and on latest canary 64.0.3256.0 using Windows 10,Ubuntu 14.04 and Mac 10.12.6 with steps mentioned in comment#0. Issue is seen in firefox as well.

This issue is seen from M50[50.0.2661.0]. Hence considering this as Non-regression and marking this as Untriaged

Comment 3 by e...@chromium.org, Nov 7 2017

Labels: Needs-Feedback Needs-Reduction
Given that it has been broken for over ten releases and is also broken in Firefox I'd assume this was due to an intentional web compat change. Does it work in Safari or Edge?
It does work correctly in Safari and Edge.

Comment 5 by kochi@chromium.org, Nov 9 2017

Have you contacted Dojo project itself?
If they have any idea where is broken in Firefox and Chrome, we may able to fix the issue
(if this is a real compat issue).

BTW, I tried this (with modification on line 9, adding https: schema to the URL) on
Chrome 62 and Safari 11.0.1, and both failed with similar error message and only
saw black screen.

Chrome:
> Uncaught TypeError: this.transitionTo is not a function
Safari:
> TypeError: this.transitionTo is not a function. (In 'this.transitionTo("logIn")', 'this.transitionTo' is undefined)

I can't believe I left that test code in there. There is no reason to have the transitionTo logic. The thing should just work as it is. I'll attach a new and simpler version that does reproduce the issue. I did test this with Dojo 1.13.0 and the problem remains. Interestingly, Dojo version 1.7 seems to work. I need to do more testing to confirm this. The reason it was undiscovered for so long is that, on Windows, Chrome never insisted upon an upgrade until version 62.0. I suspect that this issue will become fairly common now that the user base is upgrading Chrome.

I have not contacted Dojo. I will... but this really looks like a Chrome issue. It works with Edge.
test.html
1.5 KB View Download

Comment 7 by kochi@chromium.org, Nov 10 2017

Cc: kochi@chromium.org
Thanks for uploading the test!
I think I reproduced the issue locally (Mac Chrome, M62 here).

1. Open test.html
2. See only solid black screen
3. Resize the window by narrowing enough to see "Log in" UI

(View source didn't work for me, btw).
I'd also thank if Dojo people have any clue what is happening here.

Comment 8 by kochi@chromium.org, Nov 13 2017

Labels: -Type-Bug Needs-Bisect Type-Bug-Regression
As the reporter says it worked before and at least 62.0.3202 the case does not work -
marking this as regression.

Can we have bisect info for this?
We found a javascript / dojo fix for this. The original code attempted to parse the DOM prior to the browser having the DOM ready. 

The new code should be:

    <script>
        require([
            "dojox/mobile/parser",
            "dojo/ready",
            "dojox/mobile/View",
            "dojox/mobile/Heading"

        ], function (parser, ready) {
            ready(function () {
                parser.parse();
            })


        });
    </script>

Now, we wait for the DOM to be ready prior to parsing it. 

So, it turns out this was a timing issue. I suspect this issue can now be closed since we have a best practice within the dojo framework to deal with exactly this issue.

Thank you for your attention.

Comment 10 by kochi@chromium.org, Nov 14 2017

Cc: -kochi@chromium.org
Components: -Blink
Owner: kochi@chromium.org
Status: Assigned (was: Untriaged)
Thanks for the feedback!  We really appreciate you found the solution yourself.

Just in case this is a real regression that timing change caused some
incompatibility, I'd like to wait for bisect analysis before closing this.

Comment 11 by kochi@chromium.org, Nov 17 2017

Labels: -Needs-Feedback
Isn't "Needs-Bisect" flag enough to ask someone in testing team to do
bisection for this issue?
Cc: kochi@chromium.org
Components: Blink>CSS
Labels: -Pri-2 -Needs-Bisect -M-64 hasbisect-per-revision M-62 Pri-1
Owner: treib@chromium.org
Able to reproduce this issue on reported version 62.0.3202.75 and on latest stable 62.0.3202.94 but fixed on latest beta 63.0.3239.52 and latest dev 64.0.3269.3 using Windows 10, Ubuntu 14.04 and Mac 10.12.6. Hence Providing reverse bisect info.

Reverse Bisect Info:
================
Last Bad Build: 63.0.3225.0
First Good Build: 63.0.3226.0 

You are probably looking for a change made after 504560 (known good), but no later than 504561 (first known bad).
CHANGELOG URL:
  https://chromium.googlesource.com/chromium/src/+log/a9ba55c8571ab0d2624c83d413ccc7215a185f51..9f0c480da9cb188c82e16e6c6751875a60a114cf

Probably fixed by : https://chromium-review.googlesource.com/684186

@rune:Could you please confirm if its safe to merge to M-62 in case we have stable refresh?

Unable to assign to @rune. Hence assigning to @treib: Please check the above issue.

Thanks!

Comment 13 by treib@chromium.org, Nov 20 2017

Cc: nainar@chromium.org
Owner: futhark@chromium.org
Mergedinto: 785233
Status: Duplicate (was: Assigned)
Duplicate of  issue 785233 . We considered it to be to risky to backport to 62 because there are multiple patches which would need to be merged, and 63 going stable on December 5th.

Sign in to add a comment