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 6811726453d8026f3927f862fc9a26acf871144a..e7bc078ad5f52d89580c24f245dcabfb3d9df19b 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 8d35ed99e35eed51b78604af36863ad8ff37f366..4939acddaaef47c6432bd56da33448eeccaf7dee 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 2ea093a0dfa585a4e07d111a8ab0b54312a7c739..3925694c75737f505921727f4f7d479bab21c853 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 a318d517349c1c6c494b169ba0f229858fa5b7f3..a62b199d29bf36d2f1629f60915a91617cb130e0 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 6e258dfc83ae66368cd6119d7ee8292c6940cc70..f4ea62ae4b9b59f3446dd9c9e21da6477a9f427e 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();