Incorporating review feedback from https://codereview.chromium.org/24294002/

Source/bindings/dart/DartUtilities.cpp

 // Copyright 2011, 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 12 matching lines @@
 // SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
 // LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
 // DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
 // THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 #include "config.h"
 #include "bindings/dart/DartUtilities.h"
 
+#include "DartBlob.h"
+#include "DartIDBKeyRange.h"
+#include "DartImageData.h"
 #include "DartMessagePort.h"
 #include "bindings/dart/DartDOMData.h"
 #include "bindings/dart/DartHandleProxy.h"
 #include "bindings/dart/V8Converter.h"
 #include "bindings/v8/ScriptController.h"
 #include "bindings/v8/SerializedScriptValue.h"
 #include "core/dom/Document.h"
 #include "core/html/canvas/DataView.h"
 #include "core/inspector/ScriptArguments.h"
 #include "core/inspector/ScriptCallStack.h"
@@ skipping 296 matching lines @@
     return dartToDate(object, exception);
 }
 
 bool DartUtilities::isDateTime(Dart_Handle handle)
 {
     return objectIsType(handle, "dart:core", "DateTime");
 }
 
 bool DartUtilities::isBlob(Dart_Handle handle)
 {
-    return objectIsType(handle, htmlLibraryName, "Blob");
+    return objectIsClass(handle, DartDOMWrapper::dartClass<DartBlob>());
 }
 
 bool DartUtilities::isImageData(Dart_Handle handle)
 {
-    return objectIsType(handle, htmlLibraryName, "ImageData");
+    return objectIsClass(handle, DartDOMWrapper::dartClass<DartImageData>());
 }
 
 bool DartUtilities::isIDBKeyRange(Dart_Handle handle)
 {
-    return objectIsType(handle, indexedDBLibraryName, "KeyRange");
+    return objectIsClass(handle, DartDOMWrapper::dartClass<DartIDBKeyRange>());
+}
+
+bool DartUtilities::objectIsClass(Dart_Handle handle, Dart_Handle cls)
+{
+    bool isType = false;
+    Dart_Handle result = Dart_ObjectIsType(handle, cls, &isType);
+    ASSERT(!Dart_IsError(result));
+    if (Dart_IsError(result))
+        return false;
+
+    return isType;
 }
 
 bool DartUtilities::objectIsType(Dart_Handle handle, const char* libraryName, const char* typeName)
 {
     // FIXME: cache the lookup for library & type name.
     Dart_Handle library = libraryForCurrentIsolate(libraryName);
     ASSERT(!Dart_IsError(library));
 
     Dart_Handle cls = Dart_GetClass(library, Dart_NewStringFromCString(typeName));
     ASSERT(!Dart_IsError(cls));
@@ skipping 678 matching lines @@
     Dart_Handle library = htmlLibraryForCurrentIsolate();
     ASSERT(!Dart_IsError(library));
 
     Dart_Handle utilsClass = Dart_GetType(library, Dart_NewStringFromCString("_Utils"), 0, 0);
     ASSERT(!Dart_IsError(utilsClass));
 
     return Dart_Invoke(utilsClass, Dart_NewStringFromCString(methodName), argCount, args);
 }
 
 }