<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"

"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">

<html xmlns="http://www.w3.org/1999/xhtml">

<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />

<title>[179497] trunk</title>

</head>

<body>

<style type="text/css"><!--

#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }

#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }

#msg dt:after { content:':';}

#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt;  }

#msg dl a { font-weight: bold}

#msg dl a:link    { color:#fc3; }

#msg dl a:active  { color:#ff0; }

#msg dl a:visited { color:#cc6; }

h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }

#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }

#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }

#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }

#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }

#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }

#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }

#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }

#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }

#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }

#logmsg pre { background: #eee; padding: 1em; }

#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}

#logmsg dl { margin: 0; }

#logmsg dt { font-weight: bold; }

#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }

#logmsg dd:before { content:'\00bb';}

#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }

#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }

#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }

#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }

#logmsg table th.Corner { text-align: left; }

#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }

#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }

#patch { width: 100%; }

#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}

#patch .propset h4, #patch .binary h4 {margin:0;}

#patch pre {padding:0;line-height:1.2em;margin:0;}

#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}

#patch .propset .diff, #patch .binary .diff  {padding:10px 0;}

#patch span {display:block;padding:0 10px;}

#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}

#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}

#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}

#patch .lines, .info {color:#888;background:#fff;}

--></style>

<div id="msg">

<dl class="meta">

<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/179497">179497</a></dd>

<dt>Author</dt> <dd>benjamin@webkit.org</dd>

<dt>Date</dt> <dd>2015-02-02 14:00:50 -0800 (Mon, 02 Feb 2015)</dd>

</dl>

<h3>Log Message</h3>

<pre>Clean up attribute handling: part 2 - attributeNode

https://bugs.webkit.org/show_bug.cgi?id=141109

Patch by Benjamin Poulain &lt;bpoulain@apple.com&gt; on 2015-02-02

Reviewed by Andreas Kling.

Source/WebCore:

Our implementation was covering some old legacy behaviors of Firefox,

even copying bugs in some cases.

The spec (https://dom.spec.whatwg.org) now defines the behavior precisely,

let's move a bit closer to that.

Tests: fast/dom/Element/attribute-ascii-case-insensitive-3.html

       fast/dom/Element/attribute-setAttributeNode-multiple-times.html

       fast/dom/Element/attribute-setAttributeNodeNS-multiple-times.html

       fast/dom/Element/mozilla-dom-base-tests/test_bug1075702.html

       fast/dom/Element/mozilla-dom-base-tests/test_bug339494.html

       fast/dom/Element/mozilla-dom-base-tests/test_bug364092.xhtml

       fast/dom/Element/setAttributeNode-overriding-lowercase-values.html

* dom/Element.cpp:

(WebCore::findAttrNodeInList):

New getter for the name-without-namespace case.

(WebCore::Element::setAttributeNode):

This one is the tricky one: https://dom.spec.whatwg.org/#dom-element-setattributenode

When setAttributeNode() is used with an AttributeNode without namespace,

getting the old value behaves like getAttribute(), with ASCII lowercase name matching.

When used with a namespace, getting the old value behaves like getAttributeNS().

Setting the value is a whole different story, the name used always keeps

the original case.

Now that's a bit tricky for us because AttributeNodes are just legacy stuff we don't

used internally.

We have 4 cases to handle:

1) The name being set is lowercase, there was no conflicting name on the element.

   That's easy, we just override any node that would exist, set the name otherwise.

2) The name is lowercase but there was an existing attribute for it.

   -We create a new AttributeNode for the name to represent the old name.

   -We check the names are the same with attribute.name().matches(attrNode-&gt;qualifiedName())

    and override the value.

3) The name has uppercase characters, there is no conflicting name.

   We would not find an element to remove, we just use setAttributeInternal() as usual

   to add the attribute;

4) The name has uppercase characters, there is a lowercase conflicing name.

   This is the weird behavior: we need to nuke the old attribute, then add the new attribute

   with a different case.

   First we remove the attribute with a lowercase name with removeAttributeInternal().

   That becomes the old node.

   There might still be an element of the same name as what we are trying to add. We don't want

   to add another version of the same attribute. We need to use findAttributeIndexByName() again

   to find if there is a conflicting attribute. Then we call setAttributeInternal() which handle

   the both the cases where there was an element or not.

(WebCore::Element::setAttributeNodeNS):

This should work like any &quot;NS&quot; method.

(WebCore::Element::removeAttributeNode):

The method removeAttributeNode() is supposed to be exact.

(WebCore::Element::getAttributeNode):

(WebCore::Element::hasAttribute):

(WebCore::Element::attrIfExists):

* dom/Element.h:

* dom/ElementData.cpp:

(WebCore::ElementData::findAttributeIndexByNameSlowCase): Deleted.

(WebCore::ElementData::findAttributeIndexByNameForAttributeNode): Deleted.

Kill the slow case, every caller has been updated now.

* dom/ElementData.h:

(WebCore::ElementData::findAttributeIndexByName):

* dom/QualifiedName.h:

(WebCore::QualifiedName::matchesIgnoringCaseForLocalName): Deleted.

LayoutTests:

Improve the coverage a little.

Not everything is right yet: some getters return an empty string when they

should return null.

* fast/dom/Element/attribute-ascii-case-insensitive-1-expected.txt:

This is now fixed :)

* fast/dom/Element/attribute-ascii-case-insensitive-3-expected.txt: Added.

* fast/dom/Element/attribute-ascii-case-insensitive-3.html: Added.

Test prefixed-like attribute defined through the parser.

* fast/dom/Element/attribute-setAttributeNode-multiple-times-expected.txt: Added.

* fast/dom/Element/attribute-setAttributeNode-multiple-times.html: Added.

Make sure we don't accumulate nodes.

* fast/dom/Element/attribute-setAttributeNodeNS-multiple-times-expected.txt: Added.

* fast/dom/Element/attribute-setAttributeNodeNS-multiple-times.html: Added.

Same without the crazy setter.

* fast/dom/Element/script-tests/getAttribute-check-case-sensitivity.js:

* fast/dom/Element/getAttribute-check-case-sensitivity-expected.txt:

With the latest spec, getting a node with any uppercase character through

getAttributeNode() always fails. Update the test to use .getAttributeNodeNS()

were needed.

* fast/dom/Element/mozilla-dom-base-tests/test_bug1075702-expected.txt: Added.

* fast/dom/Element/mozilla-dom-base-tests/test_bug1075702.html: Added.

* fast/dom/Element/mozilla-dom-base-tests/test_bug339494-expected.txt: Added.

* fast/dom/Element/mozilla-dom-base-tests/test_bug339494.html: Added.

* fast/dom/Element/mozilla-dom-base-tests/test_bug364092-expected.txt: Added.

* fast/dom/Element/mozilla-dom-base-tests/test_bug364092.xhtml: Added.

(testGetAttributeNodeMixedCase):

(testAttribNodeNamePreservesCaseGetNode):

(testAttribNodeNamePreservesCaseGetNode2):

Some related tests from Gecko, for completeness.

* fast/dom/Element/setAttributeNode-case-insensitivity-expected.txt:

* fast/dom/Element/setAttributeNode-case-insensitivity.html:

Test that the getAttribute part of setAttributeNode() do not ignore the prefix. The spec

says to use the name, not the localname.

* fast/dom/Element/setAttributeNode-for-existing-attribute-expected.txt:

* fast/dom/Element/setAttributeNode-for-existing-attribute.html:

This test was for legacy behavior that came from Firefox. Firefox does not do that anymore.

Keep the test around for regression catching, but add a sentence explaining the 'incorrect'

behavior.

* fast/dom/Element/setAttributeNode-overriding-lowercase-values-1-expected.txt: Added.

* fast/dom/Element/setAttributeNode-overriding-lowercase-values-1.html: Added.

* fast/dom/Element/setAttributeNode-overriding-lowercase-values-2-expected.txt: Added.

* fast/dom/Element/setAttributeNode-overriding-lowercase-values-2.html: Added.

Some coverage for the name overriding craziness.</pre>

<h3>Modified Paths</h3>

<ul>

<li><a href="#trunkLayoutTestsChangeLog">trunk/LayoutTests/ChangeLog</a></li>

<li><a href="#trunkLayoutTestsfastdomElementattributeasciicaseinsensitive1expectedtxt">trunk/LayoutTests/fast/dom/Element/attribute-ascii-case-insensitive-1-expected.txt</a></li>

