1231568 - inbox.google.com never finishes loading in latest inbound builds

Status: RESOLVED WORKSFORME

(Core :: JavaScript Engine, defect)

Description

[Kostas]

To reproduce:
- install firefox 45.0a1 x64 inbound 2015-12-09 (fresh profile) in win10 x64 (e10s enabled) 
- open inbox.google.com and login
- Inbox starts loading but never finishes: https://i.imgur.com/DohD29r.jpg

There's no relevant entry in Web Console while this occurs.
The issue also occurs with e10s disabled.


Regression range:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=d55129069a1f56f63573282911ad2dac17d6e7b1&tochange=94f4cfa98356714223ee71774f27a9153b6ab188

Comment 1

[André Bargull [:anba]]

Inbox uses a certain try-catch pattern which breaks with Annex B block scoping, see [1] for more information.

[1] https://github.com/tc39/ecma262/issues/162

Comment 2

[Raul Nicolae Malea]

Confirm this bug in openSUSE 13.2, x64

Comment 3

[Jonny Barnes]

Confirming this bug on Nightly on OSX.

Comment 4

[Filippo]

Confirming this bug on Nightly on Win 10.

Comment 7

[James Willcox (:snorp) ([e-mail]) (he/him)]

(In reply to André Bargull from comment #1)
> Inbox uses a certain try-catch pattern which breaks with Annex B block
> scoping, see [1] for more information.
> 
> [1] https://github.com/tc39/ecma262/issues/162

Interesting. Are we going to use a workaround for inbox (and other GWT apps, I guess) or what's the plan?

Comment 8

[Chris Peterson [:cpeterson]]

[Tracking Requested - why for this release]:

Bug 1071646 broke Google Inbox in Nightly 45.

Comment 9

[Shu-yu Guo [:shu]]

Backed out for now: https://hg.mozilla.org/integration/mozilla-inbound/rev/d2bec6ed7b30

The plan is to wait for GWT and Inbox to fix their code. Chrome is waiting on the same thing. I will try to reland when that is fixed.

Comment 10

[Sylvestre Ledru [:Sylvestre]]

Shu-yu, could you backout also the change in aurora (45)? 
FYI, I pinged a friend working at Google on GWT.

Comment 11

[Jan de Mooij [:jandem]]

(In reply to Sylvestre Ledru [:sylvestre] from comment #10)
> Shu-yu, could you backout also the change in aurora (45)? 
> FYI, I pinged a friend working at Google on GWT.

I think this was backed out from 45 and relanded in 46, but let's wait for shu to confirm.

Comment 12

[Shu-yu Guo [:shu]]

Yes, it was backed out then relanded.

Comment 13

[Sylvestre Ledru [:Sylvestre]]

It has been fixed on the inbox side, untracking it.

Comment 14

[Sebastian Hoß]

I think this is happening again starting with at least 51.0a2 up until the latest 52.0a2. I'm getting a "Loading" spinner in the lower left corner as described in the initial description that never finishes loading. Seeing this on Ubuntu 14.04, 16.04 and 16.10.