Title: [118133] trunk/Source/WebCore
Revision
118133
Author
[email protected]
Date
2012-05-23 00:06:11 -0700 (Wed, 23 May 2012)

Log Message

[V8] Pass Isolate to v8::Null() in custom bindings (Part 1)
https://bugs.webkit.org/show_bug.cgi?id=87207

Reviewed by Adam Barth.

The objective is to pass Isolate around in V8 bindings.
This patch passes Isolate to v8::Null() in custom bindings.

Also this patch makes V8DeviceMotionEventCustom::createAccelerationObject()
and V8DeviceMotionEventCustom::createRotationRateObject() static.

No tests. No change in behavior.

* bindings/v8/custom/V8BlobCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8CSSRuleCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8CSSStyleSheetCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8CSSValueCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8ClipboardCustom.cpp:
(WebCore::V8Clipboard::typesAccessorGetter):
* bindings/v8/custom/V8CoordinatesCustom.cpp:
(WebCore::V8Coordinates::altitudeAccessorGetter):
(WebCore::V8Coordinates::altitudeAccuracyAccessorGetter):
(WebCore::V8Coordinates::headingAccessorGetter):
(WebCore::V8Coordinates::speedAccessorGetter):
* bindings/v8/custom/V8DOMStringMapCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8DOMTokenListCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8DOMWindowCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8DataViewCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8DeviceMotionEventCustom.cpp:
(WebCore::V8DeviceMotionEvent::accelerationAccessorGetter):
(WebCore::V8DeviceMotionEvent::accelerationIncludingGravityAccessorGetter):
(WebCore::V8DeviceMotionEvent::rotationRateAccessorGetter):
(WebCore::V8DeviceMotionEvent::intervalAccessorGetter):
* bindings/v8/custom/V8DeviceOrientationEventCustom.cpp:
(WebCore::V8DeviceOrientationEvent::alphaAccessorGetter):
(WebCore::V8DeviceOrientationEvent::betaAccessorGetter):
(WebCore::V8DeviceOrientationEvent::gammaAccessorGetter):
(WebCore::V8DeviceOrientationEvent::absoluteAccessorGetter):
* bindings/v8/custom/V8DocumentCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8DocumentLocationCustom.cpp:
(WebCore::V8Document::locationAccessorGetter):
* bindings/v8/custom/V8EntryCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8EntrySyncCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8EventCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8Float32ArrayCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8Float64ArrayCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8HTMLCanvasElementCustom.cpp:
(WebCore::V8HTMLCanvasElement::getContextCallback):
* bindings/v8/custom/V8HTMLDocumentCustom.cpp:
(WebCore::toV8):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (118132 => 118133)


--- trunk/Source/WebCore/ChangeLog	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/ChangeLog	2012-05-23 07:06:11 UTC (rev 118133)
@@ -1,3 +1,70 @@
+2012-05-23  Kentaro Hara  <[email protected]>
+
+        [V8] Pass Isolate to v8::Null() in custom bindings (Part 1)
+        https://bugs.webkit.org/show_bug.cgi?id=87207
+
+        Reviewed by Adam Barth.
+
+        The objective is to pass Isolate around in V8 bindings.
+        This patch passes Isolate to v8::Null() in custom bindings.
+
+        Also this patch makes V8DeviceMotionEventCustom::createAccelerationObject()
+        and V8DeviceMotionEventCustom::createRotationRateObject() static.
+
+        No tests. No change in behavior.
+
+        * bindings/v8/custom/V8BlobCustom.cpp:
+        (WebCore::toV8):
+        * bindings/v8/custom/V8CSSRuleCustom.cpp:
+        (WebCore::toV8):
+        * bindings/v8/custom/V8CSSStyleSheetCustom.cpp:
+        (WebCore::toV8):
+        * bindings/v8/custom/V8CSSValueCustom.cpp:
+        (WebCore::toV8):
+        * bindings/v8/custom/V8ClipboardCustom.cpp:
+        (WebCore::V8Clipboard::typesAccessorGetter):
+        * bindings/v8/custom/V8CoordinatesCustom.cpp:
+        (WebCore::V8Coordinates::altitudeAccessorGetter):
+        (WebCore::V8Coordinates::altitudeAccuracyAccessorGetter):
+        (WebCore::V8Coordinates::headingAccessorGetter):
+        (WebCore::V8Coordinates::speedAccessorGetter):
+        * bindings/v8/custom/V8DOMStringMapCustom.cpp:
+        (WebCore::toV8):
+        * bindings/v8/custom/V8DOMTokenListCustom.cpp:
+        (WebCore::toV8):
+        * bindings/v8/custom/V8DOMWindowCustom.cpp:
+        (WebCore::toV8):
+        * bindings/v8/custom/V8DataViewCustom.cpp:
+        (WebCore::toV8):
+        * bindings/v8/custom/V8DeviceMotionEventCustom.cpp:
+        (WebCore::V8DeviceMotionEvent::accelerationAccessorGetter):
+        (WebCore::V8DeviceMotionEvent::accelerationIncludingGravityAccessorGetter):
+        (WebCore::V8DeviceMotionEvent::rotationRateAccessorGetter):
+        (WebCore::V8DeviceMotionEvent::intervalAccessorGetter):
+        * bindings/v8/custom/V8DeviceOrientationEventCustom.cpp:
+        (WebCore::V8DeviceOrientationEvent::alphaAccessorGetter):
+        (WebCore::V8DeviceOrientationEvent::betaAccessorGetter):
+        (WebCore::V8DeviceOrientationEvent::gammaAccessorGetter):
+        (WebCore::V8DeviceOrientationEvent::absoluteAccessorGetter):
+        * bindings/v8/custom/V8DocumentCustom.cpp:
+        (WebCore::toV8):
+        * bindings/v8/custom/V8DocumentLocationCustom.cpp:
+        (WebCore::V8Document::locationAccessorGetter):
+        * bindings/v8/custom/V8EntryCustom.cpp:
+        (WebCore::toV8):
+        * bindings/v8/custom/V8EntrySyncCustom.cpp:
+        (WebCore::toV8):
+        * bindings/v8/custom/V8EventCustom.cpp:
+        (WebCore::toV8):
+        * bindings/v8/custom/V8Float32ArrayCustom.cpp:
+        (WebCore::toV8):
+        * bindings/v8/custom/V8Float64ArrayCustom.cpp:
+        (WebCore::toV8):
+        * bindings/v8/custom/V8HTMLCanvasElementCustom.cpp:
+        (WebCore::V8HTMLCanvasElement::getContextCallback):
+        * bindings/v8/custom/V8HTMLDocumentCustom.cpp:
+        (WebCore::toV8):
+
 2012-05-22  MORITA Hajime  <[email protected]>
 
         [Refactoring] ElementShadow::m_needsRedistributing  should be on ContentDistributor

