Bug 37577 - Adjust Chromium's bindings to recent changes in WebKit
Summary: Adjust Chromium's bindings to recent changes in WebKit
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-04-14 08:42 PDT by anton muhin
Modified: 2010-04-14 11:21 PDT (History)
1 user (show)

See Also:


Attachments
Plain rebaseling + skipping timing out tests (6.35 KB, patch)
2010-04-14 08:52 PDT, anton muhin
levin: review+
Details | Formatted Diff | Diff
Next iteration (6.35 KB, patch)
2010-04-14 09:05 PDT, anton muhin
no flags Details | Formatted Diff | Diff
Correct next iteration (1.90 KB, patch)
2010-04-14 09:08 PDT, anton muhin
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description anton muhin 2010-04-14 08:42:12 PDT
Adjust Chromium's bindings to recent changes in WebKit
Comment 1 anton muhin 2010-04-14 08:44:31 PDT
After http://trac.webkit.org/changeset/57535/ Chromium's canary bots started to fail for the following tests:

http/tests/xmlhttprequest/origin-whitelisting-all.html
http/tests/xmlhttprequest/origin-whitelisting-exact-match.html
http/tests/xmlhttprequest/origin-whitelisting-https.html
http/tests/xmlhttprequest/origin-whitelisting-ip-addresses-with-
subdomains.html
http/tests/xmlhttprequest/origin-whitelisting-ip-addresses.html
http/tests/xmlhttprequest/origin-whitelisting-subdomains.html
http/tests/security/local-image-from-remote-whitelisted.html
http/tests/xmlhttprequest/origin-whitelisting-removal.html

Two of those are new.

It looks like we need to expose into the bindings new API, see sample failure:

--- platform/chromium-win/http/tests/security/local-image-from-remote-whitelisted-expected.txt  (revision 0)
+++ platform/chromium-win/http/tests/security/local-image-from-remote-whitelisted-expected.txt  (revision 0)
@@ -0,0 +1,4 @@
+CONSOLE MESSAGE: line 8: Uncaught TypeError: Object [object Object] has no method 'addOriginAccessWhitelistEntry'
+This test is to see if a remote file can include a local image when the access has been white listed using addOriginAccessWhitelistEntry.
+
+Test has not run.

Planning to skip the tests for now until proper solution is done.
Comment 2 anton muhin 2010-04-14 08:52:28 PDT
Created attachment 53333 [details]
Plain rebaseling + skipping timing out tests
Comment 3 David Levin 2010-04-14 08:57:57 PDT
Comment on attachment 53333 [details]
Plain rebaseling + skipping timing out tests

Pleas fix the TABs in the ChangeLog.

Also, it looks like the two baselines you are adding are actually failures.

Please consider filing a bug to implement "addOriginAccessWhitelistEntry" and perhaps those tests should be listed as failures in test_expectations instead of having new baselines.
Comment 4 anton muhin 2010-04-14 09:05:51 PDT
Created attachment 53334 [details]
Next iteration

Submitting right now.
Comment 5 anton muhin 2010-04-14 09:08:42 PDT
Created attachment 53335 [details]
Correct next iteration

Sorry, wrong diff
Comment 6 anton muhin 2010-04-14 09:10:33 PDT
(In reply to comment #3)
> (From update of attachment 53333 [details])
> Pleas fix the TABs in the ChangeLog.
> 
> Also, it looks like the two baselines you are adding are actually failures.
> 
> Please consider filing a bug to implement "addOriginAccessWhitelistEntry" and
> perhaps those tests should be listed as failures in test_expectations instead
> of having new baselines.

David, as per your suggestion I've filed https://bugs.webkit.org/show_bug.cgi?id=37578
Comment 7 anton muhin 2010-04-14 11:17:16 PDT
https://bugs.webkit.org/attachment.cgi?id=53335 was landed as http://trac.webkit.org/changeset/57582

Closing the bug.