Revert of Allow lexically declared "arguments" in function scope in sloppy mode.

test/cctest/test-parsing.cc

Index: test/cctest/test-parsing.cc
diff --git a/test/cctest/test-parsing.cc b/test/cctest/test-parsing.cc
index 383d372770b9314e491038544fdaa1b963bfb735..befa92a5a1d0e5d953e4d9f024eb8ef15f4627fd 100644
--- a/test/cctest/test-parsing.cc
+++ b/test/cctest/test-parsing.cc
@@ -6444,9 +6444,10 @@
   RunParserSyncTest(context_data, data, kSuccess);
 
   // v8:5201
+  // TODO(lpy): The two test sets below should be merged once
+  // we fix https://bugs.chromium.org/p/v8/issues/detail?id=4577
   {
-    // clang-format off
-    const char* sloppy_context_data[][2] = {
+    const char* sloppy_context_data1[][2] = {
       {"var ", " = {};"},
       {"function f(", ") {}"},
       {"function f(argument1, ", ") {}"},
@@ -6455,17 +6456,26 @@
       {"try {} catch(", ") {}"},
       {NULL, NULL}
     };
-    const char* data[] = {
-      "{arguments}",
+    const char* data1[] = {
       "{eval}",
-      "{x: arguments}",
       "{x: eval}",
-      "{arguments = false}",
       "{eval = false}",
       NULL
     };
-    // clang-format on
-    RunParserSyncTest(sloppy_context_data, data, kSuccess);
+    RunParserSyncTest(sloppy_context_data1, data1, kSuccess);
+
+    const char* sloppy_context_data2[][2] = {
+      {"var ", " = {};"},
+      {"try {} catch(", ") {}"},
+      {NULL, NULL}
+    };
+    const char* data2[] = {
+      "{arguments}",
+      "{x: arguments}",
+      "{arguments = false}",
+      NULL,
+    };
+    RunParserSyncTest(sloppy_context_data2, data2, kSuccess);
   }
 }
 
@@ -8320,38 +8330,3 @@
   RunParserSyncTest(context_data, data, kError, NULL, 0, always_flags,
                     arraysize(always_flags));
 }
-
-TEST(ArgumentsRedeclaration) {
-  {
-    // clang-format off
-    const char* context_data[][2] = {
-      { "function f(", ") {}" },
-      { NULL, NULL }
-    };
-    const char* success_data[] = {
-      "{arguments}",
-      "{arguments = false}",
-      "arg1, arguments",
-      "arg1, ...arguments",
-      NULL
-    };
-    // clang-format on
-    RunParserSyncTest(context_data, success_data, kSuccess);
-  }
-
-  {
-    // clang-format off
-    const char* context_data[][2] = {
-      { "function f() {", "}" },
-      { NULL, NULL }
-    };
-    const char* data[] = {
-      "const arguments = 1",
-      "let arguments",
-      "var arguments",
-      NULL
-    };
-    // clang-format on
-    RunParserSyncTest(context_data, data, kSuccess);
-  }
-}