[api] Add an API function to set the GetExternallyAllocatedMemoryInBytesCallback.

[api] Add an API function to set the GetExternallyAllocatedMemoryInBytesCallback.

BUG=chromium:727569
Review-Url: https://codereview.chromium.org/2910203002
Cr-Commit-Position: refs/heads/master@{#45599}
Committed: https://chromium.googlesource.com/v8/v8/+/f314b3cdb9766eb1ba6efe1243ac89b3d0295f63

[jochen (gone - plz use gerrit), 2017-05-30 12:43:06]

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

[jochen (gone - plz use gerrit), 2017-05-30 12:43:07]

lgtm

[Hannes Payer (out of office), 2017-05-30 12:43:30]

Description was changed from

==========
[api] Add an API function to set the
GetExternallyAllocatedMemoryInBytesCallback.

BUG=chromium:727569
==========

to

==========
[api] Add an API function to set the
GetExternallyAllocatedMemoryInBytesCallback.

BUG=chromium:727569
==========

[Hannes Payer (out of office), 2017-05-30 12:43:30]

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

[Michael Lippautz, 2017-05-30 12:45:10]

LGTM

https://codereview.chromium.org/2910203002/diff/60001/include/v8.h
File include/v8.h (right):

https://codereview.chromium.org/2910203002/diff/60001/include/v8.h#newcode7154
include/v8.h:7154: * externally of the V8 heap. Ideally this memory is somehow
connect to V8
connected

[Hannes Payer (out of office), 2017-05-30 12:47:08]

https://codereview.chromium.org/2910203002/diff/60001/include/v8.h
File include/v8.h (right):

https://codereview.chromium.org/2910203002/diff/60001/include/v8.h#newcode7154
include/v8.h:7154: * externally of the V8 heap. Ideally this memory is somehow
connect to V8
On 2017/05/30 12:45:10, Michael Lippautz wrote:
> connected

Done.

[Hannes Payer (out of office), 2017-05-30 12:47:19]

The CQ bit was checked by hpayer@chromium.org

[Hannes Payer (out of office), 2017-05-30 12:47:20]

The patchset sent to the CQ was uploaded after l-g-t-m from
mlippautz@chromium.org, jochen@chromium.org
Link to the patchset: https://codereview.chromium.org/2910203002/#ps80001
(title: "comment")

[commit-bot: I haz the power, 2017-05-30 12:47:28]

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-30 13:18:18]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1496148439607270,
"parent_rev": "fb6a094db5d1c95a73c66abe5f135785fbc08314", "commit_rev":
"f314b3cdb9766eb1ba6efe1243ac89b3d0295f63"}

[commit-bot: I haz the power, 2017-05-30 13:18:23]

Description was changed from

==========
[api] Add an API function to set the
GetExternallyAllocatedMemoryInBytesCallback.

BUG=chromium:727569
==========

to

==========
[api] Add an API function to set the
GetExternallyAllocatedMemoryInBytesCallback.

BUG=chromium:727569

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

[commit-bot: I haz the power, 2017-05-30 13:18:25]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/v8/v8/+/f314b3cdb9766eb1ba6efe1243ac89b3d02...