Rewrite scopes in computed properties in destructured parameters

Rewrite scopes in computed properties in destructured parameters

While we properly handled scopes of initializers in destructured parameters,
we never did the right thing for computed properties. This patch fixes that
by factoring out PatternRewriter's scope rewriting logic and calls it
for the computed property case.

BUG=chromium:620119
Committed: https://crrev.com/f795a790100832ff8eb7aba29676efe6beff8ea8
Cr-Commit-Position: refs/heads/master@{#37228}

[adamk, 2016-06-23 20:10:57]

Description was changed from

==========
Work in progress: rewrite scopes in computed properties in destructuring

BUG=chromium:620119
==========

to

==========
Rewrite scopes in computed properties in destructured parameters

While we properly handled scopes of initializers in destructured parameters,
we never did the right thing for computed properties. This patch fixes that
by factoring out PatternRewriter's scope rewriting logic and calls it
for the computed property case.

BUG=chromium:620119
==========

[adamk, 2016-06-23 20:11:06]

adamk@chromium.org changed reviewers:
+ littledan@chromium.org

[Dan Ehrenberg, 2016-06-23 20:43:12]

lgtm

[adamk, 2016-06-23 20:45:41]

The CQ bit was checked by adamk@chromium.org

[commit-bot: I haz the power, 2016-06-23 20:45:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2084103002/40001

[commit-bot: I haz the power, 2016-06-23 20:47:07]

Description was changed from

==========
Rewrite scopes in computed properties in destructured parameters

While we properly handled scopes of initializers in destructured parameters,
we never did the right thing for computed properties. This patch fixes that
by factoring out PatternRewriter's scope rewriting logic and calls it
for the computed property case.

BUG=chromium:620119
==========

to

==========
Rewrite scopes in computed properties in destructured parameters

While we properly handled scopes of initializers in destructured parameters,
we never did the right thing for computed properties. This patch fixes that
by factoring out PatternRewriter's scope rewriting logic and calls it
for the computed property case.

BUG=chromium:620119
==========

[commit-bot: I haz the power, 2016-06-23 20:47:08]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-23 20:47:26]

Description was changed from

==========
Rewrite scopes in computed properties in destructured parameters

While we properly handled scopes of initializers in destructured parameters,
we never did the right thing for computed properties. This patch fixes that
by factoring out PatternRewriter's scope rewriting logic and calls it
for the computed property case.

BUG=chromium:620119
==========

to

==========
Rewrite scopes in computed properties in destructured parameters

While we properly handled scopes of initializers in destructured parameters,
we never did the right thing for computed properties. This patch fixes that
by factoring out PatternRewriter's scope rewriting logic and calls it
for the computed property case.

BUG=chromium:620119

Committed: https://crrev.com/f795a790100832ff8eb7aba29676efe6beff8ea8
Cr-Commit-Position: refs/heads/master@{#37228}
==========

[commit-bot: I haz the power, 2016-06-23 20:47:27]

Patchset 3 (id:??) landed as
https://crrev.com/f795a790100832ff8eb7aba29676efe6beff8ea8
Cr-Commit-Position: refs/heads/master@{#37228}