Add EscapableHandleScope to api to fix problems with HandleScope::Close

Add EscapableHandleScope to api to fix problems with HandleScope::Close

R=mstarzinger@chromium.org
BUG=

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

[dcarney, 2013-09-26 07:36:29]

ptal

[Michael Starzinger, 2013-09-26 09:48:10]

https://codereview.chromium.org/24245005/diff/1/include/v8.h
File include/v8.h (right):

https://codereview.chromium.org/24245005/diff/1/include/v8.h#newcode778
include/v8.h:778: // TODO(dcarney): deprecated - use
CloseableHandleScope::Close.
nit: s/Closeable/Escapeable/

https://codereview.chromium.org/24245005/diff/1/include/v8.h#newcode796
include/v8.h:796: V8_INLINE HandleScope() {}
I don't think we should expose these three methods, even if they are only
protected. Could we instead make EscapableHandleScope a friend of this class and
drop these three methods?

[dcarney, 2013-09-26 11:01:44]

https://codereview.chromium.org/24245005/diff/1/include/v8.h
File include/v8.h (right):

https://codereview.chromium.org/24245005/diff/1/include/v8.h#newcode778
include/v8.h:778: // TODO(dcarney): deprecated - use
CloseableHandleScope::Close.
On 2013/09/26 09:48:11, Michael Starzinger wrote:
> nit: s/Closeable/Escapeable/

Done.

https://codereview.chromium.org/24245005/diff/1/include/v8.h#newcode796
include/v8.h:796: V8_INLINE HandleScope() {}
On 2013/09/26 09:48:11, Michael Starzinger wrote:
> I don't think we should expose these three methods, even if they are only
> protected. Could we instead make EscapableHandleScope a friend of this class
and
> drop these three methods?

Done.

[Michael Starzinger, 2013-09-26 13:45:42]

LGTM.

[dcarney, 2013-09-26 13:47:00]

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