<li><a href="#trunkLayoutTestsfastdomElementgetAttributecheckcasesensitivityexpectedtxt">trunk/LayoutTests/fast/dom/Element/getAttribute-check-case-sensitivity-expected.txt</a></li>

<li><a href="#trunkLayoutTestsfastdomElementscripttestsgetAttributecheckcasesensitivityjs">trunk/LayoutTests/fast/dom/Element/script-tests/getAttribute-check-case-sensitivity.js</a></li>

<li><a href="#trunkLayoutTestsfastdomElementsetAttributeNodecaseinsensitivityexpectedtxt">trunk/LayoutTests/fast/dom/Element/setAttributeNode-case-insensitivity-expected.txt</a></li>

<li><a href="#trunkLayoutTestsfastdomElementsetAttributeNodecaseinsensitivityhtml">trunk/LayoutTests/fast/dom/Element/setAttributeNode-case-insensitivity.html</a></li>

<li><a href="#trunkLayoutTestsfastdomElementsetAttributeNodeforexistingattributeexpectedtxt">trunk/LayoutTests/fast/dom/Element/setAttributeNode-for-existing-attribute-expected.txt</a></li>

<li><a href="#trunkLayoutTestsfastdomElementsetAttributeNodeforexistingattributehtml">trunk/LayoutTests/fast/dom/Element/setAttributeNode-for-existing-attribute.html</a></li>

<li><a href="#trunkSourceWebCoreChangeLog">trunk/Source/WebCore/ChangeLog</a></li>

<li><a href="#trunkSourceWebCoredomElementcpp">trunk/Source/WebCore/dom/Element.cpp</a></li>

<li><a href="#trunkSourceWebCoredomElementh">trunk/Source/WebCore/dom/Element.h</a></li>

<li><a href="#trunkSourceWebCoredomElementDatacpp">trunk/Source/WebCore/dom/ElementData.cpp</a></li>

<li><a href="#trunkSourceWebCoredomElementDatah">trunk/Source/WebCore/dom/ElementData.h</a></li>

<li><a href="#trunkSourceWebCoredomQualifiedNameh">trunk/Source/WebCore/dom/QualifiedName.h</a></li>

</ul>

<h3>Added Paths</h3>

<ul>

<li><a href="#trunkLayoutTestsfastdomElementattributeasciicaseinsensitive3expectedtxt">trunk/LayoutTests/fast/dom/Element/attribute-ascii-case-insensitive-3-expected.txt</a></li>

<li><a href="#trunkLayoutTestsfastdomElementattributeasciicaseinsensitive3html">trunk/LayoutTests/fast/dom/Element/attribute-ascii-case-insensitive-3.html</a></li>

<li><a href="#trunkLayoutTestsfastdomElementattributesetAttributeNodemultipletimesexpectedtxt">trunk/LayoutTests/fast/dom/Element/attribute-setAttributeNode-multiple-times-expected.txt</a></li>

<li><a href="#trunkLayoutTestsfastdomElementattributesetAttributeNodemultipletimeshtml">trunk/LayoutTests/fast/dom/Element/attribute-setAttributeNode-multiple-times.html</a></li>

<li><a href="#trunkLayoutTestsfastdomElementattributesetAttributeNodeNSmultipletimesexpectedtxt">trunk/LayoutTests/fast/dom/Element/attribute-setAttributeNodeNS-multiple-times-expected.txt</a></li>

<li><a href="#trunkLayoutTestsfastdomElementattributesetAttributeNodeNSmultipletimeshtml">trunk/LayoutTests/fast/dom/Element/attribute-setAttributeNodeNS-multiple-times.html</a></li>

<li>trunk/LayoutTests/fast/dom/Element/mozilla-dom-base-tests/</li>

<li><a href="#trunkLayoutTestsfastdomElementmozilladombaseteststest_bug1075702expectedtxt">trunk/LayoutTests/fast/dom/Element/mozilla-dom-base-tests/test_bug1075702-expected.txt</a></li>

<li><a href="#trunkLayoutTestsfastdomElementmozilladombaseteststest_bug1075702html">trunk/LayoutTests/fast/dom/Element/mozilla-dom-base-tests/test_bug1075702.html</a></li>

