WebGL support.

Source/WebCore/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 108 matching lines @@
 
 struct IntegerTrait {
     typedef int64_t nativeType;
     static Dart_Handle convert(Dart_Handle object, int64_t* value)
     {
         return Dart_IntegerToInt64(object, value);
         // FIXME: support bigints.
     }
 };
 
+template <>
+double DartUtilities::toNative<double>(Dart_Handle object)
+{
+    return DartUtilities::toDouble(object);

[antonm, 2012/01/19 14:05:49]

I am not sure you can use those helpers, what if I pass something like [1, 'a']
into any GL function which uses those helpers?

[Nikolay, 2012/01/19 15:08:46]

Done.

+}
+
+template <>
+float DartUtilities::toNative<float>(Dart_Handle object)
+{
+    return DartUtilities::toDouble(object);
+}
+
+template <>
+int8_t DartUtilities::toNative<int8_t>(Dart_Handle object)
+{
+    return DartUtilities::toInteger(object);
+}
+
+template <>
+int16_t DartUtilities::toNative<int16_t>(Dart_Handle object)
+{
+    return DartUtilities::toInteger(object);
+}
+
+template <>
+int32_t DartUtilities::toNative<int32_t>(Dart_Handle object)
+{
+    return DartUtilities::toInteger(object);
+}
+
+template <>
+int64_t DartUtilities::toNative<int64_t>(Dart_Handle object)
+{
+    return DartUtilities::toInteger(object);
+}
+
+template <>
+uint8_t DartUtilities::toNative<uint8_t>(Dart_Handle object)
+{
+    return DartUtilities::toInteger(object);

[antonm, 2012/01/19 14:05:49]

What if I pass -1 into such a function?

[Nikolay, 2012/01/19 15:08:46]

c cast will apply. I think this behavior is natural.

On 2012/01/19 14:05:49, antonm wrote:

[antonm, 2012/01/19 15:33:13]

I don't think so, as in Dart numbers doesn't have fixed number of bits.  What
does JS do in those cases?

I think we should throw an exception.

On 2012/01/19 15:08:46, olonho wrote:

[Nikolay, 2012/01/20 11:38:37]

According to simple test in JS console this is behavior 
demonstrated by V8.

a = new Uint8Array(1)
Uint8Array
a[0] = -1
-1
a[0]
255

On 2012/01/19 15:33:13, antonm wrote:

[antonm, 2012/01/20 17:35:57]

Thanks a lot for verifying how JS works.

However, I don't think we should keep this logic in such a general place as
DartUtilities, please, keep it in WebGL related parts and name accordingly.

On 2012/01/20 11:38:37, olonho wrote:

+}
+
+template <>
+uint16_t DartUtilities::toNative<uint16_t>(Dart_Handle object)
+{
+    return DartUtilities::toInteger(object);
+}
+
+template <>
+uint32_t DartUtilities::toNative<uint32_t>(Dart_Handle object)
+{
+    return DartUtilities::toInteger(object);
+}
+
 int64_t DartUtilities::toInteger(Dart_Handle object, Dart_Handle& exception)
 {
     return convert<IntegerTrait>(object, exception);
 }
 
 int64_t DartUtilities::toInteger(Dart_Handle object)
 {
     Dart_Handle exception = 0;
     int64_t value = DartUtilities::toInteger(object, exception);
     ASSERT(!exception);
     return value;
 }
 
 struct DoubleTrait {
     typedef double nativeType;
     static Dart_Handle convert(Dart_Handle object, double* value)
     {
         if (!Dart_IsNumber(object))
             return Dart_Error("the object !is Number");
 
-        object = Dart_InvokeDynamic(object, Dart_NewString("toDouble"), 0, 0);
-        if (Dart_IsError(object))
-            return object;
+        if (!Dart_IsDouble(object)) {
+            object = Dart_InvokeDynamic(object, Dart_NewString("toDouble"), 0, 0);
+            if (Dart_IsError(object))
+                return object;
+        }
 
         return Dart_DoubleValue(object, value);
     }
 };
 
 double DartUtilities::toDouble(Dart_Handle object, Dart_Handle& exception)
 {
     return convert<DoubleTrait>(object, exception);
 }
 
@@ skipping 77 matching lines @@
     return DartNode::toNative(object, exception);
 }
 
 // FIXME: this function requires better testing. Currently blocking as new MessageChannel hasn't been implemented yet.
 void DartUtilities::toMessagePortArray(Dart_Handle value, MessagePortArray& portArray, Dart_Handle& exception)
 {
     Dart_Handle dom = Dart_LookupLibrary(Dart_NewString(DartUtilities::domLibraryName));
     ASSERT(!Dart_IsError(dom));
 
     Dart_Handle asList = Dart_InvokeStatic(dom, Dart_NewString("Utils"), Dart_NewString("convertToList"), 1, &value);
-    if (Dart_IsError(asList)) {
-        DartUtilities::reportProblem(DartUtilities::scriptExecutionContext(), asList);
-        exception = DartDOMWrapper::exceptionCodeToDartException(INVALID_STATE_ERR);
+    if (!checkResult(asList, exception))
         return;
-    }
     ASSERT(Dart_IsList(asList));
 
     intptr_t length = 0;
     Dart_ListLength(asList, &length);
     portArray.resize(length);
     for (int i = 0; i < length; i++) {
         Dart_Handle element = Dart_ListGetAt(asList, i);
-        if (Dart_IsError(element)) {
-            exception = DartDOMWrapper::exceptionCodeToDartException(INVALID_STATE_ERR);
+        if (!checkResult(element, exception))
             return;
-        }
 
         MessagePort* messagePort = DartMessagePort::toNative(element, exception).get();
-        if (exception) {
-            exception = DartDOMWrapper::exceptionCodeToDartException(INVALID_STATE_ERR);
+        if (exception)
             return;
-        }
 
         ASSERT(messagePort);
         portArray[i] = messagePort;
     }
 }
 
 class DartDOMData {
 public:
     DartDOMData()
         : m_scriptExecutionContext(0)
@@ skipping 192 matching lines @@
             errorMessage += String("\nError converting stack trace to a string: ") + Dart_GetError(stacktrace);
         else
             errorMessage += String("\nStack Trace: ") + DartUtilities::dartStringToString(stacktrace);
     }
 
     if (context && context->isDocument())
         static_cast<Document*>(context)->reportException(errorMessage, lineNumber, sourceFile, callStack);
 }
 
 }