Unreviewed, Chromium build fix.
authordumi@chromium.org <dumi@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 26 Mar 2010 02:12:21 +0000 (02:12 +0000)
committerdumi@chromium.org <dumi@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 26 Mar 2010 02:12:21 +0000 (02:12 +0000)
WebCore:

V8CustomVoidCallback should not check if a return value is a
boolean. It should only convert the value to a boolean.

* bindings/v8/custom/V8CustomVoidCallback.cpp:
(WebCore::invokeCallback):

LayoutTests:

Update the expectations for statement-error-callback.html for Chromium.

* platform/chromium-mac/storage/statement-error-callback-expected.txt:
* platform/chromium-win/storage/statement-error-callback-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@56588 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/platform/chromium-mac/storage/statement-error-callback-expected.txt
LayoutTests/platform/chromium-win/storage/statement-error-callback-expected.txt
WebCore/ChangeLog
WebCore/bindings/v8/custom/V8CustomVoidCallback.cpp

index 6811726..e7bc078 100644 (file)
@@ -1,3 +1,12 @@
+2010-03-25  Dumitru Daniliuc  <dumi@chromium.org>
+
+        Unreviewed, Chromium build fix.
+
+        Update the expectations for statement-error-callback.html for Chromium.
+
+        * platform/chromium-mac/storage/statement-error-callback-expected.txt:
+        * platform/chromium-win/storage/statement-error-callback-expected.txt:
+
 2010-03-25  Dmitry Titov  <dimich@chromium.org>
 
         Not reviewed, update pixel results for Chromium bots.
index 8d35ed9..4939acd 100644 (file)
@@ -1,4 +1,4 @@
-CONSOLE MESSAGE: line 75: Uncaught Exception in statement error callback
+CONSOLE MESSAGE: line 67: Uncaught Exception in statement error callback
 This test confirms that a transaction is immediately rolled back if and only if a statement's error callback throws an exception, returns true, or doesn't return any value.
 PASS - the transaction error callback was invoked.
 PASS - the transaction error callback was invoked.
@@ -6,6 +6,9 @@ PASS - the transaction error callback was invoked.
 PASS - the transaction error callback was invoked.
 PASS - the transaction error callback was invoked.
 PASS - the transaction error callback was invoked.
-PASS - the transaction error callback was invoked.
+PASS - the transaction success callback was invoked.
+PASS - the transaction success callback was invoked.
+PASS - the transaction success callback was invoked.
+PASS - the transaction success callback was invoked.
 Test Complete
 
index 2ea093a..3925694 100644 (file)
@@ -1,4 +1,4 @@
-CONSOLE MESSAGE: line 75: Uncaught Exception in statement error callback
+CONSOLE MESSAGE: line 67: Uncaught Exception in statement error callback
 This test confirms that a transaction is immediately rolled back if and only if a statement's error callback throws an exception, returns true, or doesn't return any value.
 PASS - the transaction error callback was invoked.
 PASS - the transaction error callback was invoked.
@@ -6,5 +6,8 @@ PASS - the transaction error callback was invoked.
 PASS - the transaction error callback was invoked.
 PASS - the transaction error callback was invoked.
 PASS - the transaction error callback was invoked.
-PASS - the transaction error callback was invoked.
+PASS - the transaction success callback was invoked.
+PASS - the transaction success callback was invoked.
+PASS - the transaction success callback was invoked.
+PASS - the transaction success callback was invoked.
 Test Complete
index a318d51..a62b199 100644 (file)
@@ -1,3 +1,13 @@
+2010-03-25  Dumitru Daniliuc  <dumi@chromium.org>
+
+        Unreviewed, Chromium build fix.
+
+        V8CustomVoidCallback should not check if a return value is a
+        boolean. It should only convert the value to a boolean.
+
+        * bindings/v8/custom/V8CustomVoidCallback.cpp:
+        (WebCore::invokeCallback):
+
 2010-03-25  Dmitry Titov  <dimich@chromium.org>
 
         No review, rolling out r56585.
index 6e258df..f4ea62a 100644 (file)
@@ -87,7 +87,7 @@ bool invokeCallback(v8::Persistent<v8::Object> callback, int argc, v8::Handle<v8
     ASSERT(proxy);
 
     v8::Handle<v8::Value> result = proxy->callFunction(callbackFunction, thisObject, argc, argv);
-    callbackReturnValue = !result.IsEmpty() && result->IsBoolean() && result->BooleanValue();
+    callbackReturnValue = !result.IsEmpty() && result->BooleanValue();
 
     if (exceptionCatcher.HasCaught()) {
         v8::Local<v8::Message> message = exceptionCatcher.Message();