Check that JSArray::SetElementsLength does not throw in array.length accessor.

BUG=
R=yangguo@chromium.org

Review URL: https://codereview.chromium.org/252743010

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@21045 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

diff --git a/src/accessors.cc b/src/accessors.cc
index ce8884a..8c8fcdd 100644 (file)
--- a/src/accessors.cc
+++ b/src/accessors.cc
@@ -175,7 +175,7 @@ void Accessors::ArrayLengthSetter(
     MaybeHandle<Object> maybe_result =
         JSObject::SetLocalPropertyIgnoreAttributes(
             object, isolate->factory()->length_string(), value, NONE);
-    maybe_result.ToHandleChecked();
+    maybe_result.Check();
     return;
   }
 
@@ -197,9 +197,8 @@ void Accessors::ArrayLengthSetter(
   }
 
   if (uint32_v->Number() == number_v->Number()) {
-    MaybeHandle<Object> result;
-    result = JSArray::SetElementsLength(array_handle, uint32_v);
-    USE(result);
+    maybe = JSArray::SetElementsLength(array_handle, uint32_v);
+    maybe.Check();
     return;
   }