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

Source/bindings/dart/DartUtilities.h

 // 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 457 matching lines @@
     static PassRefPtr<SerializedScriptValue> dartToSerializedScriptValue(Dart_Handle, Dart_Handle& exception);
     static Dart_Handle serializedScriptValueToDart(PassRefPtr<SerializedScriptValue>);
 
     static Dart_Handle dateToDart(double);
     static double dartToDate(Dart_Handle, Dart_Handle&);
     static double dartToDate(Dart_NativeArguments args, int idx, Dart_Handle&);
     static bool isDateTime(Dart_Handle);
     static bool isBlob(Dart_Handle);
     static bool isImageData(Dart_Handle);
     static bool isIDBKeyRange(Dart_Handle);
-    static bool objectIsType(Dart_Handle, const char* libraryName, const char* typeName);
     static bool isTypeSubclassOf(Dart_Handle type, const char* libraryName, const char* typeName);
 
     static bool isTypedData(Dart_Handle);
 
     static Dart_Handle arrayBufferToDart(WTF::ArrayBuffer*);
     static Dart_Handle arrayBufferToDart(PassRefPtr<WTF::ArrayBuffer> value)
     {
         return arrayBufferToDart(value.get());
     }
 
@@ skipping 52 matching lines @@
     static PassRefPtr<WTF::Int32Array> dartToInt32Array(Dart_NativeArguments args, int idx, Dart_Handle& exception);
     static PassRefPtr<WTF::Uint8Array> dartToUint8Array(Dart_Handle, Dart_Handle&);
     static PassRefPtr<WTF::Uint8Array> dartToUint8Array(Dart_NativeArguments args, int idx, Dart_Handle& exception);
     static PassRefPtr<WTF::Float32Array> dartToFloat32Array(Dart_Handle, Dart_Handle&);
     static PassRefPtr<WTF::Float32Array> dartToFloat32Array(Dart_NativeArguments args, int idx, Dart_Handle& exception);
 
     static v8::Local<v8::Context> currentV8Context();
 
 private:
     static PassRefPtr<StringImpl> toStringImpl(Dart_Handle, intptr_t charsize, intptr_t strlength, Dart_Handle& exception);
+    static bool objectIsClass(Dart_Handle, Dart_Handle cls);

[siva, 2013/09/20 23:18:54]

In the dart API we have moved away from using the name Class or handing out
objects to the Class object. Instead we only refer to types and hand out only
Type objects.
I am wondering if we should call it
static bool objectIsType(Dart_Handle, Dart_Handle type);

Ryan, maybe we should rename DartDOMWrapper::dartClass to
DartDOMWrapper::dartType for consistency.

+    static bool objectIsType(Dart_Handle, const char* libraryName, const char* typeName);
 };
 
 class DartIsolateScope {
 public:
     explicit DartIsolateScope(Dart_Isolate isolate)
     {
         m_isolate = isolate;
         m_previousIsolate = Dart_CurrentIsolate();
         if (m_previousIsolate != m_isolate) {
             if (m_previousIsolate)
@@ skipping 82 matching lines @@
     }
 };
 
 
 #define DART_UNIMPLEMENTED_EXCEPTION() DartUtilities::notImplementedException(__FILE__, __LINE__)
 #define DART_UNIMPLEMENTED() Dart_ThrowException(DART_UNIMPLEMENTED_EXCEPTION());
 
 } // namespace WebCore
 
 #endif // DartUtilities_h