Bug 37697

Summary: [qt] fast/url/host.html is failing
Product: WebKit Reporter: Tony Chang <tony>
Component: WebCore Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED INVALID    
Severity: Normal CC: abarth, jorlow, laszlo.gombos, ossy, tonikitoo
Priority: P2 Keywords: LayoutTestFailure, Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: PC   
OS: All   
Bug Depends on:    
Bug Blocks: 37641    
Attachments:
Description Flags
skip the test none

Description Tony Chang 2010-04-15 22:34:24 PDT
The test was adding in bug 37666 and it's failing on the QT bot. I'm going to skip the test for now.


Here's the diff output:
--- /home/webkitbuildbot/release32bit/buildslave/qt-linux-release/build/layout-test-results/fast/url/host-expected.txt	2010-04-15 19:57:06.533347063 -0700
+++ /home/webkitbuildbot/release32bit/buildslave/qt-linux-release/build/layout-test-results/fast/url/host-actual.txt	2010-04-15 19:57:06.533347063 -0700
@@ -5,18 +5,18 @@
 
 FAIL canonicalize('http://GoOgLe.CoM/') should be http://google.com/. Was http://GoOgLe.CoM/.
 FAIL canonicalize('http://Goo%20 goo%7C|.com/') should be http://goo%20%20goo%7C%7C.com/. Was http://Goo%20 goo%7C|.com/.
-FAIL canonicalize('http://GOO  goo.com/') should be http://goo%20%20goo.com/. Was http://goo  goo.com/.
+FAIL canonicalize('http://GOO  goo.com/') should be http://goo%20%20goo.com/. Was http:/.
 PASS canonicalize('http://GOO​â goo.com/') is 'http://googoo.com/'
 PASS canonicalize('http://www.foo。bar.com/') is 'http://www.foo.bar.com/'
-FAIL canonicalize('http://ï·zyx.com/') should be http://%EF%BF%BDzyx.com/. Was http:/.
+FAIL canonicalize('http://ï·zyx.com/') should be http://%EF%BF%BDzyx.com/. Was http://zyx.com/.
 FAIL canonicalize('http://%ef%b7%90zyx.com/') should be http://%EF%BF%BDzyx.com/. Was http://%ef%b7%90zyx.com/.
 PASS canonicalize('http://Gï½.com/') is 'http://go.com/'
-FAIL canonicalize('http://%41.com/') should be http://a.com/. Was http://%41.com/.
+FAIL canonicalize('http://%41.com/') should be http://a.com/. Was http:/.
 FAIL canonicalize('http://%ef%bc%85%ef%bc%94%ef%bc%91.com/') should be http://a.com/. Was http://%ef%bc%85%ef%bc%94%ef%bc%91.com/.
-PASS canonicalize('http://ï¼…ï¼ï¼.com/') is 'http://%00.com/'
+FAIL canonicalize('http://ï¼…ï¼ï¼.com/') should be http://%00.com/. Was http:/.
 FAIL canonicalize('http://%ef%bc%85%ef%bc%90%ef%bc%90.com/') should be http://%00.com/. Was http://%ef%bc%85%ef%bc%90%ef%bc%90.com/.
 PASS canonicalize('http://ä½ å¥½ä½ å¥½/') is 'http://xn--6qqa088eba/'
-FAIL canonicalize('http://%E4%BD%A0%E5%A5%BDä½ å¥½/') should be http://xn--6qqa088eba/. Was http://xn--%e4%bd%a0%e5%a5%bd-5g72ap39l/.
+FAIL canonicalize('http://%E4%BD%A0%E5%A5%BDä½ å¥½/') should be http://xn--6qqa088eba/. Was http:/.
 FAIL canonicalize('http://%zz%66%a/') should be http://%25zzf%25a/. Was http://%zz%66%a/.
 PASS canonicalize('http://%25/') is 'http://%25/'
 PASS canonicalize('http://hello%00/') is 'http://hello%00/'
@@ -24,10 +24,10 @@
 FAIL canonicalize('http://%30%78%63%30%2e%30%32%35%30.01%2e/') should be http://192.168.0.1/. Was http://%30%78%63%30%2e%30%32%35%30.01%2e/.
 FAIL canonicalize('http://%3g%78%63%30%2e%30%32%35%30%2E.01/') should be http://%253gxc0.0250..01/. Was http://%3g%78%63%30%2e%30%32%35%30%2E.01/.
 FAIL canonicalize('http://192.168.0.1 hello/') should be http://192.168.0.1%20hello/. Was http://192.168.0.1 hello/.
-FAIL canonicalize('http://ï¼ï¼¸ï½ƒï¼ï¼Žï¼ï¼’5ï¼ï¼Žï¼ï¼‘/') should be http://192.168.0.1/. Was http://0xc0.0250.01/.
+FAIL canonicalize('http://ï¼ï¼¸ï½ƒï¼ï¼Žï¼ï¼’5ï¼ï¼Žï¼ï¼‘/') should be http://192.168.0.1/. Was http://0xc0.0xc00250.0xc0025001/.
 PASS canonicalize('http://192.168.0.257/') is 'http://192.168.0.257/'
 PASS canonicalize('http://[google.com]/') is 'http://[google.com]/'
-FAIL canonicalize('http://Ñ‚(/') should be http://xn--%28-7ed/. Was http://xn--(-8tb/.
+FAIL canonicalize('http://Ñ‚(/') should be http://xn--%28-7ed/. Was http:/.
 PASS successfullyParsed is true
 
 TEST COMPLETE
Comment 1 Tony Chang 2010-04-15 22:36:11 PDT
Created attachment 53514 [details]
skip the test
Comment 2 Tony Chang 2010-04-15 22:37:03 PDT
Committed r57713: <http://trac.webkit.org/changeset/57713>
Comment 3 Csaba Osztrogonác 2010-04-16 08:45:10 PDT
pretty diff URL:
http://build.webkit.org/results/Qt%20Linux%20Release/r57711%20%2810196%29/fast/url/host-pretty-diff.html

It is a feauture/bug of QURL() implementation which can be found in Qt source.
As for me, we can ignore expected fails. I don't think if Qt should
fail same as the expected fail.

There is only one test case where Qt fails instead of pass:
PASS canonicalize('http://ï¼…ï¼ï¼.com/') is 'http://%00.com/'
FAIL canonicalize('http://ï¼…ï¼ï¼.com/') should be http://%00.com/. Was http:/.
Comment 4 Adam Barth 2010-04-16 09:13:18 PDT
> It is a feauture/bug of QURL() implementation which can be found in Qt source.
> As for me, we can ignore expected fails. I don't think if Qt should
> fail same as the expected fail.

Ok.  We're writing these tests as part of a project to clean up URL parsing.  At some point, I'll produce a matrix of what all the other browsers do.  It might make sense to decide how Qt should behave once we see that matrix.
Comment 5 Jocelyn Turcotte 2014-02-03 03:16:22 PST
=== Bulk closing of Qt bugs ===

If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it and remove [Qt] from the summary.

If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines.