Title: [139595] trunk/Source/WebCore
Revision
139595
Author
[email protected]
Date
2013-01-14 01:52:20 -0800 (Mon, 14 Jan 2013)

Log Message

[V8] Make an Isolate parameter mandatory in v8DateOrNull()
https://bugs.webkit.org/show_bug.cgi?id=106765

Reviewed by Adam Barth.

This is one of the efforts to kill all optional Isolate parameters.

No tests. No change in behavior.

* bindings/v8/V8Binding.h:
(WebCore::v8DateOrNull):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (139594 => 139595)


--- trunk/Source/WebCore/ChangeLog	2013-01-14 09:49:56 UTC (rev 139594)
+++ trunk/Source/WebCore/ChangeLog	2013-01-14 09:52:20 UTC (rev 139595)
@@ -1,3 +1,17 @@
+2013-01-14  Kentaro Hara  <[email protected]>
+
+        [V8] Make an Isolate parameter mandatory in v8DateOrNull()
+        https://bugs.webkit.org/show_bug.cgi?id=106765
+
+        Reviewed by Adam Barth.
+
+        This is one of the efforts to kill all optional Isolate parameters.
+
+        No tests. No change in behavior.
+
+        * bindings/v8/V8Binding.h:
+        (WebCore::v8DateOrNull):
+
 2013-01-14  Pablo Flouret  <[email protected]>
 
         Allow nesting of at-rules

Modified: trunk/Source/WebCore/bindings/v8/V8Binding.h (139594 => 139595)


--- trunk/Source/WebCore/bindings/v8/V8Binding.h	2013-01-14 09:49:56 UTC (rev 139594)
+++ trunk/Source/WebCore/bindings/v8/V8Binding.h	2013-01-14 09:52:20 UTC (rev 139595)
@@ -431,8 +431,9 @@
         return (object->IsDate() || object->IsNumber()) ? object->NumberValue() : std::numeric_limits<double>::quiet_NaN();
     }
 
-    inline v8::Handle<v8::Value> v8DateOrNull(double value, v8::Isolate* isolate = 0)
+    inline v8::Handle<v8::Value> v8DateOrNull(double value, v8::Isolate* isolate)
     {
+        ASSERT(isolate);
         return isfinite(value) ? v8::Date::New(value) : v8NullWithCheck(isolate);
     }
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to