639190 - bankofamerica.com - Site navigation dropdown menus don't work with Firefox 4

Status: RESOLVED WORKSFORME

(Tech Evangelism Graveyard :: English US, defect)

Description

[Damon Sicore (:damons)]

Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b13pre) Gecko/20110303 Firefox/4.0b13pre

STR:

1) Visit http://bankofamerica.com

2) Hover over the main buttons/tabs next to the red login box, any of the Bank, Borrow, Invest, Protect, or Plan items.

Expected results:  Hover should produce links under each item.  For example, hovering over Bank should show Checking, Savings, etc.

Actual results:  Hover has no effect, making all links inaccessible. 

Works fine in Chrome and Safari.  If it's not an client/agent sniffing site bug, this would block.  Nominating.

Comment 1

[Mats Palmgren (inactive)]

Regression range:  2009-12-01-03 -- 2009-12-02-03
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=b7f3c1c9fb57&tochange=a5e1b195ecf7
I'm guessing the tracemonkey merge is the culprit.

Comment 2

[Ed Morley [:emorley]]

Using tracemonkey...

Last good nightly: 2009-11-19 
First bad nightly: 2009-11-20

Pushlog: 
http://hg.mozilla.org/tracemonkey/pushloghtml?fromchange=c84622414ed3&tochange=2ca2a236da89

Errors in console...

Error: Image corrupt or truncated: <unknown>
Source File: <unknown>
Line: 0

Error: assignment to undeclared variable boa
Source File: https://www.bankofamerica.com/pa/gzip_362775863/global-assets/1.0/bundles/script/homepagebottomjawr-1.1.js
Line: 1

Given the error console message, the Tracemonkey range and the fact that this file:
https://www.bankofamerica.com/pa/gzip_362775863/global-assets/1.0/bundles/script/homepagebottomjawr-1.1.js
...starts with 'use strict'; believe this is Tech Evang, due to this changeset:

Bug 514560: Forbid assignments to undeclared variables in strict mode code. r=jorendorff

tl;dr
The site is incorrectly using 'use strict' at the start of the .js file mentioned above; Firefox is working correctly, per the ES5 Strict Mode spec.

Comment 3

[Mats Palmgren (inactive)]

I agree with that analysis.  I removed the "use strict"; at the start
of that file and that made the menus work, so it's likely the only
fix that needs to be done on their side.  BTW, the file I got when
saving the site was named "homepagebottomjawr-1.js" not "1.1".

Comment 4

[Ed Morley [:emorley]]

As a by the by, the menus don't work in Chrome 11.0.686.3 dev either (presume it supports ES5 Strict Mode too).

Comment 5

[Ed Morley [:emorley]]

I have emailed Domain.Administrator@ , hostmaster@ & webmaster@ a short message informing them of the fault and provided a link back to here. (Trying to use the contact form on the website wasn't proving very successful, so just had a peak at the emails listed on their whois entry).

Comment 6

[Damon Sicore (:damons)]

Thanks for the analysis here guys.  Solid work!  :)

Comment 7

[Mike Beltzner [:beltzner, not reading bugmail]]

This had it's blocking nomination cleared and was moved to TE, but do we really think we'd ship a browser if it didn't work on BoA?

Comment 8

[Ed Morley [:emorley]]

Online banking appears to work, it's 'just' the dropdown menus for the rest of site navigation.

See also bug 625724.

Comment 9

[Brendan Eich [:brendan]]

(In reply to comment #7)
> This had it's blocking nomination cleared and was moved to TE, but do we really
> think we'd ship a browser if it didn't work on BoA?

BofA needs to fix its content. We're not going to ignore "use strict" or add a "strict quirks" mode. Maybe Kev can help.

/be

Comment 10

[Ed Morley [:emorley]]

No replies yet to the emails sent in comment 5; site still broken.

Any other points of contact?

Comment 14

[(mostly gone) XtC4UaLL [:xtc4uall]]

This is WFM/Fixed now.

Comment 15

[j.j.]

-> WORKSFORME based on comment 14