ArrayPop builtin handlified.

ArrayPop builtin handlified.

R=yangguo@chromium.org

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

[Igor Sheludko, 2014-03-19 14:15:41]

PTAL

[Yang, 2014-03-19 15:06:29]

LGTM.

https://codereview.chromium.org/204103008/diff/1/src/builtins.cc
File src/builtins.cc (right):

https://codereview.chromium.org/204103008/diff/1/src/builtins.cc#newcode566
src/builtins.cc:566: Handle<Object> maybe_elms =
name is confusing. It's not a MaybeObject.

[Yang, 2014-03-19 15:06:42]

On 2014/03/19 15:06:29, Yang wrote:
> LGTM.
> 
> https://codereview.chromium.org/204103008/diff/1/src/builtins.cc
> File src/builtins.cc (right):
> 
> https://codereview.chromium.org/204103008/diff/1/src/builtins.cc#newcode566
> src/builtins.cc:566: Handle<Object> maybe_elms =
> name is confusing. It's not a MaybeObject.


TEST(HandleNull) {
  CcTest::InitializeVM();
  Isolate* isolate = CcTest::i_isolate();
  HandleScope outer_scope(isolate);
  LocalContext context;
  Handle<Object> n(reinterpret_cast<Object*>(NULL), isolate);
  CHECK(!n.is_null());
}

The test case we discussed.

[Igor Sheludko, 2014-03-19 15:25:11]

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