Adding checks for V8 functions returning Maybe<bool>

third_party/WebKit/Source/bindings/core/v8/V8Binding.cpp

 /*
  * Copyright (C) 2006, 2007, 2008, 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 886 matching lines @@
 {
     v8::Local<v8::Value> arrayValue = object->GetInternalField(arrayIndex);
     if (!arrayValue->IsArray())
         return;
     v8::Local<v8::Array> array = v8::Local<v8::Array>::Cast(arrayValue);
     for (int i = array->Length() - 1; i >= 0; --i) {
         v8::Local<v8::Value> item;
         if (!array->Get(isolate->GetCurrentContext(), i).ToLocal(&item))
             return;
         if (item->StrictEquals(value)) {
-            array->Delete(isolate->GetCurrentContext(), i);
+            if (array->Delete(isolate->GetCurrentContext(), i).IsNothing()) {
+                // TODO: Not sure if this should be logged or an error thrown or
+                // if it is fine to just return
+            }
             return;
         }
     }
 }
 
 void moveEventListenerToNewWrapper(v8::Isolate* isolate, v8::Local<v8::Object> object, EventListener* oldValue, v8::Local<v8::Value> newValue, int arrayIndex)
 {
     if (oldValue) {
         V8AbstractEventListener* oldListener = V8AbstractEventListener::cast(oldValue);
         if (oldListener) {
@@ skipping 30 matching lines @@
     v8SetReturnValue(info, perContextData->constructorForType(WrapperTypeInfo::unwrap(data)));
 }
 
 v8::Local<v8::Value> freezeV8Object(v8::Local<v8::Value> value, v8::Isolate* isolate)
 {
     v8CallOrCrash(value.As<v8::Object>()->SetIntegrityLevel(isolate->GetCurrentContext(), v8::IntegrityLevel::kFrozen));
     return value;
 }
 
 } // namespace blink