Fix: no-extra-parens invalid autofix in for-of statements

[not-an-aardvark]

What is the purpose of this pull request? (put an "X" next to item)

[x] Bug fix

Tell us about your environment

• ESLint Version: master
• Node Version: 7.7.4
• npm Version: 4.1.2

What parser (default, Babel-ESLint, etc.) are you using?

default

Please show your full configuration:

parserOptions:
  ecmaVersion: 6
rules:
  no-extra-parens: error

What did you do? Please include the actual source code causing the issue.

for (foo of(bar));

What did you expect to happen?

I expected the code to be fixed to valid syntax:

for (foo of bar);

What actually happened? Please include the actual, raw output from ESLint.

The code was autofixed to invalid syntax:

for (foo ofbar);

What changes did you make? (Give an overview)

This commit fixes a bug in the no-extra-parens autofixer where if the right side of a for-of predicate had extra parens, it would sometimes get combined with the of token.

Is there anything you'd like reviewers to focus on?

Nothing in particular

[mention-bot]

@not-an-aardvark, thanks for your PR! By analyzing the history of the files in this pull request, we identified @vitorbal, @kaicataldo and @mysticatea to be potential reviewers.

[eslintbot]

LGTM