From 882117c5e38100aa0c4b19277479064c7a7789a2 Mon Sep 17 00:00:00 2001
From: Jeff Walden <jwalden@mit.edu>
Date: Sun, 23 Jan 2011 03:12:28 -0800
Subject: [PATCH] Back out everything from the last several hours for turning
 into a disaster zone finally ending in total orange.  r=return-to-sanity

---
 browser/base/content/syncSetup.js             |   4 +-
 browser/components/nsBrowserGlue.js           |   6 +-
 browser/fuel/src/fuelApplication.js           |   6 +-
 content/base/test/chrome/title_window.xul     |   4 +-
 docshell/test/chrome/bug113934_window.xul     |   4 +-
 docshell/test/chrome/bug215405_window.xul     |   4 +-
 docshell/test/chrome/bug449778_window.xul     |   4 +-
 docshell/test/chrome/bug449780_window.xul     |   4 +-
 docshell/test/chrome/docshell_helpers.js      |   4 +-
 .../tests/basic/strict-eval-loop-error.js     |   3 -
 js/src/jsapi.cpp                              |   3 +-
 js/src/jsemit.cpp                             |   2 +
 js/src/jsinterp.cpp                           |   2 +
 js/src/jskeyword.tbl                          | 102 ++++++++-------
 js/src/jsopcode.cpp                           |  18 +--
 js/src/jsopcode.h                             |   2 +-
 js/src/jsparse.cpp                            |   2 +
 js/src/jsscan.cpp                             |  10 +-
 js/src/jsscan.h                               |   1 -
 js/src/jstracer.cpp                           |  17 +--
 js/src/jsversion.h                            |   9 ++
 .../ecma_5/misc/future-reserved-words.js      | 122 ------------------
 js/src/tests/ecma_5/misc/jstests.list         |   1 -
 js/src/tests/js1_5/Regress/jstests.list       |   1 +
 js/src/tests/js1_5/Regress/regress-240317.js  |  68 ++++++++++
 .../tests/js1_5/extensions/regress-355736.js  |   8 +-
 .../chrome_content_integration_window.xul     |   4 +-
 .../chrome/chrome_over_plugin_window.xul      |   4 +-
 .../chrome/default_background_window.xul      |   4 +-
 layout/reftests/editor/xul/platform.js        |   6 +-
 layout/style/test/chrome/hover_helper.html    |   4 +-
 testing/mochitest/browser-harness.xul         |   4 +-
 .../ctypes/tests/unit/test_jsctypes.js.in     |   4 +-
 .../content/tests/chrome/bug263683_window.xul |   4 +-
 .../content/tests/chrome/bug331215_window.xul |   4 +-
 .../content/tests/chrome/bug451286_window.xul |   4 +-
 .../content/tests/chrome/bug451540_window.xul |   4 +-
 37 files changed, 204 insertions(+), 253 deletions(-)
 delete mode 100644 js/src/jit-test/tests/basic/strict-eval-loop-error.js
 delete mode 100644 js/src/tests/ecma_5/misc/future-reserved-words.js
 create mode 100644 js/src/tests/js1_5/Regress/regress-240317.js

