RESOLVED DUPLICATE of bug 46951 46952
[Chromium] URL parsing tests fail since r 68854 
https://bugs.webkit.org/show_bug.cgi?id=46952
Summary [Chromium] URL parsing tests fail since r 68854
Hajime Morrita
Reported 2010-09-30 18:44:26 PDT
http://trac.webkit.org/changeset/68854 gave a large change around HTML parsing and test on chromium is broken. I suspect this is due to difference around GURL. but i'm not sure. Here is list of failures: Regressions: Unexpected text diff mismatch : (15) fast/url/segments-from-data-url.html = TEXT fast/url/segments.html = TEXT http/tests/security/javascriptURL/javascriptURL-execution-context-frame-location-htmldom.html = TEXT http/tests/security/javascriptURL/javascriptURL-execution-context-frame-src-getAttribute-value.html = TEXT http/tests/security/javascriptURL/javascriptURL-execution-context-frame-src-htmldom.html = TEXT http/tests/security/javascriptURL/javascriptURL-execution-context-frame-src-setAttribute.html = TEXT http/tests/security/javascriptURL/javascriptURL-execution-context-frame-src-setAttributeNS.html = TEXT http/tests/security/javascriptURL/javascriptURL-execution-context-frame-src-setAttributeNode.html = TEXT http/tests/security/javascriptURL/javascriptURL-execution-context-frame-src-setAttributeNodeNS.html = TEXT http/tests/security/javascriptURL/javascriptURL-execution-context-iframe-src-getAttribute-value.html = TEXT http/tests/security/javascriptURL/javascriptURL-execution-context-iframe-src-htmldom.html = TEXT http/tests/security/javascriptURL/javascriptURL-execution-context-iframe-src-setAttribute.html = TEXT http/tests/security/javascriptURL/javascriptURL-execution-context-iframe-src-setAttributeNS.html = TEXT http/tests/security/javascriptURL/javascriptURL-execution-context-iframe-src-setAttributeNode.html = TEXT http/tests/security/javascriptURL/javascriptURL-execution-context-iframe-src-setAttributeNodeNS.html = TEXT I'll skip these at this time but they need an immediate investigation.
Attachments
Add attachment proposed patch, testcase, etc.
Kent Tamura
Comment 1 2010-09-30 18:58:26 PDT
(In reply to comment #0) > http://trac.webkit.org/changeset/68854 gave a large change around HTML parsing > and test on chromium is broken. > I suspect this is due to difference around GURL. but i'm not sure. > I'll skip these at this time but they need an immediate investigation. Do not skip them. You should add expectations with =TEXT in such case. If the current result diffs in Chromium are the same as the expectation changes in r68854, we should just do rebaseline.
Hajime Morrita
Comment 2 2010-09-30 19:04:26 PDT
>Do not skip them. You should add expectations with =TEXT in such case. >If the current result diffs in Chromium are the same as the expectation changes in r68854, we should just do rebaseline. Thank you for pointing this out. They got a care anyway: http://trac.webkit.org/changeset/68859 https://bugs.webkit.org/show_bug.cgi?id=46951
Hajime Morrita
Comment 3 2010-09-30 19:04:51 PDT
*** This bug has been marked as a duplicate of bug 46951 ***
Note You need to log in before you can comment on or make changes to this bug.