- Bind CanvasPixelArray wrappers to the backing store directly without...

bindings/v8/V8DOMWrapper.cpp

Index: bindings/v8/V8DOMWrapper.cpp
===================================================================
--- bindings/v8/V8DOMWrapper.cpp	(revision 46194)
+++ bindings/v8/V8DOMWrapper.cpp	(working copy)
@@ -292,7 +292,7 @@
         descriptor->InstanceTemplate()->SetIndexedPropertyHandler(USE_INDEXED_PROPERTY_GETTER(HTMLFormElement), 0, 0, 0, nodeCollectionIndexedPropertyEnumerator<HTMLFormElement>, v8::Integer::New(V8ClassIndex::NODE));
         break;
     case V8ClassIndex::CANVASPIXELARRAY:
-        descriptor->InstanceTemplate()->SetIndexedPropertyHandler(USE_INDEXED_PROPERTY_GETTER(CanvasPixelArray), USE_INDEXED_PROPERTY_SETTER(CanvasPixelArray));
+        //descriptor->InstanceTemplate()->SetIndexedPropertyHandler(USE_INDEXED_PROPERTY_GETTER(CanvasPixelArray), USE_INDEXED_PROPERTY_SETTER(CanvasPixelArray));

[iposva, 2009/07/28 03:41:58]

This will be removed before commit, but it helps being here to debug for now.

         break;
     case V8ClassIndex::STYLESHEET:  // fall through
     case V8ClassIndex::CSSSTYLESHEET: {
@@ -559,6 +559,11 @@
                 setJSWrapperForActiveDOMObject(impl, result);
             else
                 setJSWrapperForDOMObject(impl, result);
+              
+            if (type == V8ClassIndex::CANVASPIXELARRAY) {
+              CanvasPixelArray* pixels = reinterpret_cast<CanvasPixelArray*>(impl);
+              result->SetElementsToPixelData(pixels->data()->data(), pixels->length());

[iposva, 2009/07/28 03:41:58]

I will change the API call in V8 to SetIndexedPropertiesToPixelData to align the
naming with the rest of the API.

+            }
 
             // Special case for non-node objects associated with a
             // DOMWindow. Both Safari and FF let the JS wrappers for these