[heap] Remove max_executable_size resource constraint.

[heap] Remove max_executable_size resource constraint.

BUG=chromium:716032
Review-Url: https://codereview.chromium.org/2890603007
Cr-Commit-Position: refs/heads/master@{#45400}
Committed: https://chromium.googlesource.com/v8/v8/+/a9e56f4f36d70d16a956367133dda258c4f52bf4

[Hannes Payer (out of office), 2017-05-18 13:38:59]

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

[commit-bot: I haz the power, 2017-05-18 13:39:10]

Dry run: CQ is trying da patch.

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

[Hannes Payer (out of office), 2017-05-18 13:41:14]

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

[commit-bot: I haz the power, 2017-05-18 13:41:21]

Dry run: CQ is trying da patch.

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

[Hannes Payer (out of office), 2017-05-18 14:17:22]

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

[commit-bot: I haz the power, 2017-05-18 14:17:33]

Dry run: CQ is trying da patch.

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

[ulan, 2017-05-18 14:30:46]

ulan@chromium.org changed reviewers:
+ ulan@chromium.org

[ulan, 2017-05-18 14:30:47]

lgtm

https://codereview.chromium.org/2890603007/diff/40001/include/v8.h
File include/v8.h (right):

https://codereview.chromium.org/2890603007/diff/40001/include/v8.h#newcode5908
include/v8.h:5908: V8_DEPRECATE_SOON("max_executable_size_ is subsumed
max_old_space_size_",
subsumed _by_

https://codereview.chromium.org/2890603007/diff/40001/include/v8.h#newcode5912
include/v8.h:5912: V8_DEPRECATE_SOON("max_executable_size_ is subsumed
max_old_space_size_",
ditto

[commit-bot: I haz the power, 2017-05-18 14:53:28]

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

[commit-bot: I haz the power, 2017-05-18 14:53:29]

Dry run: This issue passed the CQ dry run.

[Hannes Payer (out of office), 2017-05-18 16:57:04]

The CQ bit was checked by hpayer@chromium.org

[Hannes Payer (out of office), 2017-05-18 16:57:04]

The patchset sent to the CQ was uploaded after l-g-t-m from ulan@chromium.org
Link to the patchset: https://codereview.chromium.org/2890603007/#ps60001
(title: "comment")

[commit-bot: I haz the power, 2017-05-18 16:57:12]

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-05-18 17:37:58]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1495126624254010,
"parent_rev": "0439100a5f1264825a99a90cca7c8dd3311944c1", "commit_rev":
"a9e56f4f36d70d16a956367133dda258c4f52bf4"}

[commit-bot: I haz the power, 2017-05-18 17:38:05]

Description was changed from

==========
[heap] Remove max_executable_size resource constraint.

BUG=chromium:716032
==========

to

==========
[heap] Remove max_executable_size resource constraint.

BUG=chromium:716032

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

[commit-bot: I haz the power, 2017-05-18 17:38:07]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/v8/v8/+/a9e56f4f36d70d16a956367133dda258c4f...

[Hannes Payer (out of office), 2017-05-19 09:46:19]

https://codereview.chromium.org/2890603007/diff/40001/include/v8.h
File include/v8.h (right):

https://codereview.chromium.org/2890603007/diff/40001/include/v8.h#newcode5908
include/v8.h:5908: V8_DEPRECATE_SOON("max_executable_size_ is subsumed
max_old_space_size_",
On 2017/05/18 14:30:47, ulan wrote:
> subsumed _by_

Done.

https://codereview.chromium.org/2890603007/diff/40001/include/v8.h#newcode5912
include/v8.h:5912: V8_DEPRECATE_SOON("max_executable_size_ is subsumed
max_old_space_size_",
On 2017/05/18 14:30:47, ulan wrote:
> ditto

Done.