Modified: trunk/Source/WebCore/bindings/v8/custom/V8BlobCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8BlobCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8BlobCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -47,7 +47,7 @@
 v8::Handle<v8::Value> toV8(Blob* impl, v8::Isolate* isolate)
 {
     if (!impl)
-        return v8::Null();
+        return v8::Null(isolate);
 
     if (impl->isFile())
         return toV8(static_cast<File*>(impl), isolate);

Modified: trunk/Source/WebCore/bindings/v8/custom/V8CSSRuleCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8CSSRuleCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8CSSRuleCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -46,7 +46,7 @@
 v8::Handle<v8::Value> toV8(CSSRule* impl, v8::Isolate* isolate)
 {
     if (!impl)
-        return v8::Null();
+        return v8::Null(isolate);
     switch (impl->type()) {
     case CSSRule::UNKNOWN_RULE:
         // CSSUnknownRule.idl is explicitly excluded as it doesn't add anything

Modified: trunk/Source/WebCore/bindings/v8/custom/V8CSSStyleSheetCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8CSSStyleSheetCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8CSSStyleSheetCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -39,7 +39,7 @@
 v8::Handle<v8::Value> toV8(CSSStyleSheet* impl, v8::Isolate* isolate)
 {
     if (!impl)
-        return v8::Null();
+        return v8::Null(isolate);
     v8::Handle<v8::Object> wrapper = V8CSSStyleSheet::wrap(impl, isolate);
     // Add a hidden reference from stylesheet object to its owner node.
     Node* ownerNode = impl->ownerNode();

Modified: trunk/Source/WebCore/bindings/v8/custom/V8CSSValueCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8CSSValueCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8CSSValueCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -45,7 +45,7 @@
 v8::Handle<v8::Value> toV8(CSSValue* impl, v8::Isolate* isolate)
 {
     if (!impl)
-        return v8::Null();
+        return v8::Null(isolate);
     if (impl->isWebKitCSSTransformValue())
         return toV8(static_cast<WebKitCSSTransformValue*>(impl), isolate);
     if (impl->isValueList())

Modified: trunk/Source/WebCore/bindings/v8/custom/V8ClipboardCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8ClipboardCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8ClipboardCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -51,7 +51,7 @@
 
     HashSet<String> types = clipboard->types();
     if (types.isEmpty())
-        return v8::Null();
+        return v8::Null(info.GetIsolate());
 
     v8::Local<v8::Array> result = v8::Array::New(types.size());
     HashSet<String>::const_iterator end = types.end();

Modified: trunk/Source/WebCore/bindings/v8/custom/V8CoordinatesCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8CoordinatesCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8CoordinatesCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -38,7 +38,7 @@
     v8::Handle<v8::Object> holder = info.Holder();
     Coordinates* imp = V8Coordinates::toNative(holder);
     if (!imp->canProvideAltitude())
-        return v8::Null();
+        return v8::Null(info.GetIsolate());
     return v8::Number::New(imp->altitude());
 }
 
@@ -48,7 +48,7 @@
     v8::Handle<v8::Object> holder = info.Holder();
     Coordinates* imp = V8Coordinates::toNative(holder);
     if (!imp->canProvideAltitudeAccuracy())
-        return v8::Null();
+        return v8::Null(info.GetIsolate());
     return v8::Number::New(imp->altitudeAccuracy());
 }
 
@@ -58,7 +58,7 @@
     v8::Handle<v8::Object> holder = info.Holder();
     Coordinates* imp = V8Coordinates::toNative(holder);
     if (!imp->canProvideHeading())
-        return v8::Null();
+        return v8::Null(info.GetIsolate());
     return v8::Number::New(imp->heading());
 }
 
@@ -68,7 +68,7 @@
     v8::Handle<v8::Object> holder = info.Holder();
     Coordinates* imp = V8Coordinates::toNative(holder);
     if (!imp->canProvideSpeed())
-        return v8::Null();
+        return v8::Null(info.GetIsolate());
     return v8::Number::New(imp->speed());
 }
 

Modified: trunk/Source/WebCore/bindings/v8/custom/V8DOMStringMapCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8DOMStringMapCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8DOMStringMapCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -87,7 +87,7 @@
 v8::Handle<v8::Value> toV8(DOMStringMap* impl, v8::Isolate* isolate)
 {
     if (!impl)
-        return v8::Null();
+        return v8::Null(isolate);
     v8::Handle<v8::Object> wrapper = V8DOMStringMap::wrap(impl, isolate);
     // Add a hidden reference from the element to the DOMStringMap.
     Element* element = impl->element();

Modified: trunk/Source/WebCore/bindings/v8/custom/V8DOMTokenListCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8DOMTokenListCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8DOMTokenListCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -41,7 +41,7 @@
 v8::Handle<v8::Value> toV8(DOMTokenList* impl, v8::Isolate* isolate)
 {
     if (!impl)
-        return v8::Null();
+        return v8::Null(isolate);
     v8::Handle<v8::Object> wrapper = V8DOMTokenList::wrap(impl, isolate);
     // Add a hidden reference from the element to the DOMTokenList.
     Element* element = impl->element();

Modified: trunk/Source/WebCore/bindings/v8/custom/V8DOMWindowCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8DOMWindowCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8DOMWindowCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -603,7 +603,7 @@
 v8::Handle<v8::Value> toV8(DOMWindow* window, v8::Isolate* isolate)
 {
     if (!window)
-        return v8::Null();
+        return v8::Null(isolate);
     // Initializes environment of a frame, and return the global object
     // of the frame.
     Frame* frame = window->frame();

Modified: trunk/Source/WebCore/bindings/v8/custom/V8DataViewCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8DataViewCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8DataViewCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -60,7 +60,7 @@
 v8::Handle<v8::Value> toV8(DataView* impl, v8::Isolate* isolate)
 {
     if (!impl)
-        return v8::Null();
+        return v8::Null(isolate);
     return V8DataView::wrap(impl, isolate);
 }
 

Modified: trunk/Source/WebCore/bindings/v8/custom/V8DeviceMotionEventCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8DeviceMotionEventCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8DeviceMotionEventCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -39,21 +39,21 @@
 
 namespace {
 
-v8::Handle<v8::Value> createAccelerationObject(const DeviceMotionData::Acceleration* acceleration)
+static v8::Handle<v8::Value> createAccelerationObject(const DeviceMotionData::Acceleration* acceleration, v8::Isolate* isolate)
 {
     v8::Local<v8::Object> object = v8::Object::New();
-    object->Set(v8::String::New("x"), acceleration->canProvideX() ? v8::Number::New(acceleration->x()) : v8::Null());
-    object->Set(v8::String::New("y"), acceleration->canProvideY() ? v8::Number::New(acceleration->y()) : v8::Null());
-    object->Set(v8::String::New("z"), acceleration->canProvideZ() ? v8::Number::New(acceleration->z()) : v8::Null());
+    object->Set(v8::String::New("x"), acceleration->canProvideX() ? v8::Number::New(acceleration->x()) : v8::Null(isolate));
+    object->Set(v8::String::New("y"), acceleration->canProvideY() ? v8::Number::New(acceleration->y()) : v8::Null(isolate));
+    object->Set(v8::String::New("z"), acceleration->canProvideZ() ? v8::Number::New(acceleration->z()) : v8::Null(isolate));
     return object;
 }
 
-v8::Handle<v8::Value> createRotationRateObject(const DeviceMotionData::RotationRate* rotationRate)
+static v8::Handle<v8::Value> createRotationRateObject(const DeviceMotionData::RotationRate* rotationRate, v8::Isolate* isolate)
 {
     v8::Local<v8::Object> object = v8::Object::New();
-    object->Set(v8::String::New("alpha"), rotationRate->canProvideAlpha() ? v8::Number::New(rotationRate->alpha()) : v8::Null());
-    object->Set(v8::String::New("beta"),  rotationRate->canProvideBeta()  ? v8::Number::New(rotationRate->beta())  : v8::Null());
-    object->Set(v8::String::New("gamma"), rotationRate->canProvideGamma() ? v8::Number::New(rotationRate->gamma()) : v8::Null());
+    object->Set(v8::String::New("alpha"), rotationRate->canProvideAlpha() ? v8::Number::New(rotationRate->alpha()) : v8::Null(isolate));
+    object->Set(v8::String::New("beta"),  rotationRate->canProvideBeta()  ? v8::Number::New(rotationRate->beta())  : v8::Null(isolate));
+    object->Set(v8::String::New("gamma"), rotationRate->canProvideGamma() ? v8::Number::New(rotationRate->gamma()) : v8::Null(isolate));
     return object;
 }
 
@@ -129,8 +129,8 @@
     v8::Handle<v8::Object> holder = info.Holder();
     DeviceMotionEvent* imp = V8DeviceMotionEvent::toNative(holder);
     if (!imp->deviceMotionData()->acceleration())
-        return v8::Null();
-    return createAccelerationObject(imp->deviceMotionData()->acceleration());
+        return v8::Null(info.GetIsolate());
+    return createAccelerationObject(imp->deviceMotionData()->acceleration(), info.GetIsolate());
 }
 
 v8::Handle<v8::Value> V8DeviceMotionEvent::accelerationIncludingGravityAccessorGetter(v8::Local<v8::String> name, const v8::AccessorInfo& info)
@@ -139,8 +139,8 @@
     v8::Handle<v8::Object> holder = info.Holder();
     DeviceMotionEvent* imp = V8DeviceMotionEvent::toNative(holder);
     if (!imp->deviceMotionData()->accelerationIncludingGravity())
-        return v8::Null();
-    return createAccelerationObject(imp->deviceMotionData()->accelerationIncludingGravity());
+        return v8::Null(info.GetIsolate());
+    return createAccelerationObject(imp->deviceMotionData()->accelerationIncludingGravity(), info.GetIsolate());
 }
 
 v8::Handle<v8::Value> V8DeviceMotionEvent::rotationRateAccessorGetter(v8::Local<v8::String> name, const v8::AccessorInfo& info)
@@ -149,8 +149,8 @@
     v8::Handle<v8::Object> holder = info.Holder();
     DeviceMotionEvent* imp = V8DeviceMotionEvent::toNative(holder);
     if (!imp->deviceMotionData()->rotationRate())
-        return v8::Null();
-    return createRotationRateObject(imp->deviceMotionData()->rotationRate());
+        return v8::Null(info.GetIsolate());
+    return createRotationRateObject(imp->deviceMotionData()->rotationRate(), info.GetIsolate());
 }
 
 v8::Handle<v8::Value> V8DeviceMotionEvent::intervalAccessorGetter(v8::Local<v8::String> name, const v8::AccessorInfo& info)
@@ -159,7 +159,7 @@
     v8::Handle<v8::Object> holder = info.Holder();
     DeviceMotionEvent* imp = V8DeviceMotionEvent::toNative(holder);
     if (!imp->deviceMotionData()->canProvideInterval())
-        return v8::Null();
+        return v8::Null(info.GetIsolate());
     return v8::Number::New(imp->deviceMotionData()->interval());
 }
 

Modified: trunk/Source/WebCore/bindings/v8/custom/V8DeviceOrientationEventCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8DeviceOrientationEventCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8DeviceOrientationEventCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -43,7 +43,7 @@
     v8::Handle<v8::Object> holder = info.Holder();
     DeviceOrientationEvent* imp = V8DeviceOrientationEvent::toNative(holder);
     if (!imp->orientation()->canProvideAlpha())
-        return v8::Null();
+        return v8::Null(info.GetIsolate());
     return v8::Number::New(imp->orientation()->alpha());
 }
 