<li><a href="#trunkLayoutTestsfastdomElementmozilladombaseteststest_bug339494expectedtxt">trunk/LayoutTests/fast/dom/Element/mozilla-dom-base-tests/test_bug339494-expected.txt</a></li>

<li><a href="#trunkLayoutTestsfastdomElementmozilladombaseteststest_bug339494html">trunk/LayoutTests/fast/dom/Element/mozilla-dom-base-tests/test_bug339494.html</a></li>

<li><a href="#trunkLayoutTestsfastdomElementmozilladombaseteststest_bug364092expectedtxt">trunk/LayoutTests/fast/dom/Element/mozilla-dom-base-tests/test_bug364092-expected.txt</a></li>

<li><a href="#trunkLayoutTestsfastdomElementmozilladombaseteststest_bug364092xhtml">trunk/LayoutTests/fast/dom/Element/mozilla-dom-base-tests/test_bug364092.xhtml</a></li>

<li><a href="#trunkLayoutTestsfastdomElementsetAttributeNodeoverridinglowercasevalues1expectedtxt">trunk/LayoutTests/fast/dom/Element/setAttributeNode-overriding-lowercase-values-1-expected.txt</a></li>

<li><a href="#trunkLayoutTestsfastdomElementsetAttributeNodeoverridinglowercasevalues1html">trunk/LayoutTests/fast/dom/Element/setAttributeNode-overriding-lowercase-values-1.html</a></li>

<li><a href="#trunkLayoutTestsfastdomElementsetAttributeNodeoverridinglowercasevalues2expectedtxt">trunk/LayoutTests/fast/dom/Element/setAttributeNode-overriding-lowercase-values-2-expected.txt</a></li>

<li><a href="#trunkLayoutTestsfastdomElementsetAttributeNodeoverridinglowercasevalues2html">trunk/LayoutTests/fast/dom/Element/setAttributeNode-overriding-lowercase-values-2.html</a></li>

</ul>

</div>

<div id="patch">

<h3>Diff</h3>

<a id="trunkLayoutTestsChangeLog"></a>

<div class="modfile"><h4>Modified: trunk/LayoutTests/ChangeLog (179496 => 179497)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/LayoutTests/ChangeLog        2015-02-02 21:55:11 UTC (rev 179496)

+++ trunk/LayoutTests/ChangeLog        2015-02-02 22:00:50 UTC (rev 179497)

</span><span class="lines">@@ -1,3 +1,64 @@

</span><ins>+2015-02-02  Benjamin Poulain  &lt;bpoulain@apple.com&gt;

+

+        Clean up attribute handling: part 2 - attributeNode

+        https://bugs.webkit.org/show_bug.cgi?id=141109

+

+        Reviewed by Andreas Kling.

+

+        Improve the coverage a little.

+

+        Not everything is right yet: some getters return an empty string when they

+        should return null.

+

+        * fast/dom/Element/attribute-ascii-case-insensitive-1-expected.txt:

+        This is now fixed :)

+

+        * fast/dom/Element/attribute-ascii-case-insensitive-3-expected.txt: Added.

+        * fast/dom/Element/attribute-ascii-case-insensitive-3.html: Added.

+        Test prefixed-like attribute defined through the parser.

+

+        * fast/dom/Element/attribute-setAttributeNode-multiple-times-expected.txt: Added.

+        * fast/dom/Element/attribute-setAttributeNode-multiple-times.html: Added.

+        Make sure we don't accumulate nodes.

+

+        * fast/dom/Element/attribute-setAttributeNodeNS-multiple-times-expected.txt: Added.

+        * fast/dom/Element/attribute-setAttributeNodeNS-multiple-times.html: Added.

+        Same without the crazy setter.

+

+        * fast/dom/Element/script-tests/getAttribute-check-case-sensitivity.js:

+        * fast/dom/Element/getAttribute-check-case-sensitivity-expected.txt:

+        With the latest spec, getting a node with any uppercase character through

+        getAttributeNode() always fails. Update the test to use .getAttributeNodeNS()

+        were needed.

+

+        * fast/dom/Element/mozilla-dom-base-tests/test_bug1075702-expected.txt: Added.

+        * fast/dom/Element/mozilla-dom-base-tests/test_bug1075702.html: Added.

+        * fast/dom/Element/mozilla-dom-base-tests/test_bug339494-expected.txt: Added.

