Strict mode assignment to undefined reference.

test/mjsunit/strict-mode.js

 // Copyright 2011 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 317 matching lines @@
   eval("var x = { get " + word + " () { 'use strict'; } };");
   eval("var x = { set " + word + " (value) { 'use strict'; } };");
   assertThrows("var x = { get foo(" + word + ") { 'use strict'; } };", SyntaxError);
   assertThrows("var x = { set foo(" + word + ") { 'use strict'; } };", SyntaxError);
 }
 
 for (var i = 0; i < future_reserved_words.length; i++) {
   testFutureReservedWord(future_reserved_words[i]);
 }
 
+function testAssignToUndefined(should_throw) {
+  "use strict";
+  try {
+    possibly_undefined_variable_for_strict_mode_test = "should throw?";
+  } catch (e) {
+    assertTrue(should_throw, "strict mode");
+    assertInstanceof(e, ReferenceError, "strict mode");
+    return;
+  }
+  assertFalse(should_throw, "strict mode");
+}
 
+testAssignToUndefined(true);
+testAssignToUndefined(true);
+testAssignToUndefined(true);

[MarkM, 2011/02/11 03:06:59]

Why are you calling this three times for each case?

[Mads Ager (chromium), 2011/02/11 08:08:09]

In order to make sure that ICs are initialized and used, the first time through
the code the ICs are not yet initialized and everything is taken care of in the
runtime system.

Martin, can you put this in a loop and do it 10 times for each? That way we have
a bit more slack if we change the heuristics for when to generate new ICs and
patch the code.

[Martin Maly, 2011/02/11 21:43:28]

Done.

+
+possibly_undefined_variable_for_strict_mode_test = "value";
+
+testAssignToUndefined(false);
+testAssignToUndefined(false);
+testAssignToUndefined(false);
+
+delete possibly_undefined_variable_for_strict_mode_test;
+
+testAssignToUndefined(true);
+testAssignToUndefined(true);
+testAssignToUndefined(true);