[parser] unify metaproperty parsing and require unescaped property name

test/cctest/test-parsing.cc

 // Copyright 2012 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 7989 matching lines @@
     "for (();;) {}",
 
     // crbug.com/582626
     "{ NaN ,chA((evarA=new t ( l = !.0[((... co -a0([1]))=> greturnkf",
     NULL
   };
   // clang-format on
 
   RunParserSyncTest(context_data, error_data, kError, NULL, 0, NULL, 0);
 }
+
+TEST(FunctionSentErrors) {
+  // clang-format off
+  const char* context_data[][2] = {
+    { "'use strict'", "" },
+    { "", "" },
+    { NULL, NULL }
+  };
+  const char* error_data[] = {
+    "var x = function.sent",
+    "function* g() { yield function.s\\u0065nt; }"
+  };

[adamk, 2016/02/16 22:21:55]

Think you'll need a NULL here.

+  // clang-format on
+
+  bool old_flag = i::FLAG_harmony_function_sent;
+  i::FLAG_harmony_function_sent = true;

[adamk, 2016/02/16 22:21:55]

Hmm, I suspect this is supposed to have an allow_* flag for the parser to avoid
threading issues...not something to fix in this patch, obviously.

[caitp (gmail), 2016/02/16 22:23:48]

yeah, I'll add the parser flag accessors in a separate cleanup. Good point about
the missing NULL (I would have built this locally, but just jumped from a
totally different branch, and it takes foreverrrr I really need a better local
build infrastructure :L)

+  RunParserSyncTest(context_data, error_data, kError, NULL, 0, NULL, 0);
+  i::FLAG_harmony_function_sent = old;
+}
+
+TEST(NewTargetErrors) {
+  // clang-format off
+  const char* context_data[][2] = {
+    { "'use strict'", "" },
+    { "", "" },
+    { NULL, NULL }
+  };
+  const char* error_data[] = {
+    "var x = new.target",
+    "function f() { return new.t\\u0061rget; }"
+  };

[adamk, 2016/02/16 22:21:55]

And a NULL here too

+  // clang-format on
+  RunParserSyncTest(context_data, error_data, kError, NULL, 0, NULL, 0);

[adamk, 2016/02/16 22:21:55]

You can leave off the last four arguments in a case like this.

+}