Bug 72252

Summary: [Qt] new test http/tests/misc/xslt-bad-import.html is failing
Product: WebKit Reporter: Tamas Czene <tczene>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: gavinp, japhet, kadam, ossy
Priority: P2 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 87008    
Attachments:
Description Flags
Add http/tests/misc/xslt-bad-import.html to Skipped none

Tamas Czene
Reported 2011-11-14 01:43:41 PST
new test added r99982 --- /ramdisk/qt-linux-64-release/build/layout-test-results/http/tests/misc/xslt-bad-import-expected.txt +++ /ramdisk/qt-linux-64-release/build/layout-test-results/http/tests/misc/xslt-bad-import-actual.txt @@ -1,2 +1,4 @@ +CONSOLE MESSAGE: line 2: <html xmlns='http://www.w3.org/1999/xhtml/'><body><p>Running an XSL-T 1.0 stylesheet with a 2.0 processor.</p></body></html> +CONSOLE MESSAGE: line 5: <html xmlns='http://www.w3.org/1999/xhtml/'><body><p>The element with local name <span class='XQuery-keyword'>apply-imports</span> does not exist in XSL-T.</p></body></html> This test passes if it does not crash.
Attachments
Add http/tests/misc/xslt-bad-import.html to Skipped (1.15 KB, patch)
2011-11-14 01:49 PST, Tamas Czene 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Tamas Czene
Comment 1 2011-11-14 01:49:03 PST
Created attachment 114901 [details] Add http/tests/misc/xslt-bad-import.html to Skipped
Csaba Osztrogonác
Comment 2 2011-11-14 01:53:57 PST
Could you guys confirm if this additinal console message on Qt is correct or not?
Gavin Peters
Comment 3 2011-11-14 04:15:23 PST
I find the message a bit inscrutible, and I'm confused. I believe xsl:apply-imports was in xsl-t 1.0: http://www.w3.org/TR/xslt#apply-imports and it's also in 2.0: http://www.w3.org/TR/xslt20/#element-apply-imports Am I missing something?
Csaba Osztrogonác
Comment 4 2011-11-14 08:23:39 PST
I skipped it until fix to make buildbot happy: http://trac.webkit.org/changeset/100144
Ádám Kallai
Comment 5 2012-09-26 06:12:55 PDT
Now this test is working on Qt. I'm going to remove from Qt skipped list and close the bug.
Ádám Kallai
Comment 6 2012-10-04 06:46:36 PDT
I unskipped it: Committed r130385: <http://trac.webkit.org/changeset/130385> Closing bug.
Note You need to log in before you can comment on or make changes to this bug.