405104 - "Error: theDocument is not defined" and other js errors in pageInfo

Status: RESOLVED FIXED

(SeaMonkey :: Page Info, defect)

Description

[Philip Chee]

Error: theDocument is not defined
Source file: chrome://navigator/content/pageinfo/pageInfo.js
Line: 651

From:
<https://bugzilla.mozilla.org/attachment.cgi?id=281812>
-// mmm, yummy. global variables.
-var theWindow = null;
-var theDocument = null;
[...]
+// mmm, yummy. global variables.
+var gWindow = null;
+var gDocument = null;

Should use gDocument instead.

Comment 2

[Philip Chee]

Taking bug and morphing it slightly to cover other js errors.

Error: theDocument is not defined
Source file: chrome://navigator/content/pageinfo/pageInfo.js
Line: 648

Error: window.arguments is undefined
Source file: chrome://communicator/content/wallet/SignonViewer.js
Line: 69

Comment 3

[Philip Chee]

Attachment: Patch v1.0

This patch makes the two js errors that crop up in the pageInfo window go away.

Comment 4

[Daniel Brooks [:db48x]]

Comment on attachment 294480 [details] [diff] [review]
Patch v1.0

Thank you Philip, I really appreciate it

r=db48x

Comment 5

[Reed Loden [:reed]]

Checking in suite/browser/pageinfo/pageInfo.js;
/cvsroot/mozilla/suite/browser/pageinfo/pageInfo.js,v  <--  pageInfo.js
new revision: 1.6; previous revision: 1.5
done
Checking in suite/browser/pageinfo/security.js;
/cvsroot/mozilla/suite/browser/pageinfo/security.js,v  <--  security.js
new revision: 1.3; previous revision: 1.2
done