expose AssertNoAllocation to api

expose AssertNoAllocation to api

R=svenpanne@chromium.org
BUG=

Committed: https://code.google.com/p/v8/source/detail?r=14531

[Sven Panne, 2013-04-30 12:21:30]

https://codereview.chromium.org/14625003/diff/1/src/heap-inl.h
File src/heap-inl.h (right):

https://codereview.chromium.org/14625003/diff/1/src/heap-inl.h#newcode872
src/heap-inl.h:872: return 0x10 || old_state;
This is "a bit" obscure. Why don't we simply remember the previous state (a
bool)?

https://codereview.chromium.org/14625003/diff/1/src/heap-inl.h#newcode879
src/heap-inl.h:879: if (data) isolate->heap()->allow_allocation(data);
Passing a uint8_t as a bool looks wrong. Again, a simple (remembered) bool
should be less confusing.

[marja, 2013-04-30 13:39:29]

https://codereview.chromium.org/14625003/diff/1/src/heap-inl.h
File src/heap-inl.h (right):

https://codereview.chromium.org/14625003/diff/1/src/heap-inl.h#newcode872
src/heap-inl.h:872: return 0x10 || old_state;
|| ? Maybe you meant |? Maybe something else?

[marja, 2013-04-30 13:42:43]

https://codereview.chromium.org/14625003/diff/1/src/heap-inl.h
File src/heap-inl.h (right):

https://codereview.chromium.org/14625003/diff/1/src/heap-inl.h#newcode872
src/heap-inl.h:872: return 0x10 || old_state;
Maybe you meant old_state || 0x10? :)

[Sven Panne, 2013-05-02 11:46:43]

lgtm

[dcarney, 2013-05-02 20:28:13]

Committed patchset #2 manually as r14531 (presubmit successful).