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

src/accessors.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "v8.h"
 #include "accessors.h"
 
 #include "compiler.h"
 #include "contexts.h"
 #include "deoptimizer.h"
@@ skipping 179 matching lines @@
     return;
   }
   Handle<Object> number_v;
   maybe = Execution::ToNumber(isolate, value);
   if (!maybe.ToHandle(&number_v)) {
     isolate->OptionalRescheduleException(false);
     return;
   }
 
   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.ToHandleChecked();

[Yang, 2014/04/29 12:45:58]

Using .Check() should suffice.

     return;
   }
 
   isolate->ScheduleThrow(
       *isolate->factory()->NewRangeError("invalid_array_length",
                                          HandleVector<Object>(NULL, 0)));
 }
 
 
 Handle<AccessorInfo> Accessors::ArrayLengthInfo(
@@ skipping 1100 matching lines @@
   info->set_data(Smi::FromInt(index));
   Handle<Object> getter = v8::FromCData(isolate, &ModuleGetExport);
   Handle<Object> setter = v8::FromCData(isolate, &ModuleSetExport);
   info->set_getter(*getter);
   if (!(attributes & ReadOnly)) info->set_setter(*setter);
   return info;
 }
 
 
 } }  // namespace v8::internal