Unify TypeError messages

Unify TypeError messages

This CL fixes some nits in TypeError messages, unifying the form of
kDefineDisallowed and kObjectNotExtensible to match what is used by the
majority of the other messages:
  * "Cannot" vs. "Can't" -> choose "Cannot"
  * "property:%" -> "property %"
  * omit the full-stop at the end of the message

BUG=v8:5673
Review-Url: https://codereview.chromium.org/2686233008
Cr-Commit-Position: refs/heads/master@{#43150}
Committed: https://chromium.googlesource.com/v8/v8/+/e08f85fcfe293a67baf2f7b7d558133ba9d2acba

[vabr (Chromium), 2017-02-12 19:47:39]

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

[commit-bot: I haz the power, 2017-02-12 19:47:50]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-12 20:04:28]

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

[commit-bot: I haz the power, 2017-02-12 20:04:28]

Dry run: Try jobs failed on following builders:
  v8_linux64_asan_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel_ng/buil...)
  v8_linux64_asan_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel_ng_trig...)

[vabr (Chromium), 2017-02-12 20:27:42]

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

[commit-bot: I haz the power, 2017-02-12 20:27:48]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-12 20:50:14]

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

[commit-bot: I haz the power, 2017-02-12 20:50:15]

Dry run: This issue passed the CQ dry run.

[vabr (Chromium), 2017-02-12 21:05:26]

vabr@chromium.org changed reviewers:
+ hpayer@chromium.org

[vabr (Chromium), 2017-02-12 21:05:27]

Hi Hannes,

Could you please review this?

Cheers,
Vaclav

[Hannes Payer (out of office), 2017-02-13 08:20:25]

lgtm

[vabr (Chromium), 2017-02-13 10:25:24]

The CQ bit was checked by vabr@chromium.org

[commit-bot: I haz the power, 2017-02-13 10:25:29]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-13 10:27:04]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1486981524796690,
"parent_rev": "6ee0b6cec6b83791af8d3a7e122f14e4eb376974", "commit_rev":
"e08f85fcfe293a67baf2f7b7d558133ba9d2acba"}

[commit-bot: I haz the power, 2017-02-13 10:27:09]

Description was changed from

==========
Unify TypeError messages

This CL fixes some nits in TypeError messages, unifying the form of
kDefineDisallowed and kObjectNotExtensible to match what is used by the
majority of the other messages:
  * "Cannot" vs. "Can't" -> choose "Cannot"
  * "property:%" -> "property %"
  * omit the full-stop at the end of the message

BUG=v8:5673
==========

to

==========
Unify TypeError messages

This CL fixes some nits in TypeError messages, unifying the form of
kDefineDisallowed and kObjectNotExtensible to match what is used by the
majority of the other messages:
  * "Cannot" vs. "Can't" -> choose "Cannot"
  * "property:%" -> "property %"
  * omit the full-stop at the end of the message

BUG=v8:5673

Review-Url: https://codereview.chromium.org/2686233008
Cr-Commit-Position: refs/heads/master@{#43150}
Committed:
https://chromium.googlesource.com/v8/v8/+/e08f85fcfe293a67baf2f7b7d558133ba9d...
==========

[commit-bot: I haz the power, 2017-02-13 10:27:09]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/v8/v8/+/e08f85fcfe293a67baf2f7b7d558133ba9d...