Bug 1334356 - Don't attach a missing-property stub if extra warnings are enabled. r=h4writer

js/src/jit-test/tests/ion/bug1334356.js

@@ -0,0 +1,16 @@
+setJitCompilerOption('ion.forceinlineCaches', 1);
+options('strict');
+options('werror');
+
+var o = {y: 0};
+Object.defineProperty(o, "x", {get: function() { o.y } });
+
+for (var i = 0; i < 10; i++) {
+    var fail = false;
+    try {
+        o.x.u;
+        fail = true;
+    } catch (e) {}
+    assertEq(fail, false);
+    delete o.y;
+}

js/src/jit/CacheIR.cpp

@@ -244,8 +244,9 @@ IsCacheableNoProperty(JSContext* cx, JSObject* obj, JSObject* holder, Shape* sha
         return false;
     }
 
-    // If we're doing a name lookup, we have to throw a ReferenceError.
-    if (*pc == JSOP_GETXPROP)
+    // If we're doing a name lookup, we have to throw a ReferenceError. If
+    // extra warnings are enabled, we may have to report a warning.
+    if (*pc == JSOP_GETXPROP || cx->compartment()->behaviors().extraWarnings(cx))
         return false;
 
     return CheckHasNoSuchProperty(cx, obj, id);