Lolcode candidate: Both Expression and FunctionLiteral define the same accessor

Lolcode candidate: Both Expression and FunctionLiteral define an accessor is_parenthesized(), which access different flags. FunctionLiteral derives from Expression.

Given
      FunctionLiteral* a; a->is_parenthesized()
const FunctionLiteral* b; b->is_parenthesized()

the first accesses FunctionLiteral::IsParenthesized, the second accesses Expression::IsParenthesizedField.

Since these are distinct uses, we could rename them based on their use:
- Expression::is_parenthesized -> is_single_parenthesized
  Count # of parenthesis, for parsing & error handling:
  no parenthesis -> single parenthesis -> multi parenthesis
- FunctionLiteral::eager_compile_hint()
  Hint from parser to compiler about whether the parser suggests this function for eager compilation.

BUG=

Committed: https://crrev.com/63f7fbfea64fc6102a8fad7e570d95976e297d9a
Cr-Commit-Position: refs/heads/master@{#28042}

[vogelheim, 2015-04-23 17:31:57]

vogelheim@chromium.org changed reviewers:
+ dslomov@chromium.org, marja@chromium.org

[vogelheim, 2015-04-23 17:38:33]

The CQ bit was checked by vogelheim@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-04-23 17:38:45]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1097723005/1

[commit-bot: I haz the power, 2015-04-23 17:40:05]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-04-23 17:40:06]

Dry run: Try jobs failed on following builders:
  v8_linux64_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_rel/builds/5194)
  v8_linux_arm64_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm64_rel/builds/5113)
  v8_linux_mips64el_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mips64el_compile_r...)
  v8_linux_mipsel_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mipsel_compile_rel...)
  v8_linux_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_rel/builds/5125)
  v8_presubmit on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/2240)

[Dmitry Lomov (no reviews), 2015-04-23 18:51:00]

lol indeed.
lgtm

[Dmitry Lomov (no reviews), 2015-04-23 18:51:24]

(C++ is awesome)

[vogelheim, 2015-04-24 11:05:58]

The CQ bit was checked by vogelheim@chromium.org

[vogelheim, 2015-04-24 11:05:58]

The patchset sent to the CQ was uploaded after l-g-t-m from dslomov@chromium.org
Link to the patchset: https://codereview.chromium.org/1097723005/#ps20001
(title: "rebase")

[commit-bot: I haz the power, 2015-04-24 11:06:13]

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

[commit-bot: I haz the power, 2015-04-24 11:07:56]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-04-24 11:08:12]

Patchset 2 (id:??) landed as
https://crrev.com/63f7fbfea64fc6102a8fad7e570d95976e297d9a
Cr-Commit-Position: refs/heads/master@{#28042}