diff --git a/browser/base/content/syncSetup.js b/browser/base/content/syncSetup.js
index 7d379445981..8fdf68d94f0 100644
--- a/browser/base/content/syncSetup.js
+++ b/browser/base/content/syncSetup.js
@@ -880,9 +880,9 @@ var gSyncSetup = {
   // if no property string is passed in, we clear label/style
   _setFeedback: function (element, success, string) {
     element.hidden = success || !string;
-    let classname = success ? "success" : "error";
+    let class = success ? "success" : "error";
     let image = element.getElementsByAttribute("class", "statusIcon")[0];
-    image.setAttribute("status", classname);
+    image.setAttribute("status", class);
     let label = element.getElementsByAttribute("class", "status")[0];
     label.value = string;
   },
diff --git a/browser/components/nsBrowserGlue.js b/browser/components/nsBrowserGlue.js
index 216dca3df00..38c6482c495 100644
--- a/browser/components/nsBrowserGlue.js
+++ b/browser/components/nsBrowserGlue.js
@@ -146,9 +146,9 @@ BrowserGlue.prototype = {
     // delays are in seconds
     const MAX_DELAY = 300;
     let delay = 3;
-    let browserEnum = Services.wm.getEnumerator("navigator:browser");
-    while (browserEnum.hasMoreElements()) {
-      delay += browserEnum.getNext().gBrowser.tabs.length;
+    let enum = Services.wm.getEnumerator("navigator:browser");
+    while (enum.hasMoreElements()) {
+      delay += enum.getNext().gBrowser.tabs.length;
     }
     delay = delay <= MAX_DELAY ? delay : MAX_DELAY;
 
diff --git a/browser/fuel/src/fuelApplication.js b/browser/fuel/src/fuelApplication.js
index 6310af5501b..d64035f908c 100644
--- a/browser/fuel/src/fuelApplication.js
+++ b/browser/fuel/src/fuelApplication.js
@@ -703,10 +703,10 @@ Application.prototype = {
 
   get windows() {
     var win = [];
-    var browserEnum = Utilities.windowMediator.getEnumerator("navigator:browser");
+    var enum = Utilities.windowMediator.getEnumerator("navigator:browser");
 
-    while (browserEnum.hasMoreElements())
-      win.push(new Window(browserEnum.getNext()));
+    while (enum.hasMoreElements())
+      win.push(new Window(enum.getNext()));
 
     return win;
   },
diff --git a/content/base/test/chrome/title_window.xul b/content/base/test/chrome/title_window.xul
index 88bae491baf..9134d65c184 100644
--- a/content/base/test/chrome/title_window.xul
+++ b/content/base/test/chrome/title_window.xul
@@ -21,8 +21,8 @@
   <script type="application/javascript">
     <![CDATA[
     var imports = [ "SimpleTest", "is", "isnot", "ok" ];
-    for each (var name in imports) {
-      window[name] = window.opener.wrappedJSObject[name];
+    for each (var import in imports) {
+      window[import] = window.opener.wrappedJSObject[import];
     }
 
     function testStatics() {
diff --git a/docshell/test/chrome/bug113934_window.xul b/docshell/test/chrome/bug113934_window.xul
index e9e3b245018..4f6e79688ae 100644
--- a/docshell/test/chrome/bug113934_window.xul
+++ b/docshell/test/chrome/bug113934_window.xul
@@ -15,8 +15,8 @@
   <script type="application/javascript"><![CDATA[
     var imports = [ "SimpleTest", "is", "isnot", "ok", "snapshotWindow",
                     "compareSnapshots", "onerror" ];
-    for each (var name in imports) {
-      window[name] = window.opener.wrappedJSObject[name];
+    for each (var import in imports) {
+      window[import] = window.opener.wrappedJSObject[import];
     }
 
     function $(id) {
diff --git a/docshell/test/chrome/bug215405_window.xul b/docshell/test/chrome/bug215405_window.xul
index c5c6c9b7c1c..6006af254c1 100644
--- a/docshell/test/chrome/bug215405_window.xul
+++ b/docshell/test/chrome/bug215405_window.xul
@@ -47,8 +47,8 @@
 
   <script type="application/javascript"><![CDATA[
     var imports = [ "SimpleTest", "is", "isnot", "ok"];
-    for each (var name in imports) {
-      window[name] = window.opener.wrappedJSObject[name];
+    for each (var import in imports) {
+      window[import] = window.opener.wrappedJSObject[import];
     }
 
     const text="MOZILLA";
diff --git a/docshell/test/chrome/bug449778_window.xul b/docshell/test/chrome/bug449778_window.xul
index dd311997ecd..3873f5a2587 100644
--- a/docshell/test/chrome/bug449778_window.xul
+++ b/docshell/test/chrome/bug449778_window.xul
@@ -9,8 +9,8 @@
   <!-- test code goes here -->
   <script type="application/javascript"><![CDATA[
     var imports = [ "SimpleTest", "is", "isnot", "ok", "onerror" ];
-    for each (var name in imports) {
-      window[name] = window.opener.wrappedJSObject[name];
+    for each (var import in imports) {
+      window[import] = window.opener.wrappedJSObject[import];
     }
 
     function $(id) {
diff --git a/docshell/test/chrome/bug449780_window.xul b/docshell/test/chrome/bug449780_window.xul
index 5733f93d61f..187fd34f33b 100644
--- a/docshell/test/chrome/bug449780_window.xul
+++ b/docshell/test/chrome/bug449780_window.xul
@@ -9,8 +9,8 @@
   <!-- test code goes here -->
   <script type="application/javascript"><![CDATA[
     var imports = [ "SimpleTest", "is", "isnot", "ok", "onerror" ];
-    for each (var name in imports) {
-      window[name] = window.opener.wrappedJSObject[name];
+    for each (var import in imports) {
+      window[import] = window.opener.wrappedJSObject[import];
     }
 
     function $(id) {
diff --git a/docshell/test/chrome/docshell_helpers.js b/docshell/test/chrome/docshell_helpers.js
index d55a2748560..237537908c6 100755
--- a/docshell/test/chrome/docshell_helpers.js
+++ b/docshell/test/chrome/docshell_helpers.js
@@ -3,8 +3,8 @@
  */
 var imports = [ "SimpleTest", "is", "isnot", "ok", "onerror", "todo", 
   "todo_is", "todo_isnot" ];
-for each (var name in imports) {
-  window[name] = window.opener.wrappedJSObject[name];
+for each (var import in imports) {
+  window[import] = window.opener.wrappedJSObject[import];
 }
 
 /**
diff --git a/js/src/jit-test/tests/basic/strict-eval-loop-error.js b/js/src/jit-test/tests/basic/strict-eval-loop-error.js
deleted file mode 100644
index f28467740ef..00000000000
--- a/js/src/jit-test/tests/basic/strict-eval-loop-error.js
+++ /dev/null
@@ -1,3 +0,0 @@
-var COUNT = RUNLOOP;
-eval("'use strict'; for (let j = 0; j < COUNT; j++); try { x; throw new Error(); } catch (e) { if (!(e instanceof ReferenceError)) throw e; }");
-assertEq(typeof j, "undefined");
diff --git a/js/src/jsapi.cpp b/js/src/jsapi.cpp
index 0b6ed529e27..cbb60b0cafc 100644
--- a/js/src/jsapi.cpp
+++ b/js/src/jsapi.cpp
@@ -4845,8 +4845,7 @@ JS_DecompileScript(JSContext *cx, JSScript *script, const char *name, uintN inde
         str = js_GetPrinterOutput(jp);
     else
         str = NULL;
-    if (!js_DestroyPrinter(jp))
-        return NULL;
+    js_DestroyPrinter(jp);
     return str;
 }
 
diff --git a/js/src/jsemit.cpp b/js/src/jsemit.cpp
index 05c60d8fb57..53405eed843 100644
--- a/js/src/jsemit.cpp
+++ b/js/src/jsemit.cpp
@@ -7090,10 +7090,12 @@ js_EmitTree(JSContext *cx, JSCodeGenerator *cg, JSParseNode *pn)
             return JS_FALSE;
         break;
 
+#if JS_HAS_DEBUGGER_KEYWORD
       case TOK_DEBUGGER:
         if (js_Emit1(cx, cg, JSOP_DEBUGGER) < 0)
             return JS_FALSE;
         break;
+#endif /* JS_HAS_DEBUGGER_KEYWORD */
 
 #if JS_HAS_XML_SUPPORT
       case TOK_XMLELEM:
diff --git a/js/src/jsinterp.cpp b/js/src/jsinterp.cpp
index a505c266e72..9c30c0d1268 100644
--- a/js/src/jsinterp.cpp
+++ b/js/src/jsinterp.cpp
@@ -6259,6 +6259,7 @@ BEGIN_CASE(JSOP_INSTANCEOF)
 }
 END_CASE(JSOP_INSTANCEOF)
 
+#if JS_HAS_DEBUGGER_KEYWORD
 BEGIN_CASE(JSOP_DEBUGGER)
 {
     JSDebuggerHandler handler = cx->debugHooks->debuggerHandler;
@@ -6282,6 +6283,7 @@ BEGIN_CASE(JSOP_DEBUGGER)
     }
 }
 END_CASE(JSOP_DEBUGGER)
+#endif /* JS_HAS_DEBUGGER_KEYWORD */
 
 #if JS_HAS_XML_SUPPORT
 BEGIN_CASE(JSOP_DEFXMLNS)
diff --git a/js/src/jskeyword.tbl b/js/src/jskeyword.tbl
index ca79ea1eeed..adcbf0e4074 100644
--- a/js/src/jskeyword.tbl
+++ b/js/src/jskeyword.tbl
@@ -38,75 +38,87 @@
  *
  * ***** END LICENSE BLOCK ***** */
 
-/*
- * Keywords used as primary expressions, sharing the TOK_PRIMARY token kind,
- * distinguished by opcode.
- */
-JS_KEYWORD(false,       TOK_PRIMARY,    JSOP_FALSE,     JSVERSION_DEFAULT)
-JS_KEYWORD(true,        TOK_PRIMARY,    JSOP_TRUE,      JSVERSION_DEFAULT)
-JS_KEYWORD(null,        TOK_PRIMARY,    JSOP_NULL,      JSVERSION_DEFAULT)
-
-/* ES5 Keywords. */
 JS_KEYWORD(break,       TOK_BREAK,      JSOP_NOP,       JSVERSION_DEFAULT)
 JS_KEYWORD(case,        TOK_CASE,       JSOP_NOP,       JSVERSION_DEFAULT)
-JS_KEYWORD(catch,       TOK_CATCH,      JSOP_NOP,       JSVERSION_DEFAULT)
 JS_KEYWORD(continue,    TOK_CONTINUE,   JSOP_NOP,       JSVERSION_DEFAULT)
-JS_KEYWORD(debugger,    TOK_DEBUGGER,   JSOP_NOP,       JSVERSION_DEFAULT)
 JS_KEYWORD(default,     TOK_DEFAULT,    JSOP_NOP,       JSVERSION_DEFAULT)
 JS_KEYWORD(delete,      TOK_DELETE,     JSOP_NOP,       JSVERSION_DEFAULT)
 JS_KEYWORD(do,          TOK_DO,         JSOP_NOP,       JSVERSION_DEFAULT)
 JS_KEYWORD(else,        TOK_ELSE,       JSOP_NOP,       JSVERSION_DEFAULT)
-JS_KEYWORD(finally,     TOK_FINALLY,    JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(export,      TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(false,       TOK_PRIMARY,    JSOP_FALSE,     JSVERSION_DEFAULT)
 JS_KEYWORD(for,         TOK_FOR,        JSOP_NOP,       JSVERSION_DEFAULT)
 JS_KEYWORD(function,    TOK_FUNCTION,   JSOP_NOP,       JSVERSION_DEFAULT)
 JS_KEYWORD(if,          TOK_IF,         JSOP_NOP,       JSVERSION_DEFAULT)
 JS_KEYWORD(in,          TOK_IN,         JSOP_IN,        JSVERSION_DEFAULT)
-JS_KEYWORD(instanceof,  TOK_INSTANCEOF, JSOP_INSTANCEOF,JSVERSION_DEFAULT)
 JS_KEYWORD(new,         TOK_NEW,        JSOP_NEW,       JSVERSION_DEFAULT)
+JS_KEYWORD(null,        TOK_PRIMARY,    JSOP_NULL,      JSVERSION_DEFAULT)
 JS_KEYWORD(return,      TOK_RETURN,     JSOP_NOP,       JSVERSION_DEFAULT)
 JS_KEYWORD(switch,      TOK_SWITCH,     JSOP_NOP,       JSVERSION_DEFAULT)
 JS_KEYWORD(this,        TOK_PRIMARY,    JSOP_THIS,      JSVERSION_DEFAULT)
-JS_KEYWORD(throw,       TOK_THROW,      JSOP_NOP,       JSVERSION_DEFAULT)
-JS_KEYWORD(try,         TOK_TRY,        JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(true,        TOK_PRIMARY,    JSOP_TRUE,      JSVERSION_DEFAULT)
 JS_KEYWORD(typeof,      TOK_UNARYOP,    JSOP_TYPEOF,    JSVERSION_DEFAULT)
 JS_KEYWORD(var,         TOK_VAR,        JSOP_DEFVAR,    JSVERSION_DEFAULT)
 JS_KEYWORD(void,        TOK_UNARYOP,    JSOP_VOID,      JSVERSION_DEFAULT)
 JS_KEYWORD(while,       TOK_WHILE,      JSOP_NOP,       JSVERSION_DEFAULT)
 JS_KEYWORD(with,        TOK_WITH,       JSOP_NOP,       JSVERSION_DEFAULT)
-
-/* ES5 FutureReservedWord keywords. */
-JS_KEYWORD(class,       TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
-JS_KEYWORD(enum,        TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
-JS_KEYWORD(export,      TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
-JS_KEYWORD(extends,     TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
-JS_KEYWORD(import,      TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
-JS_KEYWORD(super,       TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
-
-/*
- * ES5 reserved keywords with long-implemented behavior, allowed in our
- * implementation to ease code migration.
- */
 #if JS_HAS_CONST
 JS_KEYWORD(const,       TOK_VAR,        JSOP_DEFCONST,  JSVERSION_DEFAULT)
 #else
 JS_KEYWORD(const,       TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
 #endif
-#if JS_HAS_BLOCK_SCOPE
-JS_KEYWORD(let,         TOK_LET,             JSOP_NOP,  JSVERSION_1_7)
-#else
-JS_KEYWORD(let,         TOK_STRICT_RESERVED, JSOP_NOP,  JSVERSION_1_7)
-#endif
-#if JS_HAS_GENERATORS
-JS_KEYWORD(yield,       TOK_YIELD,           JSOP_NOP,  JSVERSION_1_7)
-#else
-JS_KEYWORD(yield,       TOK_STRICT_RESERVED, JSOP_NOP,  JSVERSION_1_7)
+
+JS_KEYWORD(try,         TOK_TRY,        JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(catch,       TOK_CATCH,      JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(finally,     TOK_FINALLY,    JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(throw,       TOK_THROW,      JSOP_NOP,       JSVERSION_DEFAULT)
+
+JS_KEYWORD(instanceof,  TOK_INSTANCEOF, JSOP_INSTANCEOF,JSVERSION_DEFAULT)
+
+#if JS_HAS_RESERVED_JAVA_KEYWORDS
+JS_KEYWORD(abstract,    TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(boolean,     TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(byte,        TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(char,        TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(class,       TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(double,      TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(extends,     TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(final,       TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(float,       TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(goto,        TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(implements,  TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(import,      TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(int,         TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(interface,   TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(long,        TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(native,      TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(package,     TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(private,     TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(protected,   TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(public,      TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(short,       TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(static,      TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(super,       TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(synchronized,TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(throws,      TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(transient,   TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+JS_KEYWORD(volatile,    TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
 #endif
 
-/* ES5 future reserved keywords in strict mode. */
-JS_KEYWORD(implements,  TOK_STRICT_RESERVED, JSOP_NOP,  JSVERSION_DEFAULT)
-JS_KEYWORD(interface,   TOK_STRICT_RESERVED, JSOP_NOP,  JSVERSION_DEFAULT)
-JS_KEYWORD(package,     TOK_STRICT_RESERVED, JSOP_NOP,  JSVERSION_DEFAULT)
-JS_KEYWORD(private,     TOK_STRICT_RESERVED, JSOP_NOP,  JSVERSION_DEFAULT)
-JS_KEYWORD(protected,   TOK_STRICT_RESERVED, JSOP_NOP,  JSVERSION_DEFAULT)
-JS_KEYWORD(public,      TOK_STRICT_RESERVED, JSOP_NOP,  JSVERSION_DEFAULT)
-JS_KEYWORD(static,      TOK_STRICT_RESERVED, JSOP_NOP,  JSVERSION_DEFAULT)
+#if JS_HAS_RESERVED_ECMA_KEYWORDS
+JS_KEYWORD(enum,        TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+#endif
+
+#if JS_HAS_DEBUGGER_KEYWORD
+JS_KEYWORD(debugger,    TOK_DEBUGGER,   JSOP_NOP,       JSVERSION_DEFAULT)
+#elif JS_HAS_RESERVED_ECMA_KEYWORDS
+JS_KEYWORD(debugger,    TOK_RESERVED,   JSOP_NOP,       JSVERSION_DEFAULT)
+#endif
+
+#if JS_HAS_GENERATORS
+JS_KEYWORD(yield,       TOK_YIELD,      JSOP_NOP,       JSVERSION_1_7)
+#endif
+
+#if JS_HAS_BLOCK_SCOPE
+JS_KEYWORD(let,         TOK_LET,        JSOP_NOP,       JSVERSION_1_7)
+#endif
diff --git a/js/src/jsopcode.cpp b/js/src/jsopcode.cpp
index bcf17be85a7..869463fc229 100644
--- a/js/src/jsopcode.cpp
+++ b/js/src/jsopcode.cpp
@@ -805,7 +805,6 @@ struct JSPrinter {
     bool            pretty;         /* pretty-print: indent, use newlines */
     bool            grouped;        /* in parenthesized expression context */
     bool            strict;         /* in code marked strict */
-    bool            error;          /* error occured */
     JSScript        *script;        /* script being printed */
     jsbytecode      *dvgfence;      /* DecompileExpression fencepost */
     jsbytecode      **pcstack;      /* DecompileExpression modeled stack */
@@ -828,7 +827,6 @@ js_NewPrinter(JSContext *cx, const char *name, JSFunction *fun,
     jp->pretty = !!pretty;
     jp->grouped = !!grouped;
     jp->strict = !!strict;
-    jp->error = false;
     jp->script = NULL;
     jp->dvgfence = NULL;
     jp->pcstack = NULL;
@@ -844,13 +842,11 @@ js_NewPrinter(JSContext *cx, const char *name, JSFunction *fun,
     return jp;
 }
 
-JSBool
+void
 js_DestroyPrinter(JSPrinter *jp)
 {
     JS_FinishArenaPool(&jp->pool);
-    bool error = jp->error;
     jp->sprinter.context->free(jp);
-    return error;
 }
 
 JSString *
@@ -2333,7 +2329,7 @@ Decompile(SprintStack *ss, jsbytecode *pc, intN nb, JSOp nextop)
                     ok = js_DecompileFunction(jp2);
                     if (ok && jp2->sprinter.base)
                         js_puts(jp, jp2->sprinter.base);
-                    ok &= !!js_DestroyPrinter(jp2);
+                    js_DestroyPrinter(jp2);
                     if (!ok)
                         return NULL;
                     js_puts(jp, "\n\n");
@@ -4617,10 +4613,12 @@ Decompile(SprintStack *ss, jsbytecode *pc, intN nb, JSOp nextop)
                 break;
 #endif /* JS_HAS_SHARP_VARS */
 
+#if JS_HAS_DEBUGGER_KEYWORD
               case JSOP_DEBUGGER:
                 js_printf(jp, "\tdebugger;\n");
                 todo = -2;
                 break;
+#endif /* JS_HAS_DEBUGGER_KEYWORD */
 
 #if JS_HAS_XML_SUPPORT
               case JSOP_STARTXML:
@@ -4925,8 +4923,7 @@ js_DecompileToString(JSContext *cx, const char *name, JSFunction *fun,
         str = js_GetPrinterOutput(jp);
     else
         str = NULL;
-    if (!js_DestroyPrinter(jp))
-        return NULL;
+    js_DestroyPrinter(jp);
     return str;
 }
 
@@ -5309,10 +5306,7 @@ DecompileExpression(JSContext *cx, JSScript *script, JSFunction *fun,
             name = (jp->sprinter.base) ? jp->sprinter.base : (char *) "";
             name = JS_strdup(cx, name);
         }
-        if (!js_DestroyPrinter(jp)) {
-            cx->free(name);
-            name = NULL;
-        }
+        js_DestroyPrinter(jp);
     }
 
 out:
diff --git a/js/src/jsopcode.h b/js/src/jsopcode.h
index 6add18ffd40..68cf5140d8b 100644
--- a/js/src/jsopcode.h
+++ b/js/src/jsopcode.h
@@ -311,7 +311,7 @@ extern JSPrinter *
 js_NewPrinter(JSContext *cx, const char *name, JSFunction *fun,
               uintN indent, JSBool pretty, JSBool grouped, JSBool strict);
 
-extern JSBool
+extern void
 js_DestroyPrinter(JSPrinter *jp);
 
 extern JSString *
diff --git a/js/src/jsparse.cpp b/js/src/jsparse.cpp
index c898462488a..04c2fda931d 100644
--- a/js/src/jsparse.cpp
+++ b/js/src/jsparse.cpp
@@ -6203,6 +6203,7 @@ Parser::statement()
         pn->pn_type = TOK_SEMI;
         return pn;
 
+#if JS_HAS_DEBUGGER_KEYWORD
       case TOK_DEBUGGER:
         pn = NullaryNode::create(tc);
         if (!pn)
@@ -6210,6 +6211,7 @@ Parser::statement()
         pn->pn_type = TOK_DEBUGGER;
         tc->flags |= TCF_FUN_HEAVYWEIGHT;
         break;
+#endif /* JS_HAS_DEBUGGER_KEYWORD */
 
 #if JS_HAS_XML_SUPPORT
       case TOK_DEFAULT:
diff --git a/js/src/jsscan.cpp b/js/src/jsscan.cpp
index 63a4413f1e8..9e6b5dd7bf0 100644
--- a/js/src/jsscan.cpp
+++ b/js/src/jsscan.cpp
@@ -1043,18 +1043,10 @@ TokenStream::getTokenInternal()
             !(flags & TSF_KEYWORD_IS_NAME) &&
             (kw = FindKeyword(tokenbuf.begin(), tokenbuf.length()))) {
             if (kw->tokentype == TOK_RESERVED) {
-                if (!ReportCompileErrorNumber(cx, this, NULL, JSREPORT_ERROR,
+                if (!ReportCompileErrorNumber(cx, this, NULL, JSREPORT_WARNING | JSREPORT_STRICT,
                                               JSMSG_RESERVED_ID, kw->chars)) {
                     goto error;
                 }
-            } else if (kw->tokentype == TOK_STRICT_RESERVED) {
-                if (isStrictMode()
-                    ? !ReportStrictModeError(cx, this, NULL, NULL, JSMSG_RESERVED_ID, kw->chars)
-                    : !ReportCompileErrorNumber(cx, this, NULL,
-                                                JSREPORT_STRICT | JSREPORT_WARNING,
-                                                JSMSG_RESERVED_ID, kw->chars)) {
-                    goto error;
-                }
             } else if (kw->version <= VersionNumber(version)) {
                 tt = kw->tokentype;
                 tp->t_op = (JSOp) kw->op;
diff --git a/js/src/jsscan.h b/js/src/jsscan.h
index ee102cf1e22..ee33c5620d7 100644
--- a/js/src/jsscan.h
+++ b/js/src/jsscan.h
@@ -144,7 +144,6 @@ enum TokenKind {
                                            of definitions paired with a parse
                                            tree full of uses of those names */
     TOK_RESERVED,                       /* reserved keywords */
-    TOK_STRICT_RESERVED,                /* reserved keywords in strict mode */
     TOK_LIMIT                           /* domain size */
 };
 
diff --git a/js/src/jstracer.cpp b/js/src/jstracer.cpp
index d42fb932cfb..7802dbd9695 100644
--- a/js/src/jstracer.cpp
+++ b/js/src/jstracer.cpp
@@ -14978,13 +14978,6 @@ TraceRecorder::record_JSOP_POPN()
     return ARECORD_CONTINUE;
 }
 
-static inline bool
-IsFindableCallObj(JSObject *obj)
-{
-    return obj->isCall() &&
-           (obj->callIsForEval() || obj->getCallObjCalleeFunction()->isHeavyweight());
-}
-
 /*
  * Generate LIR to reach |obj2| from |obj| by traversing the scope chain. The
  * generated code also ensures that any call objects found have not changed shape.
@@ -15026,10 +15019,13 @@ TraceRecorder::traverseScopeChain(JSObject *obj, LIns *obj_ins, JSObject *target
 
     for (;;) {
         if (searchObj != globalObj) {
-            if (searchObj->isBlock())
+            Class* clasp = searchObj->getClass();
+            if (clasp == &js_BlockClass) {
                 foundBlockObj = true;
-            else if (IsFindableCallObj(searchObj))
+            } else if (clasp == &js_CallClass &&
+                       searchObj->getCallObjCalleeFunction()->isHeavyweight()) {
                 foundCallObj = true;
+            }
         }
 
         if (searchObj == targetObj)
@@ -15058,7 +15054,8 @@ TraceRecorder::traverseScopeChain(JSObject *obj, LIns *obj_ins, JSObject *target
             // We must guard on the shape of all call objects for heavyweight functions
             // that we traverse on the scope chain: if the shape changes, a variable with
             // the same name may have been inserted in the scope chain.
-            if (IsFindableCallObj(obj)) {
+            if (obj->isCall() &&
+                obj->getCallObjCalleeFunction()->isHeavyweight()) {
                 if (!exit)
                     exit = snapshot(BRANCH_EXIT);
                 guard(true,
diff --git a/js/src/jsversion.h b/js/src/jsversion.h
index f72af4981b0..73969bd3b12 100644
--- a/js/src/jsversion.h
+++ b/js/src/jsversion.h
@@ -89,6 +89,7 @@
 #define JS_HAS_SHARP_VARS       0       /* has #n=, #n# for object literals */
 #define JS_HAS_XDR              0       /* has XDR API and internal support */
 #define JS_HAS_TOSOURCE         0       /* has Object/Array toSource method */
+#define JS_HAS_DEBUGGER_KEYWORD 0       /* has hook for debugger keyword */
 #define JS_HAS_CATCH_GUARD      0       /* has exception handling catch guard */
 #define JS_HAS_SPARSE_ARRAYS    0       /* array methods preserve empty elems */
 #define JS_HAS_UNEVAL           0       /* has uneval() top-level function */
@@ -116,6 +117,7 @@
 #define JS_HAS_SHARP_VARS       1       /* has #n=, #n# for object literals */
 #define JS_HAS_XDR              1       /* has XDR API and internal support */
 #define JS_HAS_TOSOURCE         1       /* has Object/Array toSource method */
+#define JS_HAS_DEBUGGER_KEYWORD 1       /* has hook for debugger keyword */
 #define JS_HAS_CATCH_GUARD      1       /* has exception handling catch guard */
 #define JS_HAS_SPARSE_ARRAYS    0       /* array methods preserve empty elems */
 #define JS_HAS_UNEVAL           1       /* has uneval() top-level function */
@@ -139,6 +141,7 @@
 #define JS_HAS_SHARP_VARS       1       /* has #n=, #n# for object literals */
 #define JS_HAS_XDR              1       /* has XDR API and internal support */
 #define JS_HAS_TOSOURCE         1       /* has Object/Array toSource method */
+#define JS_HAS_DEBUGGER_KEYWORD 1       /* has hook for debugger keyword */
 #define JS_HAS_CATCH_GUARD      1       /* has exception handling catch guard */
 #define JS_HAS_SPARSE_ARRAYS    0       /* array methods preserve empty elems */
 #define JS_HAS_UNEVAL           1       /* has uneval() top-level function */
@@ -162,6 +165,7 @@
 #define JS_HAS_SHARP_VARS       1       /* has #n=, #n# for object literals */
 #define JS_HAS_XDR              1       /* has XDR API and internal support */
 #define JS_HAS_TOSOURCE         1       /* has Object/Array toSource method */
+#define JS_HAS_DEBUGGER_KEYWORD 1       /* has hook for debugger keyword */
 #define JS_HAS_CATCH_GUARD      1       /* has exception handling catch guard */
 #define JS_HAS_SPARSE_ARRAYS    0       /* array methods preserve empty elems */
 #define JS_HAS_UNEVAL           1       /* has uneval() top-level function */
@@ -185,6 +189,7 @@
 #define JS_HAS_SHARP_VARS       1       /* has #n=, #n# for object literals */
 #define JS_HAS_XDR              1       /* has XDR API and internal support */
 #define JS_HAS_TOSOURCE         1       /* has Object/Array toSource method */
+#define JS_HAS_DEBUGGER_KEYWORD 1       /* has hook for debugger keyword */
 #define JS_HAS_CATCH_GUARD      1       /* has exception handling catch guard */
 #define JS_HAS_SPARSE_ARRAYS    0       /* array methods preserve empty elems */
 #define JS_HAS_UNEVAL           1       /* has uneval() top-level function */
@@ -205,6 +210,10 @@
 
 #endif
 
+/* Features that are present in all versions. */
+#define JS_HAS_RESERVED_JAVA_KEYWORDS   1
+#define JS_HAS_RESERVED_ECMA_KEYWORDS   1
+
 /* Support for JS_NewGlobalObject. */
 #define JS_HAS_NEW_GLOBAL_OBJECT        1
 
diff --git a/js/src/tests/ecma_5/misc/future-reserved-words.js b/js/src/tests/ecma_5/misc/future-reserved-words.js
deleted file mode 100644
index 0a394892dbc..00000000000
--- a/js/src/tests/ecma_5/misc/future-reserved-words.js
+++ /dev/null
@@ -1,122 +0,0 @@
-/*
- * Any copyright is dedicated to the Public Domain.
- * http://creativecommons.org/licenses/publicdomain/
- */
-
-//-----------------------------------------------------------------------------
-var BUGNUMBER = 497869;
-var summary = "Implement FutureReservedWords per-spec";
-
-print(BUGNUMBER + ": " + summary);
-
-/**************
- * BEGIN TEST *
- **************/
-
-var futureReservedWords =
-  ["class",
-   // "const", // Mozilla extension enabled even for versionless code
-   "enum",
-   "export",
-   "extends",
-   "import",
-   "super"];
-
-var strictFutureReservedWords =
-  [
-   "implements",
-   "interface",
-   // "let", // Mozilla extension, strict checks disabled for now
-   "package",
-   "private",
-   "protected",
-   "public",
-   "static",
-   // "yield", // Mozilla extension, strict checks disabled for now
-  ];
-
-function testWord(word, expectNormal, expectStrict)
-{
-  var actual, status;
-
-  // USE IN VARIABLE DECLARATION
-
-  actual = "";
-  status = summary + ", normal var: " + word;
-  try
-  {
-    eval("var " + word + ";");
-    actual = "no error";
-  }
-  catch (e)
-  {
-    actual = "error";
-    status +=  ", " + e.name + ": " + e.message + " ";
-  }
-  reportCompare(expectNormal, actual, status);
-
-  actual = "";
-  status = summary + ", strict var: " + word;
-  try
-  {
-    eval("'use strict'; var " + word + ";");
-    actual = "no error";
-  }
-  catch (e)
-  {
-    actual = "error";
-    status +=  ", " + e.name + ": " + e.message + " ";
-  }
-  reportCompare(expectStrict, actual, status);
-
-
-  // USE AS LHS FOR ASSIGNMENT
-
-  actual = "";
-  status = summary + ", normal assignment: " + word;
-  try
-  {
-    eval(word + " = 'foo';");
-    actual = "no error";
-  }
-  catch(e)
-  {
-    actual = "error";
-    status +=  ", " + e.name + ": " + e.message + " ";
-  }
-  reportCompare(expectNormal, actual, status);
-
-  actual = "";
-  status = summary + ", strict assignment: " + word;
-  try
-  {
-    eval("'use strict'; " + word + " = 'foo';");
-    actual = "no error";
-  }
-  catch(e)
-  {
-    actual = "error";
-    status +=  ", " + e.name + ": " + e.message + " ";
-  }
-  reportCompare(expectStrict, actual, status);
-}
-
-function testFutureReservedWord(word)
-{
-  testWord(word, "error", "error");
-}
-
-function testStrictFutureReservedWord(word)
-{
-  testWord(word, "no error", "error");
-}
-
-futureReservedWords.forEach(testFutureReservedWord);
-strictFutureReservedWords.forEach(testStrictFutureReservedWord);
-
-/******************************************************************************/
-
-if (typeof reportCompare === "function")
-  reportCompare(true, true);
-
-print("All tests passed!");
diff --git a/js/src/tests/ecma_5/misc/jstests.list b/js/src/tests/ecma_5/misc/jstests.list
index 4f3fa911cfd..eb251da31ef 100644
--- a/js/src/tests/ecma_5/misc/jstests.list
+++ b/js/src/tests/ecma_5/misc/jstests.list
@@ -6,4 +6,3 @@ script unwrapped-no-such-method.js
 script explicit-undefined-optional-argument.js
 script function-definition-eval.js
 skip-if(!xulRuntime.shell) script function-definition-evaluate.js # needs evaluate()
-script future-reserved-words.js
diff --git a/js/src/tests/js1_5/Regress/jstests.list b/js/src/tests/js1_5/Regress/jstests.list
index 87c0c5e8290..bada3aecaa4 100644
--- a/js/src/tests/js1_5/Regress/jstests.list
+++ b/js/src/tests/js1_5/Regress/jstests.list
@@ -60,6 +60,7 @@ script regress-233483.js
 script regress-234389.js
 script regress-238881.js
 script regress-238945.js
+script regress-240317.js
 script regress-240577.js
 script regress-243174.js
 script regress-243389-n.js
diff --git a/js/src/tests/js1_5/Regress/regress-240317.js b/js/src/tests/js1_5/Regress/regress-240317.js
new file mode 100644
index 00000000000..c501342bf4b
--- /dev/null
+++ b/js/src/tests/js1_5/Regress/regress-240317.js
@@ -0,0 +1,68 @@
+/* -*- Mode: C++; tab-width: 2; indent-tabs-mode: nil; c-basic-offset: 2 -*- */
+/*
+ * Any copyright is dedicated to the Public Domain.
+ * http://creativecommons.org/licenses/publicdomain/
+ * Contributor: Bob Clary
+ */
+
+//-----------------------------------------------------------------------------
+var BUGNUMBER = 240317;
+var summary = 'Using Reserved identifiers warns';
+var actual = '';
+var expect = 'no error';
+
+printBugNumber(BUGNUMBER);
+printStatus (summary);
+
+function testvar(words)
+{
+  var e;
+  expect = 'no error';
+  for (var i = 0; i < words.length; i++)
+  {
+    var word = words[i];
+
+    actual = '';
+    status = summary + ': ' + word;
+    try
+    {
+      eval('var ' + word + ';');
+      actual = 'no error';
+    }
+    catch(e)
+    {
+      actual = 'error';
+      status +=  ', ' + e.name + ': ' + e.message + ' ';
+    }
+    reportCompare(expect, actual, status);
+
+    actual = '';
+    status = summary + ': ' + word;
+    try
+    {
+      eval(word + ' = "foo";');
+      actual = 'no error';
+    }
+    catch(e)
+    {
+      actual = 'error';
+      status +=  ', ' + e.name + ': ' + e.message + ' ';
+    }
+    reportCompare(expect, actual, status);
+
+  }
+}
+
+// future reserved words
+var reserved =
+  ['abstract',    'enum',      'int',      'short',      'boolean',
+   'interface', 'static',   'byte',       'extends',
+   'long',         'super',     'char',     'final',      'native',
+   'synchronized', 'class',     'float',    'package',    'throws',
+   'goto',      'private',  'transient',             
+   'implements',   'protected', 'volatile', 'double',               
+   'public'];
+
+testvar(reserved);
+ 
+
diff --git a/js/src/tests/js1_5/extensions/regress-355736.js b/js/src/tests/js1_5/extensions/regress-355736.js
index 61db65bd826..ef9104fdaf8 100644
--- a/js/src/tests/js1_5/extensions/regress-355736.js
+++ b/js/src/tests/js1_5/extensions/regress-355736.js
@@ -52,13 +52,13 @@ function test()
   printBugNumber(BUGNUMBER);
   printStatus (summary);
  
-  f = function() { [implements] = q; };
-  expect = 'function() { [implements] = q; }';
+  f = function() { [super] = q; };
+  expect = 'function() { [super] = q; }';
   actual = f + '';
   compareSource(expect, actual, summary + ': 1');
 
-  f = function() { return { get implements() { } } };
-  expect = 'function() { return { get implements() { } }; }';
+  f = function() { return { get super() { } } };
+  expect = 'function() { return { get super() { } }; }';
   actual = f + '';
   compareSource(expect, actual, summary + ': 2');
 
diff --git a/layout/base/tests/chrome/chrome_content_integration_window.xul b/layout/base/tests/chrome/chrome_content_integration_window.xul
index 2ba9e902424..f85e9ef6040 100644
--- a/layout/base/tests/chrome/chrome_content_integration_window.xul
+++ b/layout/base/tests/chrome/chrome_content_integration_window.xul
@@ -21,8 +21,8 @@
   <script type="application/javascript">
     <![CDATA[
     var imports = [ "SimpleTest", "is", "isnot", "ok" ];
-    for each (var name in imports) {
-      window[name] = window.opener.wrappedJSObject[name];
+    for each (var import in imports) {
+      window[import] = window.opener.wrappedJSObject[import];
     }
     
     function runTests() {
diff --git a/layout/base/tests/chrome/chrome_over_plugin_window.xul b/layout/base/tests/chrome/chrome_over_plugin_window.xul
index 4e167ed10e1..b9792447c8b 100644
--- a/layout/base/tests/chrome/chrome_over_plugin_window.xul
+++ b/layout/base/tests/chrome/chrome_over_plugin_window.xul
@@ -22,8 +22,8 @@
   <script type="application/javascript">
     <![CDATA[
     var imports = [ "SimpleTest", "is", "isnot", "ok", "todo" ];
-    for each (var name in imports) {
-      window[name] = window.opener.wrappedJSObject[name];
+    for each (var import in imports) {
+      window[import] = window.opener.wrappedJSObject[import];
     }
 
     var plugin;
diff --git a/layout/base/tests/chrome/default_background_window.xul b/layout/base/tests/chrome/default_background_window.xul
index 8e6b5f91da6..a87d675cb51 100644
--- a/layout/base/tests/chrome/default_background_window.xul
+++ b/layout/base/tests/chrome/default_background_window.xul
@@ -18,8 +18,8 @@
     SimpleTest.waitForExplicitFinish();
 
     var imports = [ "SimpleTest", "is", "isnot", "ok" ];
-    for each (var name in imports) {
-      window[name] = window.opener.wrappedJSObject[name];
+    for each (var import in imports) {
+      window[import] = window.opener.wrappedJSObject[import];
     }
     
     function snapshot(win) {
diff --git a/layout/reftests/editor/xul/platform.js b/layout/reftests/editor/xul/platform.js
index f45e6d1f5c8..40adcf91324 100644
--- a/layout/reftests/editor/xul/platform.js
+++ b/layout/reftests/editor/xul/platform.js
@@ -9,7 +9,7 @@ var ua = navigator.userAgent;
 if (/Windows/.test(ua)) {
   id = "win";
   if (/NT 5\.1/.test(ua) || /NT 5\.2; Win64/.test(ua))
-    var classname = "winxp";
+    var class = "winxp";
 }
 else if (/Linux/.test(ua))
   id = "linux";
@@ -24,5 +24,5 @@ else
   document.documentElement.appendChild(
     document.createTextNode("Unrecognized platform")
   );
-if (classname)
-  document.documentElement.setAttribute("class", classname);
+if (class)
+  document.documentElement.setAttribute("class", class);
diff --git a/layout/style/test/chrome/hover_helper.html b/layout/style/test/chrome/hover_helper.html
index 9f9680597ce..ca063dcb22f 100644
--- a/layout/style/test/chrome/hover_helper.html
+++ b/layout/style/test/chrome/hover_helper.html
@@ -36,8 +36,8 @@
 <script type="application/javascript">
 
 var imports = [ "SimpleTest", "is", "isnot", "ok" ];
-for each (var name in imports) {
-    window[name] = window.opener.wrappedJSObject[name];
+for each (var import in imports) {
+    window[import] = window.opener.wrappedJSObject[import];
 }
 
 var div = document.getElementById("display");
diff --git a/testing/mochitest/browser-harness.xul b/testing/mochitest/browser-harness.xul
index a834ed8087c..bff8db237c9 100644
--- a/testing/mochitest/browser-harness.xul
+++ b/testing/mochitest/browser-harness.xul
@@ -204,8 +204,8 @@
         }
         var path = _entityEncode(this.path);
         var html = this.results.map(function (t) {
-          var classname = t.info ? "info" : "result " + (t.pass ? "passed" : "failed");
-          return "<p class=\"" + classname + "\">" + t.result + " | " + path +
+          var class = t.info ? "info" : "result " + (t.pass ? "passed" : "failed");
+          return "<p class=\"" + class + "\">" + t.result + " | " + path +
                  " | " + _entityEncode(t.msg) + "</p>";
         }).join("\n");
         if (this.duration) {
diff --git a/toolkit/components/ctypes/tests/unit/test_jsctypes.js.in b/toolkit/components/ctypes/tests/unit/test_jsctypes.js.in
index 5f0089bf0e8..0857cf6c969 100644
--- a/toolkit/components/ctypes/tests/unit/test_jsctypes.js.in
+++ b/toolkit/components/ctypes/tests/unit/test_jsctypes.js.in
@@ -82,12 +82,12 @@ function do_check_throws(f, type, stack)
   do_throw("expected " + type.name + " exception, none thrown", stack);
 }
 
-function do_check_class(obj, classname, stack)
+function do_check_class(obj, class, stack)
 {
   if (!stack)
     stack = Components.stack.caller;
 
-  do_check_eq(Object.prototype.toString.call(obj), "[object " + classname + "]", stack);
+  do_check_eq(Object.prototype.toString.call(obj), "[object " + class + "]", stack);
 }
 
 function run_test()
diff --git a/toolkit/content/tests/chrome/bug263683_window.xul b/toolkit/content/tests/chrome/bug263683_window.xul
index 2cb37a065e3..44b380258b9 100644
--- a/toolkit/content/tests/chrome/bug263683_window.xul
+++ b/toolkit/content/tests/chrome/bug263683_window.xul
@@ -55,8 +55,8 @@
     var gBrowser;
 
     var imports = ["SimpleTest", "ok"];
-    for each (var name in imports) {
-      window[name] = window.opener.wrappedJSObject[name];
+    for each (var import in imports) {
+      window[import] = window.opener.wrappedJSObject[import];
     }
 
     function finish() {
diff --git a/toolkit/content/tests/chrome/bug331215_window.xul b/toolkit/content/tests/chrome/bug331215_window.xul
index 914c73ec1b3..ea755c2aaca 100644
--- a/toolkit/content/tests/chrome/bug331215_window.xul
+++ b/toolkit/content/tests/chrome/bug331215_window.xul
@@ -55,8 +55,8 @@
     var gBrowser;
 
     var imports = ["SimpleTest", "ok"];
-    for each (var name in imports) {
-      window[name] = window.opener.wrappedJSObject[name];
+    for each (var import in imports) {
+      window[import] = window.opener.wrappedJSObject[import];
     }
 
     function finish() {
diff --git a/toolkit/content/tests/chrome/bug451286_window.xul b/toolkit/content/tests/chrome/bug451286_window.xul
index 535aeb2ff96..505b1fa0f28 100644
--- a/toolkit/content/tests/chrome/bug451286_window.xul
+++ b/toolkit/content/tests/chrome/bug451286_window.xul
@@ -57,8 +57,8 @@
     var gBrowser;
 
     var imports = ["SimpleTest", "ok", "snapshotWindow", "compareSnapshots"];
-    for each (var name in imports) {
-      window[name] = window.opener.wrappedJSObject[name];
+    for each (var import in imports) {
+      window[import] = window.opener.wrappedJSObject[import];
     }
 
     function finish() {
diff --git a/toolkit/content/tests/chrome/bug451540_window.xul b/toolkit/content/tests/chrome/bug451540_window.xul
index 3254db7967b..6a0c127d036 100644
--- a/toolkit/content/tests/chrome/bug451540_window.xul
+++ b/toolkit/content/tests/chrome/bug451540_window.xul
@@ -62,8 +62,8 @@
     }
 
     var imports = [ "SimpleTest", "ok"];
-      for each (var name in imports) {
-        window[name] = window.opener.wrappedJSObject[name];
+      for each (var import in imports) {
+        window[import] = window.opener.wrappedJSObject[import];
       }