[Extensions Bindings] Add errors to signature parsing

extensions/renderer/argument_spec.cc

 // Copyright 2016 The Chromium 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 "extensions/renderer/argument_spec.h"
 
 #include "base/memory/ptr_util.h"
 #include "base/strings/string_piece.h"
 #include "base/values.h"
 #include "content/public/child/v8_value_converter.h"
@@ skipping 402 matching lines @@
 
     v8::Local<v8::Value> prop_value;
     // Fun: It's possible that a previous getter has removed the property from
     // the object. This isn't that big of a deal, since it would only manifest
     // in the case of some reasonably-crazy script objects, and it's probably
     // not worth optimizing for the uncommon case to the detriment of the
     // common (and either should be totally safe). We can always add a
     // HasOwnProperty() check here in the future, if we desire.
     // See also comment in ParseArgumentToArray() about passing in custom
     // crazy values here.
-    if (!object->Get(context, key).ToLocal(&prop_value))
+    if (!object->Get(context, key).ToLocal(&prop_value)) {
+      *error = api_errors::ScriptThrewError();
       return false;
+    }
 
     // Note: We don't serialize undefined or null values.
     // TODO(devlin): This matches current behavior, but it is correct?
     if (prop_value->IsUndefined() || prop_value->IsNull()) {
       if (!property_spec->optional_) {
         *error = api_errors::MissingRequiredProperty(*utf8_key);
         return false;
       }
       continue;
     }
@@ skipping 160 matching lines @@
       break;
     case ArgumentType::CHOICES:
     case ArgumentType::ANY:
       NOTREACHED();
   }
 
   return api_errors::InvalidType(expected_type, GetV8ValueTypeString(value));
 }
 
 }  // namespace extensions