610178 - [IE] download-stats.js throwing: 'd.get(...)' is null or not an object

Status: VERIFIED FIXED

(www.mozilla.org :: General, defect)

Description

[Reed Loden [:reed]]

Had to install Firefox on a Win7 box today, and I got this error message when using IE8 for its only use case (downloading Firefox, of course):

Webpage error details

User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.1; WOW64; Trident/4.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0)
Timestamp: Sun, 7 Nov 2010 05:51:01 UTC


Message: 'd.get(...)' is null or not an object
Line: 13
Char: 7
Code: 0
URI: http://www.mozilla.com/firefox/stats/js/download-stats.js


Message: 'd.get(...)' is null or not an object
Line: 13
Char: 7
Code: 0
URI: http://www.mozilla.com/firefox/stats/js/download-stats.js

Comment 1

[Alex Buchanan [:abuchanan]]

Attachment: patch, v1

Comment 2

[Steven Garrity [:sgarrity]]

Patch looks good from here. Should probably still drop the including of this file on pages that don't need it though.

Comment 3

[Alex Buchanan [:abuchanan]]

(In reply to comment #2)
> Patch looks good from here. Should probably still drop the including of this
> file on pages that don't need it though.

Heh, good point.  I'll do that.

Comment 4

[Alex Buchanan [:abuchanan]]

r77451 on trunk

Raymond, this affects these pages,

Sending        trunk/en-US/firefox/firefox.html
Sending        trunk/en-US/firefox/ie-test-footer-a.html
Sending        trunk/en-US/firefox/ie-test-footer-b.html
Sending        trunk/en-US/firefox/ie-test-footer-c.html
Sending        trunk/en-US/firefox/ie.html
Sending        trunk/en-US/firefox/personal.html
Sending        trunk/en-US/firefox/stats/js/download-stats.js
Sending        trunk/en-US/firefox/upgrade.html
Sending        trunk/pt-BR/firefox/index.html

Comment 5

[raymond [:retornam] (inactive)]

qa-verified
http://www-trunk.stage.mozilla.com/en-US/firefox/ie.html
http://www-trunk.stage.mozilla.com/en-US/firefox/firefox.html
http://www-trunk.stage.mozilla.com/en-US/firefox/ie-test-footer-a.html
http://www-trunk.stage.mozilla.com/en-US/firefox/ie-test-footer-b.html
http://www-trunk.stage.mozilla.com/en-US/firefox/ie-test-footer-c.html
http://www-trunk.stage.mozilla.com/en-US/firefox/personal.html
http://www-trunk.stage.mozilla.com/pt-BR/firefox/index.html
http://www-trunk.stage.mozilla.com/en-US/firefox/upgrade.html

Comment 6

[Alex Buchanan [:abuchanan]]

on production

Comment 7

[raymond [:retornam] (inactive)]

http://www.mozilla.com/en-US/firefox/ie.html
http://www.mozilla.com/en-US/firefox/firefox.html
http://www.mozilla.com/en-US/firefox/ie-test-footer-a.html
http://www.mozilla.com/en-US/firefox/ie-test-footer-b.html
http://www.mozilla.com/en-US/firefox/ie-test-footer-c.html
http://www.mozilla.com/en-US/firefox/personal.html
http://www.mozilla.com/pt-BR/firefox/index.html
http://www.mozilla.com/en-US/firefox/upgrade.html