+        * fast/dom/Element/mozilla-dom-base-tests/test_bug339494.html: Added.

+        * fast/dom/Element/mozilla-dom-base-tests/test_bug364092-expected.txt: Added.

+        * fast/dom/Element/mozilla-dom-base-tests/test_bug364092.xhtml: Added.

+        (testGetAttributeNodeMixedCase):

+        (testAttribNodeNamePreservesCaseGetNode):

+        (testAttribNodeNamePreservesCaseGetNode2):

+        Some related tests from Gecko, for completeness.

+

+        * fast/dom/Element/setAttributeNode-case-insensitivity-expected.txt:

+        * fast/dom/Element/setAttributeNode-case-insensitivity.html:

+        Test that the getAttribute part of setAttributeNode() do not ignore the prefix. The spec

+        says to use the name, not the localname.

+

+        * fast/dom/Element/setAttributeNode-for-existing-attribute-expected.txt:

+        * fast/dom/Element/setAttributeNode-for-existing-attribute.html:

+        This test was for legacy behavior that came from Firefox. Firefox does not do that anymore.

+        Keep the test around for regression catching, but add a sentence explaining the 'incorrect'

+        behavior.

+

+        * fast/dom/Element/setAttributeNode-overriding-lowercase-values-1-expected.txt: Added.

+        * fast/dom/Element/setAttributeNode-overriding-lowercase-values-1.html: Added.

+        * fast/dom/Element/setAttributeNode-overriding-lowercase-values-2-expected.txt: Added.

+        * fast/dom/Element/setAttributeNode-overriding-lowercase-values-2.html: Added.

+        Some coverage for the name overriding craziness.

+

</ins><span class="cx"> 2015-02-02  Joseph Pecoraro  &lt;pecoraro@apple.com&gt;

</span><span class="cx"> 

<span class="cx">         Rebaseline test with new expected results.

</span></span></pre></div>

<a id="trunkLayoutTestsfastdomElementattributeasciicaseinsensitive1expectedtxt"></a>

<div class="modfile"><h4>Modified: trunk/LayoutTests/fast/dom/Element/attribute-ascii-case-insensitive-1-expected.txt (179496 => 179497)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/LayoutTests/fast/dom/Element/attribute-ascii-case-insensitive-1-expected.txt        2015-02-02 21:55:11 UTC (rev 179496)

+++ trunk/LayoutTests/fast/dom/Element/attribute-ascii-case-insensitive-1-expected.txt        2015-02-02 22:00:50 UTC (rev 179497)

</span><span class="lines">@@ -11,9 +11,9 @@

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;Data-æøå&quot;) is true

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;Data-æøå&quot;) is &quot;WebKit!&quot;

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;data-ÆØÅ&quot;) is false

<del>-FAIL document.getElementById(&quot;target&quot;).getAttribute(&quot;data-ÆØÅ&quot;) should be null (of type object). Was WebKit! (of type string).

</del><ins>+PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;data-ÆØÅ&quot;) is null

</ins><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;data-Æøå&quot;) is false

<del>-FAIL document.getElementById(&quot;target&quot;).getAttribute(&quot;data-Æøå&quot;) should be null (of type object). Was WebKit! (of type string).

</del><ins>+PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;data-Æøå&quot;) is null

</ins><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;data-æøå&quot;) is true

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;data-æøå&quot;) is &quot;WebKit!&quot;

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;DATA-æøå&quot;) is true

</span><span class="lines">@@ -27,9 +27,9 @@

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;Data-æøå&quot;) is true

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;Data-æøå&quot;) is &quot;WebKit!&quot;

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;data-ÆØÅ&quot;) is false

<del>-FAIL document.getElementById(&quot;target&quot;).getAttribute(&quot;data-ÆØÅ&quot;) should be null (of type object). Was WebKit! (of type string).

</del><ins>+PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;data-ÆØÅ&quot;) is null

</ins><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;data-Æøå&quot;) is false

<del>-FAIL document.getElementById(&quot;target&quot;).getAttribute(&quot;data-Æøå&quot;) should be null (of type object). Was WebKit! (of type string).

</del><ins>+PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;data-Æøå&quot;) is null

</ins><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;data-æøå&quot;) is true

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;data-æøå&quot;) is &quot;WebKit!&quot;

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;DATA-æøå&quot;) is true

