[api] Deprecate *MemoryAllocationCallback

[api] Deprecate *MemoryAllocationCallback

To keep track of memory characteristics of a JS program use
GetHeapSpaceStatistics from a combination of
- a GC epilogue callback registered using AddGCEpilogueCallback
- an interrupt registered using RequestInterrupt

Rationale:
The feature is currently broken as those callbacks are (also) executed from
background threads, breaking our implicit contract for only performing callbacks
on the foreground thread. Furhtermore, it's not clear for the embedder whether
these allocations originate in JS, or are required for a GC.

Note:
- The already deprecated non-isolate versions are removed with this CL.

R=jochen@chromium.org
BUG=v8:4813
LOG=Y

Committed: https://crrev.com/97e3909a53a1865df1032d31c3870395c210579a
Cr-Commit-Position: refs/heads/master@{#35474}

[Michael Lippautz, 2016-04-14 07:35:07]

Jochen: ptal
Hannes: fyi

[jochen (gone - plz use gerrit), 2016-04-14 07:41:23]

lgtm

[Michael Lippautz, 2016-04-14 07:44:34]

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

[commit-bot: I haz the power, 2016-04-14 07:44:39]

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

[commit-bot: I haz the power, 2016-04-14 07:53:52]

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

[commit-bot: I haz the power, 2016-04-14 07:53:53]

Dry run: Try jobs failed on following builders:
  v8_linux_gcc_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_gcc_compile_rel/bu...)

[Michael Lippautz, 2016-04-14 08:00:50]

Description was changed from

==========
[api] Deprecate *MemoryAllocationCallback

To keep track of memory characteristics of a JS program use
GetHeapSpaceStatistics from a combination of
- a GC epilogue callback registered using AddGCEpilogueCallback
- an interrupt registered using RequestInterrupt

Rationale:
The feature is currently broken as those callbacks are (also) executed from
background threads, breaking our implicit contract for only performing callbacks
on the foreground thread. Furhtermore, it's not clear for the embedder whether
these allocations originate in JS, or are required for a GC.

R=jochen@chromium.org
BUG=v8:4813
LOG=Y
==========

to

==========
[api] Deprecate *MemoryAllocationCallback

To keep track of memory characteristics of a JS program use
GetHeapSpaceStatistics from a combination of
- a GC epilogue callback registered using AddGCEpilogueCallback
- an interrupt registered using RequestInterrupt

Rationale:
The feature is currently broken as those callbacks are (also) executed from
background threads, breaking our implicit contract for only performing callbacks
on the foreground thread. Furhtermore, it's not clear for the embedder whether
these allocations originate in JS, or are required for a GC.

Note:
- The already deprecated non-isolate versions are removed with this CL.

R=jochen@chromium.org
BUG=v8:4813
LOG=Y
==========

[Michael Lippautz, 2016-04-14 08:03:42]

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

[commit-bot: I haz the power, 2016-04-14 08:03:45]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1886573006/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1886573006/20001

[commit-bot: I haz the power, 2016-04-14 08:27:58]

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

[commit-bot: I haz the power, 2016-04-14 08:27:59]

Dry run: This issue passed the CQ dry run.

[Michael Lippautz, 2016-04-14 08:51:47]

The CQ bit was checked by mlippautz@chromium.org

[Michael Lippautz, 2016-04-14 08:51:47]

The patchset sent to the CQ was uploaded after l-g-t-m from jochen@chromium.org
Link to the patchset: https://codereview.chromium.org/1886573006/#ps20001
(title: "Remove already deprecated non-isolate versions of the API")

[commit-bot: I haz the power, 2016-04-14 08:51:59]

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

[commit-bot: I haz the power, 2016-04-14 08:53:34]

Description was changed from

==========
[api] Deprecate *MemoryAllocationCallback

To keep track of memory characteristics of a JS program use
GetHeapSpaceStatistics from a combination of
- a GC epilogue callback registered using AddGCEpilogueCallback
- an interrupt registered using RequestInterrupt

Rationale:
The feature is currently broken as those callbacks are (also) executed from
background threads, breaking our implicit contract for only performing callbacks
on the foreground thread. Furhtermore, it's not clear for the embedder whether
these allocations originate in JS, or are required for a GC.

Note:
- The already deprecated non-isolate versions are removed with this CL.

R=jochen@chromium.org
BUG=v8:4813
LOG=Y
==========

to

==========
[api] Deprecate *MemoryAllocationCallback

To keep track of memory characteristics of a JS program use
GetHeapSpaceStatistics from a combination of
- a GC epilogue callback registered using AddGCEpilogueCallback
- an interrupt registered using RequestInterrupt

Rationale:
The feature is currently broken as those callbacks are (also) executed from
background threads, breaking our implicit contract for only performing callbacks
on the foreground thread. Furhtermore, it's not clear for the embedder whether
these allocations originate in JS, or are required for a GC.

Note:
- The already deprecated non-isolate versions are removed with this CL.

R=jochen@chromium.org
BUG=v8:4813
LOG=Y
==========

[commit-bot: I haz the power, 2016-04-14 08:53:35]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-14 08:53:47]

Description was changed from

==========
[api] Deprecate *MemoryAllocationCallback

To keep track of memory characteristics of a JS program use
GetHeapSpaceStatistics from a combination of
- a GC epilogue callback registered using AddGCEpilogueCallback
- an interrupt registered using RequestInterrupt

Rationale:
The feature is currently broken as those callbacks are (also) executed from
background threads, breaking our implicit contract for only performing callbacks
on the foreground thread. Furhtermore, it's not clear for the embedder whether
these allocations originate in JS, or are required for a GC.

Note:
- The already deprecated non-isolate versions are removed with this CL.

R=jochen@chromium.org
BUG=v8:4813
LOG=Y
==========

to

==========
[api] Deprecate *MemoryAllocationCallback

To keep track of memory characteristics of a JS program use
GetHeapSpaceStatistics from a combination of
- a GC epilogue callback registered using AddGCEpilogueCallback
- an interrupt registered using RequestInterrupt

Rationale:
The feature is currently broken as those callbacks are (also) executed from
background threads, breaking our implicit contract for only performing callbacks
on the foreground thread. Furhtermore, it's not clear for the embedder whether
these allocations originate in JS, or are required for a GC.

Note:
- The already deprecated non-isolate versions are removed with this CL.

R=jochen@chromium.org
BUG=v8:4813
LOG=Y

Committed: https://crrev.com/97e3909a53a1865df1032d31c3870395c210579a
Cr-Commit-Position: refs/heads/master@{#35474}
==========

[commit-bot: I haz the power, 2016-04-14 08:53:48]

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