[parser] parenthesized Literals are not valid AssignmentPatterns

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 7024 matching lines @@
     "{ x: (foo()) }",
     "{ x: (function() {}) }",
     "{ x: y } = 'str'",
     "[x, y] = 'str'",
     "[(x,y) => z]",
     "{x: (y) => z}",
     "[x, ...y, z]",
     "[...x,]",
     "[x, y, ...z = 1]",
     "[...z = 1]",
+
+    // v8:4657

[adamk, 2016/01/08 16:24:23]

Please add a few tests with multiple levels of parens?

[caitp (gmail), 2016/01/08 16:36:01]

Done

+    "({ x: x4, x: (x+=1e4) })",
+    "({ x: x4, x: (x+=1e4) } = {})",
+    "({ x: y } = {})",
+    "([a])",
+    "([a] = [])",
     NULL};
   // clang-format on
   static const ParserFlag always_flags[] = {
       kAllowHarmonyDestructuringAssignment, kAllowHarmonyDestructuring,
       kAllowHarmonyDefaultParameters};
   RunParserSyncTest(context_data, data, kError, NULL, 0, always_flags,
                     arraysize(always_flags));
 
   const char* empty_context_data[][2] = {
       {"'use strict';", ""}, {"", ""}, {NULL, NULL}};
@@ skipping 717 matching lines @@
 }
 
 
 TEST(MiscSyntaxErrors) {
   const char* context_data[][2] = {
       {"'use strict'", ""}, {"", ""}, {NULL, NULL}};
   const char* error_data[] = {"for (();;) {}", NULL};
 
   RunParserSyncTest(context_data, error_data, kError, NULL, 0, NULL, 0);
 }