</span><span class="lines">@@ -45,9 +45,9 @@

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;Data-æøå&quot;) is true

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;Data-æøå&quot;) is &quot;WebKit!&quot;

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;data-ÆØÅ&quot;) is false

<del>-FAIL document.getElementById(&quot;target&quot;).getAttribute(&quot;data-ÆØÅ&quot;) should be null (of type object). Was WebKit! (of type string).

</del><ins>+PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;data-ÆØÅ&quot;) is null

</ins><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;data-Æøå&quot;) is false

<del>-FAIL document.getElementById(&quot;target&quot;).getAttribute(&quot;data-Æøå&quot;) should be null (of type object). Was WebKit! (of type string).

</del><ins>+PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;data-Æøå&quot;) is null

</ins><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;data-æøå&quot;) is true

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;data-æøå&quot;) is &quot;WebKit!&quot;

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;DATA-æøå&quot;) is true

</span><span class="lines">@@ -61,9 +61,9 @@

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;Data-æøå&quot;) is true

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;Data-æøå&quot;) is &quot;WebKit!&quot;

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;data-ÆØÅ&quot;) is false

<del>-FAIL document.getElementById(&quot;target&quot;).getAttribute(&quot;data-ÆØÅ&quot;) should be null (of type object). Was WebKit! (of type string).

</del><ins>+PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;data-ÆØÅ&quot;) is null

</ins><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;data-Æøå&quot;) is false

<del>-FAIL document.getElementById(&quot;target&quot;).getAttribute(&quot;data-Æøå&quot;) should be null (of type object). Was WebKit! (of type string).

</del><ins>+PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;data-Æøå&quot;) is null

</ins><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;data-æøå&quot;) is true

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;data-æøå&quot;) is &quot;WebKit!&quot;

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;DATA-æøå&quot;) is true

</span><span class="lines">@@ -79,9 +79,9 @@

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;Data-æøå&quot;) is true

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;Data-æøå&quot;) is &quot;WebKit!&quot;

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;data-ÆØÅ&quot;) is false

<del>-FAIL document.getElementById(&quot;target&quot;).getAttribute(&quot;data-ÆØÅ&quot;) should be null (of type object). Was WebKit! (of type string).

</del><ins>+PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;data-ÆØÅ&quot;) is null

</ins><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;data-Æøå&quot;) is false

<del>-FAIL document.getElementById(&quot;target&quot;).getAttribute(&quot;data-Æøå&quot;) should be null (of type object). Was WebKit! (of type string).

</del><ins>+PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;data-Æøå&quot;) is null

</ins><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;data-æøå&quot;) is true

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;data-æøå&quot;) is &quot;WebKit!&quot;

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;DATA-æøå&quot;) is true

</span><span class="lines">@@ -95,9 +95,9 @@

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;Data-æøå&quot;) is true

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;Data-æøå&quot;) is &quot;WebKit!&quot;

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;data-ÆØÅ&quot;) is false

<del>-FAIL document.getElementById(&quot;target&quot;).getAttribute(&quot;data-ÆØÅ&quot;) should be null (of type object). Was WebKit! (of type string).

</del><ins>+PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;data-ÆØÅ&quot;) is null

</ins><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;data-Æøå&quot;) is false

<del>-FAIL document.getElementById(&quot;target&quot;).getAttribute(&quot;data-Æøå&quot;) should be null (of type object). Was WebKit! (of type string).

</del><ins>+PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;data-Æøå&quot;) is null

</ins><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;data-æøå&quot;) is true

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;data-æøå&quot;) is &quot;WebKit!&quot;

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;DATA-æøå&quot;) is true

</span><span class="lines">@@ -113,9 +113,9 @@

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;Data-Æøå&quot;) is true

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;Data-Æøå&quot;) is &quot;WebKit!&quot;

</span><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;data-ÆØÅ&quot;) is false

<del>-FAIL document.getElementById(&quot;target&quot;).getAttribute(&quot;data-ÆØÅ&quot;) should be null (of type object). Was WebKit! (of type string).

</del><ins>+PASS document.getElementById(&quot;target&quot;).getAttribute(&quot;data-ÆØÅ&quot;) is null

</ins><span class="cx"> PASS document.getElementById(&quot;target&quot;).hasAttribute(&quot;data-æøå&quot;) is false