@@ -53,7 +53,7 @@
     v8::Handle<v8::Object> holder = info.Holder();
     DeviceOrientationEvent* imp = V8DeviceOrientationEvent::toNative(holder);
     if (!imp->orientation()->canProvideBeta())
-        return v8::Null();
+        return v8::Null(info.GetIsolate());
     return v8::Number::New(imp->orientation()->beta());
 }
 
@@ -63,7 +63,7 @@
     v8::Handle<v8::Object> holder = info.Holder();
     DeviceOrientationEvent* imp = V8DeviceOrientationEvent::toNative(holder);
     if (!imp->orientation()->canProvideGamma())
-        return v8::Null();
+        return v8::Null(info.GetIsolate());
     return v8::Number::New(imp->orientation()->gamma());
 }
 
@@ -73,7 +73,7 @@
     v8::Handle<v8::Object> holder = info.Holder();
     DeviceOrientationEvent* imp = V8DeviceOrientationEvent::toNative(holder);
     if (!imp->orientation()->canProvideAbsolute())
-        return v8::Null();
+        return v8::Null(info.GetIsolate());
     return v8::Boolean::New(imp->orientation()->absolute());
 }
 

Modified: trunk/Source/WebCore/bindings/v8/custom/V8DocumentCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8DocumentCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8DocumentCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -119,7 +119,7 @@
 v8::Handle<v8::Value> toV8(Document* impl, v8::Isolate* isolate, bool forceNewObject)
 {
     if (!impl)
-        return v8::Null();
+        return v8::Null(isolate);
     if (impl->isHTMLDocument())
         return toV8(static_cast<HTMLDocument*>(impl), isolate, forceNewObject);
 #if ENABLE(SVG)

Modified: trunk/Source/WebCore/bindings/v8/custom/V8DocumentLocationCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8DocumentLocationCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8DocumentLocationCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -38,7 +38,7 @@
 {
     Document* document = V8Document::toNative(info.Holder());
     if (!document->frame())
-        return v8::Null();
+        return v8::Null(info.GetIsolate());
 
     DOMWindow* window = document->frame()->domWindow();
     return toV8(window->location(), info.GetIsolate());

Modified: trunk/Source/WebCore/bindings/v8/custom/V8EntryCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8EntryCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8EntryCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -47,7 +47,7 @@
 v8::Handle<v8::Value> toV8(Entry* impl, v8::Isolate* isolate)
 {
     if (!impl)
-        return v8::Null();
+        return v8::Null(isolate);
 
     if (impl->isFile())
         return toV8(static_cast<FileEntry*>(impl), isolate);

Modified: trunk/Source/WebCore/bindings/v8/custom/V8EntrySyncCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8EntrySyncCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8EntrySyncCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -47,7 +47,7 @@
 v8::Handle<v8::Value> toV8(EntrySync* impl, v8::Isolate* isolate)
 {
     if (!impl)
-        return v8::Null();
+        return v8::Null(isolate);
 
     if (impl->isFile())
         return toV8(static_cast<FileEntrySync*>(impl), isolate);

Modified: trunk/Source/WebCore/bindings/v8/custom/V8EventCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8EventCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8EventCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -76,7 +76,7 @@
 v8::Handle<v8::Value> toV8(Event* event, v8::Isolate *isolate)
 {
     if (!event)
-        return v8::Null();
+        return v8::Null(isolate);
 
     String desiredInterface = event->interfaceName();
 

Modified: trunk/Source/WebCore/bindings/v8/custom/V8Float32ArrayCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8Float32ArrayCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8Float32ArrayCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -57,7 +57,7 @@
 v8::Handle<v8::Value> toV8(Float32Array* impl, v8::Isolate* isolate)
 {
     if (!impl)
-        return v8::Null();
+        return v8::Null(isolate);
     v8::Handle<v8::Object> wrapper = V8Float32Array::wrap(impl, isolate);
     if (!wrapper.IsEmpty())
         wrapper->SetIndexedPropertiesToExternalArrayData(impl->baseAddress(), v8::kExternalFloatArray, impl->length());

Modified: trunk/Source/WebCore/bindings/v8/custom/V8Float64ArrayCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8Float64ArrayCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8Float64ArrayCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -52,7 +52,7 @@
 v8::Handle<v8::Value> toV8(Float64Array* impl, v8::Isolate* isolate)
 {
     if (!impl)
-        return v8::Null();
+        return v8::Null(isolate);
     v8::Handle<v8::Object> wrapper = V8Float64Array::wrap(impl, isolate);
     if (!wrapper.IsEmpty())
         wrapper->SetIndexedPropertiesToExternalArrayData(impl->baseAddress(), v8::kExternalDoubleArray, impl->length());

Modified: trunk/Source/WebCore/bindings/v8/custom/V8HTMLCanvasElementCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8HTMLCanvasElementCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8HTMLCanvasElementCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -84,7 +84,7 @@
 #endif
     CanvasRenderingContext* result = imp->getContext(contextId, attrs.get());
     if (!result)
-        return v8::Null();
+        return v8::Null(args.GetIsolate());
     if (result->is2d())
         return toV8(static_cast<CanvasRenderingContext2D*>(result), args.GetIsolate());
 #if ENABLE(WEBGL)
@@ -92,7 +92,7 @@
         return toV8(static_cast<WebGLRenderingContext*>(result), args.GetIsolate());
 #endif
     ASSERT_NOT_REACHED();
-    return v8::Null();
+    return v8::Null(args.GetIsolate());
 }
 
 v8::Handle<v8::Value> V8HTMLCanvasElement::toDataURLCallback(const v8::Arguments& args)

Modified: trunk/Source/WebCore/bindings/v8/custom/V8HTMLDocumentCustom.cpp (118132 => 118133)


--- trunk/Source/WebCore/bindings/v8/custom/V8HTMLDocumentCustom.cpp	2012-05-23 06:59:28 UTC (rev 118132)
+++ trunk/Source/WebCore/bindings/v8/custom/V8HTMLDocumentCustom.cpp	2012-05-23 07:06:11 UTC (rev 118133)
@@ -187,7 +187,7 @@
 v8::Handle<v8::Value> toV8(HTMLDocument* impl, v8::Isolate* isolate, bool forceNewObject)
 {
     if (!impl)
-        return v8::Null();
+        return v8::Null(isolate);
     v8::Handle<v8::Object> wrapper = V8HTMLDocument::wrap(impl, isolate, forceNewObject);
     if (wrapper.IsEmpty())
         return wrapper;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to