297406 - javascript strict warning in aboutDialog.js

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[Florian Quèze [:florian]]

[Mozilla/5.0 (Windows; U; Windows NT 5.1; fr-FR; rv:1.7.8) Gecko/20050511
Firefox/1.0.4]

Avertissement : assignment to undeclared variable gSelectedPage
Fichier Source : chrome://browser/content/aboutDialog.js
Ligne : 11

Also reproduced in:
[Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b2) Gecko/20050610
Firefox/1.0+]

Comment 1

[Romain BEZUT]

Attachment: patch

Comment 2

[Romain BEZUT]

Comment on attachment 185935 [details] [diff] [review]
patch

Index: base/content/aboutDialog.js
===================================================================
RCS file: /cvsroot/mozilla/browser/base/content/aboutDialog.js,v
retrieving revision 1.5
diff -u -r1.5 aboutDialog.js
--- base/content/aboutDialog.js 30 Nov 2004 08:22:43 -0000	1.5
+++ base/content/aboutDialog.js 11 Jun 2005 12:03:39 -0000
@@ -34,6 +34,8 @@
 #
 # ***** END LICENSE BLOCK ***** -->

+var gSelectedPage = 0;
+
 function init(aEvent) 
 {
   if (aEvent.target != document)
@@ -43,7 +45,6 @@

   var button = document.documentElement.getButton("extra2");
   button.setAttribute("label",
document.documentElement.getAttribute("creditslabel"));
-  gSelectedPage = 0;
   button.addEventListener("command", switchPage, false);

   document.documentElement.getButton("accept").focus();

Comment 3

[Romain BEZUT]

Attachment: patch

This new patch should fix a problem about credits page which was not reachable
with the last one ...

Comment 4

[Daniel Brooks [:db48x]]

fix checked in