28744 2009-08-26 10:24:25 -0700 XSLTProcessor transformToFragment mistakenly treats source node as root one 2023-06-02 04:56:28 -0700 1 1 1 Unclassified WebKit XML 528+ (Nightly build) All All NEW https://bugs.webkit.org/show_bug.cgi?id=14101 P2 Normal --- 1 martin.honnen webkit-unassigned ahmad.saleem792 ap bfulgham rniwa oldest_to_newest 142881 0 38618 martin.honnen 2009-08-26 10:24:25 -0700 Created attachment 38618 HTML document with a JavaScript test case using XSLTProcessor transformToFragment WebKit allows XSLT transformations to be run by Javascript code, exposing the same API that Mozilla introduced: https://developer.mozilla.org/index.php?title=en/The_XSLT%2F%2FJavaScript_Interface_in_Gecko/Interface_List There is however a major incompatibility between WebKit and between Mozilla. The first argument to the transformToFragment method takes a DOM node and Mozilla (and Opera) then apply the XSLT transformation with that node as the initial context node while WebKit seems to always apply the XSLT transformation to the ownerDocument node of the DOM node passed to transformToFragment. So with Mozilla the API can be used to apply an XSLT transformation to a certain DOM node (e.g. an element in a larger document) while WebKit always applies the transformation to the complete document. That is an incompatibility. As the API was introduced by Mozilla and then adapted by WebKit I think WebKit should aim to behave like Mozilla. The test case I am going to attach runs a simple XSLT transformation with transformToFragment. The stylesheet has two templates, one matching the document node (match="/") and a second matching a 'foo' element (match="foo"). Then transformToFragment is called with a 'foo' element node in a larger XML document being passed in as the first argument. Mozilla and Opera that way start the transformation by applying the match="foo" template while WebKit starts the transformation by applying the match="/" template leading to a transformation result that differs strongly between Mozilla and WebKit. 143454 1 ap 2009-08-28 14:10:48 -0700 This is indeed a serious bug, but the mechanism is somewhat different than described above. The implementation of transformToFragment takes a subtree with a root at source node, and creates a new document from it. So, the "<xsl:template match="/">" rule matches "<foo>" node. Like bug 14101, this will be quite difficult to fix, because our DOM tree cannot be directly processed by XSL, so we cannot just pass a node from DOM. 1887408 2 ahmad.saleem792 2022-07-28 12:22:55 -0700 I am still able to reproduce this bug in Safari 15.6 on macOS 12.5 using attached test case and it shows "Test failed." and it is same with Chrome Canary 106. Only Firefox Nightly 105 shows "Test succeeded.". Thanks! 1959527 3 ahmad.saleem792 2023-06-02 04:56:28 -0700 *** Chrome Canary 116 & Safari 16.5 / STP171 *** <div xmlns="http://www.w3.org/1999/xhtml"><p>bar</p></div> *** Firefox Nightly 115 *** <p xmlns="http://www.w3.org/1999/xhtml">bar</p> 38618 2009-08-26 10:24:25 -0700 2009-08-26 10:24:25 -0700 HTML document with a JavaScript test case using XSLTProcessor transformToFragment test2009082601.html text/html 1610 martin.honnen PCFET0NUWVBFIEhUTUwgUFVCTElDICItLy9XM0MvL0RURCBIVE1MIDQuMDEvL0VOIg0KICAgICAg ICAiaHR0cDovL3d3dy53My5vcmcvVFIvaHRtbDQvc3RyaWN0LmR0ZCI+DQo8aHRtbCBsYW5nPSJl biI+DQo8aGVhZD4NCjxtZXRhIGh0dHAtZXF1aXY9IkNvbnRlbnQtVHlwZSIgY29udGVudD0idGV4 dC9odG1sOyBjaGFyc2V0PVVURi04Ij4NCjx0aXRsZT50cmFuc2Zvcm1Ub0ZyYWdtZW50IHRlc3Q8 L3RpdGxlPg0KPHNjcmlwdCB0eXBlPSJ0ZXh0L2phdmFzY3JpcHQiPg0KZnVuY3Rpb24gZG9UZXN0 KCkNCnsNCiAgdmFyIGRvYyA9IG5ldyBET01QYXJzZXIoKS5wYXJzZUZyb21TdHJpbmcoWw0KICAn PHJvb3Q+JywNCiAgJzxmb28+YmFyPFwvZm9vPicsDQogICc8XC9yb290PicNCiAgXS5qb2luKCcn KSwgJ2FwcGxpY2F0aW9uL3htbCcpOw0KICB2YXIgc3R5bGVzaGVldCA9IG5ldyBET01QYXJzZXIo KS5wYXJzZUZyb21TdHJpbmcoWw0KICAnPHhzbDpzdHlsZXNoZWV0IHhtbG5zOnhzbD0iaHR0cDov L3d3dy53My5vcmcvMTk5OS9YU0wvVHJhbnNmb3JtIiB2ZXJzaW9uPSIxLjAiPicsDQogICc8eHNs OnRlbXBsYXRlIG1hdGNoPSIvIj4nLA0KICAnPGRpdj4nLA0KICAnPHhzbDphcHBseS10ZW1wbGF0 ZXMvPicsDQogICc8XC9kaXY+JywNCiAgJzxcL3hzbDp0ZW1wbGF0ZT4nLA0KICAnPHhzbDp0ZW1w bGF0ZSBtYXRjaD0iZm9vIj4nLA0KICAnPHA+PHhzbDp2YWx1ZS1vZiBzZWxlY3Q9Ii4iLz48XC9w PicsDQogICc8XC94c2w6dGVtcGxhdGU+JywNCiAgJzxcL3hzbDpzdHlsZXNoZWV0PicNCiAgXS5q b2luKCdcclxuJyksICdhcHBsaWNhdGlvbi94bWwnKTsNCiAgdmFyIHByb2MgPSBuZXcgWFNMVFBy b2Nlc3NvcigpOw0KICBwcm9jLmltcG9ydFN0eWxlc2hlZXQoc3R5bGVzaGVldCk7DQogIHZhciBm cmFnbWVudCA9IHByb2MudHJhbnNmb3JtVG9GcmFnbWVudChkb2MuZG9jdW1lbnRFbGVtZW50LmZp cnN0Q2hpbGQsIGRvY3VtZW50KTsNCiAgdmFyIHByZSA9IGRvY3VtZW50LmNyZWF0ZUVsZW1lbnQo J3ByZScpOw0KICBwcmUuYXBwZW5kQ2hpbGQoZG9jdW1lbnQuY3JlYXRlVGV4dE5vZGUobmV3IFhN TFNlcmlhbGl6ZXIoKS5zZXJpYWxpemVUb1N0cmluZyhmcmFnbWVudCkpKTsNCiAgZG9jdW1lbnQu Z2V0RWxlbWVudEJ5SWQoJ3Jlc3VsdCcpLmFwcGVuZENoaWxkKHByZSk7DQogIHZhciBwID0gZG9j dW1lbnQuY3JlYXRlRWxlbWVudCgncCcpOw0KICBwLmFwcGVuZENoaWxkKGRvY3VtZW50LmNyZWF0 ZVRleHROb2RlKCdUZXN0ICcgKyAoZnJhZ21lbnQuZmlyc3RDaGlsZC5ub2RlVHlwZSA9PT0gMSAm JiBmcmFnbWVudC5maXJzdENoaWxkLnRhZ05hbWUudG9Mb3dlckNhc2UoKSA9PT0gJ3AnID8gJyBz dWNjZWVkZWQuJyA6ICcgZmFpbGVkLicpKSk7DQogIGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCdy ZXN1bHQnKS5hcHBlbmRDaGlsZChwKTsNCn0NCg0Kd2luZG93Lm9ubG9hZCA9IGRvVGVzdDsNCjwv c2NyaXB0Pg0KPC9oZWFkPg0KPGJvZHk+DQo8aDE+dHJhbnNmb3JtVG9GcmFnbWVudCB0ZXN0PC9o MT4NCjxkaXYgaWQ9InJlc3VsdCI+DQo8aDI+VGVzdCByZXN1bHQ8L2gyPg0KPC9kaXY+DQo8L2Jv ZHk+DQo8L2h0bWw+DQo=