[interpreter] Prepare for shipping --harmony-instanceof.

test/cctest/interpreter/test-interpreter.cc

Index: test/cctest/interpreter/test-interpreter.cc
diff --git a/test/cctest/interpreter/test-interpreter.cc b/test/cctest/interpreter/test-interpreter.cc
index f477b37443e3b5e8448209d57861acd1ae3f1a54..67191caabafe758598abd458bfef1ee08a8049b4 100644
--- a/test/cctest/interpreter/test-interpreter.cc
+++ b/test/cctest/interpreter/test-interpreter.cc
@@ -1450,6 +1450,11 @@ TEST(InterpreterStrictNotEqual) {
 
 TEST(InterpreterInstanceOf) {
   HandleAndZoneScope handles;
+  // TODO(4447): The new ES6 'instanceof' operator is fully desugared in the
+  // parser and the Token::INSTANCEOF is not needed anymore. This test only
+  // makes sense with --no-harmony-instanceof and can be removed once we
+  // deprecate the ability to switch to old skool ES5 'instanceof' for good.
+  FLAG_harmony_instanceof = false;

[rmcilroy, 2016/03/21 11:53:26]

(fine to do this in a followup CL which removes the FLAG_harmony_instanceof)
could we rewrite this test as a JS source test to test the instanceof
implementation?

[Michael Starzinger, 2016/03/21 12:00:08]

Acknowledged. Yes, that would be another way to address this TODO. Either is
fine with me. I'll keep it in mind. Removing the flag is still some time away,
we first need to be confident that ES6 instanceof is shipping for good.

   i::Factory* factory = handles.main_isolate()->factory();
   Handle<i::String> name = factory->NewStringFromAsciiChecked("cons");
   Handle<i::JSFunction> func = factory->NewFunction(name);