117764 2013-06-18 17:10:28 -0700 Serializer doesn't handling inconsistent prefixes properly 2013-08-29 06:49:43 -0700 1 1 1 Unclassified WebKit XML 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 alex webkit-unassigned ap rwlbuis oldest_to_newest 901590 0 alex 2013-06-18 17:10:28 -0700 When constructing an XML document programmatically, the namespaces in use and prefixes may be inconsistent. For example, the results of merging cloned nodes from different sources may end up with conflicting prefix to namespace mappings. These conflicts must be detected and the prefixes adjusted. The current implementation allows multiple namespace attributes for the same prefix which results in an XML parsing error for any receiving application. 901841 1 ap 2013-06-19 10:33:15 -0700 Could you please attach a test case fixing which would be the purpose of this bug? We have quite a few bugs about namespace serialization already, and it's difficult to track them without explicitly distinct test cases. 901863 2 205019 alex 2013-06-19 11:11:09 -0700 Created attachment 205019 An example of inconsistent handling of namespaces for attributes Here is the missing attachment that I thought I filed with the original bug. The test does: 1. Creates two attributes with different namespaces but the same prefix. 2. Redefined a common prefix (xlink). The outcome right now is: 1. The "p" prefix is declared twice with to different namespace URIs. 2. A extra xmlns:xlink attribute that maps the the XLink namespace URI is added. Of course, that won't parser properly on the way back in. 905204 3 rwlbuis 2013-07-01 18:49:50 -0700 Bug 19121 has a patch up that fixes this bug too. I incorporated the testcase in it. 922437 4 rwlbuis 2013-08-29 06:49:43 -0700 Committed r154779: <http://trac.webkit.org/changeset/154779>. 205019 2013-06-19 11:11:09 -0700 2013-06-19 11:11:09 -0700 An example of inconsistent handling of namespaces for attributes ns-scope-serialization.xhtml application/xhtml+xml 989 alex PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0iVVRGLTgiPz4KPCFET0NUWVBFIGh0bWw+Cjxo dG1sIHhtbG5zPSJodHRwOi8vd3d3LnczLm9yZy8xOTk5L3hodG1sIj4KICAgIDxoZWFkPgogICAg ICAgIDx0aXRsZT5YTUwgU2VyaWFsaXphdGlvbiBUZXN0PC90aXRsZT4KICAgICAgICA8c2NyaXB0 IHR5cGU9InRleHQvamF2YXNjcmlwdCI+CiAgIHZhciBzZXJpYWxpemVyID0gbmV3IFhNTFNlcmlh bGl6ZXIoKTsKCiAgIHZhciBkb2MgPSBkb2N1bWVudC5pbXBsZW1lbnRhdGlvbi5jcmVhdGVEb2N1 bWVudCgiaHR0cDovL3d3dy5leGFtcGxlLmNvbS9uczEiLCJkb2MiLG51bGwpOwogICBkb2MuZG9j dW1lbnRFbGVtZW50LnNldEF0dHJpYnV0ZU5TKCJodHRwOi8vd3d3LnczLm9yZy8yMDAwL3htbG5z LyIsInhtbG5zOnAiLCJodHRwOi8vd3d3LmV4YW1wbGUuY29tL3AiKTsKICAgdmFyIHRlc3QgPSBk b2MuY3JlYXRlRWxlbWVudE5TKCJodHRwOi8vd3d3LmV4YW1wbGUuY29tL25zMiIsInRlc3QiKTsK ICAgZG9jLmRvY3VtZW50RWxlbWVudC5hcHBlbmRDaGlsZCh0ZXN0KTsKICAgdGVzdC5zZXRBdHRy aWJ1dGVOUygiaHR0cDovL3d3dy5leGFtcGxlLmNvbS9xIiwicDphIiwiSSBhbSBxIik7CiAgIHRl c3Quc2V0QXR0cmlidXRlTlMoImh0dHA6Ly93d3cuZXhhbXBsZS5jb20vcCIsInA6YiIsIkkgYW0g cCIpOwogICB0ZXN0LnNldEF0dHJpYnV0ZU5TKCJodHRwOi8vd3d3LnczLm9yZy8yMDAwL3htbG5z LyIsInhtbG5zOnhsaW5rIiwiaHR0cDovL3d3dy5leGFtcGxlLmNvbS94bGluayIpOwogICB0ZXN0 LnNldEF0dHJpYnV0ZU5TKCJodHRwOi8vd3d3LnczLm9yZy8xOTk5L3hsaW5rIiwieGxpbms6aHJl ZiIsIiNmb28iKTsKICAgdmFyIHhtbCA9IHNlcmlhbGl6ZXIuc2VyaWFsaXplVG9TdHJpbmcoZG9j KTsKICAgY29uc29sZS5sb2coeG1sKTsKICAgCiAgICAgICAgPC9zY3JpcHQ+CiAgICA8L2hlYWQ+ CiAgICA8Ym9keS8+CjwvaHRtbD4=