Remove static CallCompletedCallback handlers

Remove static CallCompletedCallback handlers

BUG=none
R=svenpanne@chromium.org
LOG=y

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

[Sven Panne, 2014-04-24 07:08:20]

LGTM (although it contains weird code :-)

https://codereview.chromium.org/249313002/diff/1/src/isolate.cc
File src/isolate.cc (right):

https://codereview.chromium.org/249313002/diff/1/src/isolate.cc#newcode2281
src/isolate.cc:2281: if (!handle_scope_implementer()->CallDepthIsZero()) return;
This code has just moved, but nevertheless: I don't have the slightest clue why
the call depth counter is in the HandleScopeImplementer, this seems totally
weird, because it has nothing to do with Handles. o_O

[Sven Panne, 2014-04-24 07:08:21]

LGTM (although it contains weird code :-)

[Yang, 2014-04-24 07:16:38]

https://codereview.chromium.org/249313002/diff/1/src/isolate.cc
File src/isolate.cc (right):

https://codereview.chromium.org/249313002/diff/1/src/isolate.cc#newcode2281
src/isolate.cc:2281: if (!handle_scope_implementer()->CallDepthIsZero()) return;
On 2014/04/24 07:08:21, Sven Panne wrote:
> This code has just moved, but nevertheless: I don't have the slightest clue
why
> the call depth counter is in the HandleScopeImplementer, this seems totally
> weird, because it has nothing to do with Handles. o_O

Agree. This should just be moved into the isolate. But in a different CL.

[jochen (gone - plz use gerrit), 2014-04-25 13:49:42]

Committed patchset #1 manually as r20985 (presubmit successful).