1156624 - Wikisky.org will not display properly in FF 64x 40.0a1 Apr. 20 2015

Status: RESOLVED WORKSFORME

(Core :: JavaScript Engine, defect, P1)

Description

[Helge Skjeveland]

User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:40.0) Gecko/20100101 Firefox/40.0
Build ID: 20150420030204

Steps to reproduce:

Type wikisky.org or sky-map.org into address line, hit {Enter}


Actual results:

Web site opens, but doesn't display default Andromeda galaxy screen, but a placeholder screen. MSIE 11 and Opera 28.0 work fine with the URL. None of my history URLs for these with parameters open the interactive screen either. It used to work fine with x64 FF. Tried turning off all sorts of security lockouts, adding the URLs to whitelists, w/o effect.


Expected results:

Should display Andromeda galaxy or any astronomical object whose name is typed into the Find Object field.

Comment 1

[Alice0775 White]

Error in Error Console:

Error: SyntaxError: for-in loop head declarations may not have initializers
Source File: http://server3.wikisky.org/jsc/coolbar|astro_math|astro_labels|planetPos|DWR_engine|DWR_ChatService|DWR_CoolPlaces|DWR_TopUsers|DWR_RSS|DWR_Search|DWR_Login|rgb_color|rounded_corners|dialogs|login|chatDWR|RSS_panel|cool_places_panel|homepage
Line: 314, Column: 410
Source Code:
hichCorners,options:options};};this.init=function(){for(var targetCounter=0 in this.targets){var trgt=this.targets[targe



Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=40f06a0c961b&tochange=76326ec30cd6

Triggered by:
	dfdcce6b319a	Jeff Walden — Bug 748550 - Remove support for |for (... = ... in/of ...)| now that ES6 has removed it. r=jorendorff


So, This should be TE bug.

Comment 2

[Helge Skjeveland]

Also happens on my laptop in its Win8.1 64x installation, so is not 7 specific.

Comment 3

[Helge Skjeveland]

Also happens if -safe-mode is specified, so addons & etc. aren't a factor.

Comment 4

[Jeff Walden [:Waldo]]

Sent mail to their dev@ contact address telling them what the problem was.  We'll see what sort of response that gets.

Comment 5

[Boris Zbarsky [:bzbarsky]]

[Tracking Requested - why for this release]: Web compat breakage.

Comment 6

[jen strickland :jenstrickland]

Tracking for 40.

Comment 7

[Helge Skjeveland]

Continues not to work in 41.0a1. Works in Opera 29.0.1795.47 (x32 browser), only partially in MSIE 11.0.9600.17728 (x64 browser, and yellow on-hover info boxes don't appear reliably, do in Opera 29 x32). Haven't tested in 32-bit FF.

Comment 8

[Helge Skjeveland]

Now works in 41.0a1 2015-05-22. Whether changes to FFx64 or WikiSky did it, I'm marking this RESOLVED WORKSFORME.

Comment 9

[Jeff Walden [:Waldo]]

That'd be bug 1164741, sadly.  The script in comment 1 still has the bad syntax in it; we're just ignoring the initializer now, instead of considering it a syntax error.