[Extensions Bindings] Consider argument type more in signature parsing

extensions/renderer/bindings/api_signature.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/bindings/api_signature.h"
 
 #include <algorithm>
 
 #include "base/memory/ptr_util.h"
 #include "base/strings/string_util.h"
@@ skipping 192 matching lines @@
       return false;
     }
     // This is safe to call even if |arguments| is at the end (which can happen
     // if n optional arguments are omitted at the end of the signature).
     ConsumeArgument();
 
     AddNull();
     return true;
   }
 
-  if (!spec.ParseArgument(context_, value, type_refs_, GetBuffer(),
-                          &parse_error_)) {
+  if (!spec.IsCorrectType(value, type_refs_, &parse_error_)) {
     if (!spec.optional()) {
       *error_ = api_errors::ArgumentError(spec.name(), parse_error_);
       return false;
     }
 
     AddNull();
     return true;
   }
 
+  if (!spec.ParseArgument(context_, value, type_refs_, GetBuffer(),
+                          &parse_error_)) {
+    *error_ = api_errors::ArgumentError(spec.name(), parse_error_);
+    return false;
+  }
+
   ConsumeArgument();
   AddParsedArgument(value);
   return true;
 }
 
 bool ArgumentParser::ParseCallback(const ArgumentSpec& spec) {
   v8::Local<v8::Value> value = next_argument();
   if (value.IsEmpty() || value->IsNull() || value->IsUndefined()) {
     if (!spec.optional()) {
       *error_ = api_errors::MissingRequiredArgument(spec.name().c_str());
@@ skipping 120 matching lines @@
     pieces.push_back(
         base::StringPrintf("%s%s %s", spec->optional() ? kOptionalPrefix : "",
                            spec->GetTypeName().c_str(), spec->name().c_str()));
   }
   expected_signature_ = base::JoinString(pieces, ", ");
 
   return expected_signature_;
 }
 
 }  // namespace extensions