<!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>[182677] 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/182677">182677</a></dd>
<dt>Author</dt> <dd>benjamin@webkit.org</dd>
<dt>Date</dt> <dd>2015-04-12 21:42:53 -0700 (Sun, 12 Apr 2015)</dd>
</dl>
<h3>Log Message</h3>
<pre>Legacy scroll behavior on HTMLBodyElement should only apply to the first body element of a document
https://bugs.webkit.org/show_bug.cgi?id=143651
Source/WebCore:
Reviewed by Sam Weinig.
WebKit has some very weird behaviors for the scroll methods on body. This patch
address the first bug: only the first body element should have the legacy behavior.
The relevant text in spec:
-http://dev.w3.org/csswg/cssom-view/#dom-element-scrollleft
-http://dev.w3.org/csswg/cssom-view/#dom-element-scrolltop
-http://dev.w3.org/csswg/cssom-view/#dom-element-scrollwidth
-http://dev.w3.org/csswg/cssom-view/#dom-element-scrollheight
Tests: fast/dom/Element/body-scrollHeight-basics-quirks.html
fast/dom/Element/body-scrollLeft-basics-quirks.html
fast/dom/Element/body-scrollTop-basics-quirks.html
fast/dom/Element/body-scrollWidth-basics-quirks.html
fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollLeft.html
fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollTop.html
* html/HTMLBodyElement.cpp:
(WebCore::HTMLBodyElement::isFirstBodyElementOfDocument):
(WebCore::HTMLBodyElement::scrollLeft):
(WebCore::HTMLBodyElement::setScrollLeft):
(WebCore::HTMLBodyElement::scrollTop):
(WebCore::HTMLBodyElement::setScrollTop):
(WebCore::HTMLBodyElement::scrollHeight):
(WebCore::HTMLBodyElement::scrollWidth):
* html/HTMLBodyElement.h:
LayoutTests:
Patch by Benjamin Poulain <benjamin@webkit.org> and Diego Perini <diego.perini@gmail.com> on 2015-04-12
Reviewed by Sam Weinig.
* fast/dom/Element/body-scrollHeight-basics-quirks-expected.txt: Added.
* fast/dom/Element/body-scrollHeight-basics-quirks.html: Added.
* fast/dom/Element/body-scrollLeft-basics-quirks-expected.txt: Added.
* fast/dom/Element/body-scrollLeft-basics-quirks.html: Added.
* fast/dom/Element/body-scrollTop-basics-quirks-expected.txt: Added.
* fast/dom/Element/body-scrollTop-basics-quirks.html: Added.
* fast/dom/Element/body-scrollWidth-basics-quirks-expected.txt: Added.
* fast/dom/Element/body-scrollWidth-basics-quirks.html: Added.
Test the various behaviors.
* fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollLeft-expected.txt: Added.
* fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollLeft.html: Added.
* fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollTop-expected.txt: Added.
* fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollTop.html: Added.
Those tests target specifically the bug.</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkLayoutTestsChangeLog">trunk/LayoutTests/ChangeLog</a></li>
<li><a href="#trunkSourceWebCoreChangeLog">trunk/Source/WebCore/ChangeLog</a></li>
<li><a href="#trunkSourceWebCorehtmlHTMLBodyElementcpp">trunk/Source/WebCore/html/HTMLBodyElement.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlHTMLBodyElementh">trunk/Source/WebCore/html/HTMLBodyElement.h</a></li>
</ul>
<h3>Added Paths</h3>
<ul>
<li><a href="#trunkLayoutTestsfastdomElementbodyscrollHeightbasicsquirksexpectedtxt">trunk/LayoutTests/fast/dom/Element/body-scrollHeight-basics-quirks-expected.txt</a></li>
<li><a href="#trunkLayoutTestsfastdomElementbodyscrollHeightbasicsquirkshtml">trunk/LayoutTests/fast/dom/Element/body-scrollHeight-basics-quirks.html</a></li>
<li><a href="#trunkLayoutTestsfastdomElementbodyscrollLeftbasicsquirksexpectedtxt">trunk/LayoutTests/fast/dom/Element/body-scrollLeft-basics-quirks-expected.txt</a></li>
<li><a href="#trunkLayoutTestsfastdomElementbodyscrollLeftbasicsquirkshtml">trunk/LayoutTests/fast/dom/Element/body-scrollLeft-basics-quirks.html</a></li>
<li><a href="#trunkLayoutTestsfastdomElementbodyscrollTopbasicsquirksexpectedtxt">trunk/LayoutTests/fast/dom/Element/body-scrollTop-basics-quirks-expected.txt</a></li>
<li><a href="#trunkLayoutTestsfastdomElementbodyscrollTopbasicsquirkshtml">trunk/LayoutTests/fast/dom/Element/body-scrollTop-basics-quirks.html</a></li>
<li><a href="#trunkLayoutTestsfastdomElementbodyscrollWidthbasicsquirksexpectedtxt">trunk/LayoutTests/fast/dom/Element/body-scrollWidth-basics-quirks-expected.txt</a></li>
<li><a href="#trunkLayoutTestsfastdomElementbodyscrollWidthbasicsquirkshtml">trunk/LayoutTests/fast/dom/Element/body-scrollWidth-basics-quirks.html</a></li>
<li><a href="#trunkLayoutTestsfastdomElementdetachedbodyelementdoesnotscrollmainframewithscrollLeftexpectedtxt">trunk/LayoutTests/fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollLeft-expected.txt</a></li>
<li><a href="#trunkLayoutTestsfastdomElementdetachedbodyelementdoesnotscrollmainframewithscrollLefthtml">trunk/LayoutTests/fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollLeft.html</a></li>
<li><a href="#trunkLayoutTestsfastdomElementdetachedbodyelementdoesnotscrollmainframewithscrollTopexpectedtxt">trunk/LayoutTests/fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollTop-expected.txt</a></li>
<li><a href="#trunkLayoutTestsfastdomElementdetachedbodyelementdoesnotscrollmainframewithscrollTophtml">trunk/LayoutTests/fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollTop.html</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkLayoutTestsChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/ChangeLog (182676 => 182677)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/ChangeLog 2015-04-13 01:29:59 UTC (rev 182676)
+++ trunk/LayoutTests/ChangeLog 2015-04-13 04:42:53 UTC (rev 182677)
</span><span class="lines">@@ -1,3 +1,26 @@
</span><ins>+2015-04-12 Benjamin Poulain <benjamin@webkit.org> and Diego Perini <diego.perini@gmail.com>
+
+ Legacy scroll behavior on HTMLBodyElement should only apply to the first body element of a document
+ https://bugs.webkit.org/show_bug.cgi?id=143651
+
+ Reviewed by Sam Weinig.
+
+ * fast/dom/Element/body-scrollHeight-basics-quirks-expected.txt: Added.
+ * fast/dom/Element/body-scrollHeight-basics-quirks.html: Added.
+ * fast/dom/Element/body-scrollLeft-basics-quirks-expected.txt: Added.
+ * fast/dom/Element/body-scrollLeft-basics-quirks.html: Added.
+ * fast/dom/Element/body-scrollTop-basics-quirks-expected.txt: Added.
+ * fast/dom/Element/body-scrollTop-basics-quirks.html: Added.
+ * fast/dom/Element/body-scrollWidth-basics-quirks-expected.txt: Added.
+ * fast/dom/Element/body-scrollWidth-basics-quirks.html: Added.
+ Test the various behaviors.
+
+ * fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollLeft-expected.txt: Added.
+ * fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollLeft.html: Added.
+ * fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollTop-expected.txt: Added.
+ * fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollTop.html: Added.
+ Those tests target specifically the bug.
+
</ins><span class="cx"> 2015-04-12 Gyuyoung Kim <gyuyoung.kim@webkit.org>
</span><span class="cx">
</span><span class="cx"> [EFL] Enable Media Source
</span></span></pre></div>
<a id="trunkLayoutTestsfastdomElementbodyscrollHeightbasicsquirksexpectedtxt"></a>
<div class="addfile"><h4>Added: trunk/LayoutTests/fast/dom/Element/body-scrollHeight-basics-quirks-expected.txt (0 => 182677)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/fast/dom/Element/body-scrollHeight-basics-quirks-expected.txt (rev 0)
+++ trunk/LayoutTests/fast/dom/Element/body-scrollHeight-basics-quirks-expected.txt 2015-04-13 04:42:53 UTC (rev 182677)
</span><span class="lines">@@ -0,0 +1,29 @@
</span><ins>+Test the "scrollHeight" properties on the body element in quirks mode.
+
+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
+
+
+Initial values
+PASS document.body.scrollHeight is document.documentElement.scrollHeight
+
+Now we create a body element, it should not have a scrollHeight since it is not the first body element of the document.
+PASS document.body.scrollHeight is document.documentElement.scrollHeight
+PASS floatingBody.scrollHeight is 0
+
+We add a new body element after the first body element, this should have no impact on the state.
+PASS document.body.scrollHeight is document.documentElement.scrollHeight
+PASS secondSiblingBody.scrollHeight is secondSiblingBody.clientHeight
+PASS document.body.scrollHeight is not secondSiblingBody.scrollHeight
+
+We add a new body element inside the first body element, this should have no impact on the state.
+PASS document.body.scrollHeight is document.documentElement.scrollHeight
+PASS bodyChildOfMainBody.scrollHeight is bodyChildOfMainBody.clientHeight
+PASS document.body.scrollHeight is not bodyChildOfMainBody.scrollHeight
+
+Getting the scrollHeight should not require a renderer.
+PASS document.body.style.display = 'none' is 'none'
+PASS document.body.scrollHeight is document.documentElement.scrollHeight
+PASS successfullyParsed is true
+
+TEST COMPLETE
+
</ins></span></pre></div>
<a id="trunkLayoutTestsfastdomElementbodyscrollHeightbasicsquirkshtml"></a>
<div class="addfile"><h4>Added: trunk/LayoutTests/fast/dom/Element/body-scrollHeight-basics-quirks.html (0 => 182677)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/fast/dom/Element/body-scrollHeight-basics-quirks.html (rev 0)
+++ trunk/LayoutTests/fast/dom/Element/body-scrollHeight-basics-quirks.html 2015-04-13 04:42:53 UTC (rev 182677)
</span><span class="lines">@@ -0,0 +1,58 @@
</span><ins>+<html>
+ <head>
+ <style>
+ div {
+ height: 9999px;
+ width: 9999px;
+ }
+ </style>
+ <script src="../../../resources/js-test-pre.js"></script>
+ <script>
+ jsTestIsAsync = true;
+
+ function runTest() {
+ description('Test the "scrollHeight" properties on the body element in quirks mode.');
+
+ debug("Initial values");
+ shouldBe("document.body.scrollHeight", "document.documentElement.scrollHeight");
+
+ debug("");
+ debug("Now we create a body element, it should not have a scrollHeight since it is not the first body element of the document.");
+ floatingBody = document.createElement('body');
+ shouldBe("document.body.scrollHeight", "document.documentElement.scrollHeight");
+ shouldBe("floatingBody.scrollHeight", "0");
+
+ secondSiblingBody = document.createElement('body');
+ document.documentElement.appendChild(secondSiblingBody);
+ debug("");
+ debug("We add a new body element after the first body element, this should have no impact on the state.");
+ shouldBe("document.body.scrollHeight", "document.documentElement.scrollHeight");
+ shouldBe("secondSiblingBody.scrollHeight", "secondSiblingBody.clientHeight");
+ shouldNotBe("document.body.scrollHeight", "secondSiblingBody.scrollHeight");
+
+ bodyChildOfMainBody = document.createElement('body');
+ document.body.appendChild(bodyChildOfMainBody);
+ debug("");
+ debug("We add a new body element inside the first body element, this should have no impact on the state.");
+ shouldBe("document.body.scrollHeight", "document.documentElement.scrollHeight");
+ shouldBe("bodyChildOfMainBody.scrollHeight", "bodyChildOfMainBody.clientHeight");
+ shouldNotBe("document.body.scrollHeight", "bodyChildOfMainBody.scrollHeight");
+
+ debug("");
+ debug("Getting the scrollHeight should not require a renderer.");
+ shouldBe("document.body.style.display = 'none'", "'none'");
+ shouldBe("document.body.scrollHeight", "document.documentElement.scrollHeight");
+
+ // Finally, let's clear the state to make this simpler for manual testing.
+ document.body.style.display = "block";
+ window.scrollTo(0, 0);
+
+ finishJSTest();
+ }
+ </script>
+ </head>
+ <body onload="runTest()">
+ <div></div>
+ <script src="../../../resources/js-test-post.js"></script>
+ </body>
+</html>
</ins></span></pre></div>
<a id="trunkLayoutTestsfastdomElementbodyscrollLeftbasicsquirksexpectedtxt"></a>
<div class="addfile"><h4>Added: trunk/LayoutTests/fast/dom/Element/body-scrollLeft-basics-quirks-expected.txt (0 => 182677)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/fast/dom/Element/body-scrollLeft-basics-quirks-expected.txt (rev 0)
+++ trunk/LayoutTests/fast/dom/Element/body-scrollLeft-basics-quirks-expected.txt 2015-04-13 04:42:53 UTC (rev 182677)
</span><span class="lines">@@ -0,0 +1,134 @@
</span><ins>+Test the "scrollLeft" properties on the body element in quirks mode.
+
+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
+
+
+Initial values
+PASS document.body.scrollLeft is 0
+PASS document.body.scrollTop is 0
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 0
+PASS window.scrollY is 0
+
+Let set the scrollLeft value on the main body element (document.body), in quirks mode, this should change the main frame's scroll position.
+PASS document.body.scrollLeft = 42 is 42
+PASS document.body.scrollLeft is 42
+PASS document.body.scrollTop is 0
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 42
+PASS window.scrollY is 0
+
+Change scrollX, this should have no effect on the horizontal scrolling
+PASS window.scrollBy(0, 21) is undefined
+PASS document.body.scrollLeft is 42
+PASS document.body.scrollTop is 21
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 42
+PASS window.scrollY is 21
+
+Now we create a body element, completely detached from the DOM tree. This should not change the scroll position in any way.
+PASS floatingBody.scrollLeft is 0
+PASS floatingBody.scrollTop is 0
+PASS document.body.scrollLeft is 42
+PASS document.body.scrollTop is 21
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 42
+PASS window.scrollY is 21
+
+Setting the scroll position on that detached body element should not affect the main frame since it is not the first body element of the document.
+PASS floatingBody.scrollLeft = 654 is 654
+PASS floatingBody.scrollLeft is 0
+PASS floatingBody.scrollTop is 0
+PASS document.body.scrollLeft is 42
+PASS document.body.scrollTop is 21
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 42
+PASS window.scrollY is 21
+
+We add a new body element after the first body element, this should have no impact on the state.
+PASS secondSiblingBody.scrollLeft is 0
+PASS secondSiblingBody.scrollTop is 0
+PASS document.body.scrollLeft is 42
+PASS document.body.scrollTop is 21
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 42
+PASS window.scrollY is 21
+
+Setting a "scrollTop" on secondSiblingBody should not affect the main frame.
+PASS secondSiblingBody.scrollLeft = 987 is 987
+PASS secondSiblingBody.scrollLeft is 0
+PASS secondSiblingBody.scrollTop is 0
+PASS document.body.scrollLeft is 42
+PASS document.body.scrollTop is 21
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 42
+PASS window.scrollY is 21
+
+We add a new body element inside the first body element, this should have no impact on the state.
+PASS bodyChildOfMainBody.scrollLeft is 0
+PASS bodyChildOfMainBody.scrollTop is 0
+PASS document.body.scrollLeft is 42
+PASS document.body.scrollTop is 21
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 42
+PASS window.scrollY is 21
+
+Setting a "scrollTop" on bodyChildOfMainBody should not affect the main frame.
+PASS bodyChildOfMainBody.scrollLeft = 57 is 57
+PASS bodyChildOfMainBody.scrollLeft is 0
+PASS bodyChildOfMainBody.scrollTop is 0
+PASS document.body.scrollLeft is 42
+PASS document.body.scrollTop is 21
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 42
+PASS window.scrollY is 21
+
+Now we insert a new body element *before* the existing one. It will replace the existing element as the first body element of the document.
+PASS document.body === newMainBody is true
+PASS document.body !== oldBodyElement is true
+Since we are in quirks mode, the new body element gets the scroll position from the frame, and the old one lose its scroll position.
+PASS oldBodyElement.scrollLeft is 0
+PASS oldBodyElement.scrollTop is 0
+PASS document.body.scrollLeft is 42
+PASS document.body.scrollTop is 21
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 42
+PASS window.scrollY is 21
+
+Scrolling that new body should work as expected.
+PASS document.body.scrollLeft = 68 is 68
+PASS oldBodyElement.scrollLeft = 894 is 894
+PASS oldBodyElement.scrollLeft is 0
+PASS oldBodyElement.scrollTop is 0
+PASS document.body.scrollLeft is 68
+PASS document.body.scrollTop is 21
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 68
+PASS window.scrollY is 21
+
+Scrolling the main frame should not require a renderer.
+PASS document.body.style.display = 'none' is 'none'
+PASS document.body.scrollLeft = 54 is 54
+PASS oldBodyElement.scrollLeft is 0
+PASS oldBodyElement.scrollTop is 0
+PASS document.body.scrollLeft is 54
+PASS document.body.scrollTop is 21
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 54
+PASS window.scrollY is 21
+PASS successfullyParsed is true
+
+TEST COMPLETE
+
</ins></span></pre></div>
<a id="trunkLayoutTestsfastdomElementbodyscrollLeftbasicsquirkshtml"></a>
<div class="addfile"><h4>Added: trunk/LayoutTests/fast/dom/Element/body-scrollLeft-basics-quirks.html (0 => 182677)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/fast/dom/Element/body-scrollLeft-basics-quirks.html (rev 0)
+++ trunk/LayoutTests/fast/dom/Element/body-scrollLeft-basics-quirks.html 2015-04-13 04:42:53 UTC (rev 182677)
</span><span class="lines">@@ -0,0 +1,175 @@
</span><ins>+<html>
+ <head>
+ <style>
+ div {
+ height: 9999px;
+ width: 9999px;
+ }
+ </style>
+ <script src="../../../resources/js-test-pre.js"></script>
+ <script>
+ jsTestIsAsync = true;
+
+ function runTest() {
+ description('Test the "scrollLeft" properties on the body element in quirks mode.');
+
+ debug("Initial values");
+ shouldBe("document.body.scrollLeft", "0");
+ shouldBe("document.body.scrollTop", "0");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "0");
+ shouldBe("window.scrollY", "0");
+
+ debug("");
+ debug("Let set the scrollLeft value on the main body element (document.body), in quirks mode, this should change the main frame's scroll position.");
+ shouldBe("document.body.scrollLeft = 42", "42");
+ shouldBe("document.body.scrollLeft", "42");
+ shouldBe("document.body.scrollTop", "0");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "42");
+ shouldBe("window.scrollY", "0");
+
+ debug("");
+ debug("Change scrollX, this should have no effect on the horizontal scrolling");
+ shouldBe("window.scrollBy(0, 21)", "undefined");
+ shouldBe("document.body.scrollLeft", "42");
+ shouldBe("document.body.scrollTop", "21");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "42");
+ shouldBe("window.scrollY", "21");
+
+ debug("");
+ debug("Now we create a body element, completely detached from the DOM tree. This should not change the scroll position in any way.");
+ floatingBody = document.createElement('body');
+ shouldBe("floatingBody.scrollLeft", "0");
+ shouldBe("floatingBody.scrollTop", "0");
+ shouldBe("document.body.scrollLeft", "42");
+ shouldBe("document.body.scrollTop", "21");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "42");
+ shouldBe("window.scrollY", "21");
+
+ debug("");
+ debug("Setting the scroll position on that detached body element should not affect the main frame since it is not the first body element of the document.");
+ shouldBe("floatingBody.scrollLeft = 654", "654");
+ shouldBe("floatingBody.scrollLeft", "0");
+ shouldBe("floatingBody.scrollTop", "0");
+ shouldBe("document.body.scrollLeft", "42");
+ shouldBe("document.body.scrollTop", "21");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "42");
+ shouldBe("window.scrollY", "21");
+
+ secondSiblingBody = document.createElement('body');
+ document.documentElement.appendChild(secondSiblingBody);
+ debug("");
+ debug("We add a new body element after the first body element, this should have no impact on the state.");
+ shouldBe("secondSiblingBody.scrollLeft", "0");
+ shouldBe("secondSiblingBody.scrollTop", "0");
+ shouldBe("document.body.scrollLeft", "42");
+ shouldBe("document.body.scrollTop", "21");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "42");
+ shouldBe("window.scrollY", "21");
+
+ debug("");
+ debug("Setting a \"scrollTop\" on secondSiblingBody should not affect the main frame.");
+ shouldBe("secondSiblingBody.scrollLeft = 987", "987");
+ shouldBe("secondSiblingBody.scrollLeft", "0");
+ shouldBe("secondSiblingBody.scrollTop", "0");
+ shouldBe("document.body.scrollLeft", "42");
+ shouldBe("document.body.scrollTop", "21");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "42");
+ shouldBe("window.scrollY", "21");
+
+ bodyChildOfMainBody = document.createElement('body');
+ document.body.appendChild(bodyChildOfMainBody);
+ debug("");
+ debug("We add a new body element inside the first body element, this should have no impact on the state.");
+ shouldBe("bodyChildOfMainBody.scrollLeft", "0");
+ shouldBe("bodyChildOfMainBody.scrollTop", "0");
+ shouldBe("document.body.scrollLeft", "42");
+ shouldBe("document.body.scrollTop", "21");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "42");
+ shouldBe("window.scrollY", "21");
+
+ debug("");
+ debug("Setting a \"scrollTop\" on bodyChildOfMainBody should not affect the main frame.");
+ shouldBe("bodyChildOfMainBody.scrollLeft = 57", "57");
+ shouldBe("bodyChildOfMainBody.scrollLeft", "0");
+ shouldBe("bodyChildOfMainBody.scrollTop", "0");
+ shouldBe("document.body.scrollLeft", "42");
+ shouldBe("document.body.scrollTop", "21");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "42");
+ shouldBe("window.scrollY", "21");
+
+ debug("");
+ debug("Now we insert a new body element *before* the existing one. It will replace the existing element as the first body element of the document.");
+ oldBodyElement = document.body;
+ newMainBody = document.createElement('body');
+ document.documentElement.insertBefore(newMainBody, oldBodyElement);
+ shouldBeTrue("document.body === newMainBody");
+ shouldBeTrue("document.body !== oldBodyElement");
+
+ debug("Since we are in quirks mode, the new body element gets the scroll position from the frame, and the old one lose its scroll position.");
+ shouldBe("oldBodyElement.scrollLeft", "0");
+ shouldBe("oldBodyElement.scrollTop", "0");
+ shouldBe("document.body.scrollLeft", "42");
+ shouldBe("document.body.scrollTop", "21");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "42");
+ shouldBe("window.scrollY", "21");
+
+ debug("");
+ debug("Scrolling that new body should work as expected.");
+ shouldBe("document.body.scrollLeft = 68", "68");
+ shouldBe("oldBodyElement.scrollLeft = 894", "894");
+
+ shouldBe("oldBodyElement.scrollLeft", "0");
+ shouldBe("oldBodyElement.scrollTop", "0");
+ shouldBe("document.body.scrollLeft", "68");
+ shouldBe("document.body.scrollTop", "21");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "68");
+ shouldBe("window.scrollY", "21");
+
+ debug("");
+ debug("Scrolling the main frame should not require a renderer.");
+ shouldBe("document.body.style.display = 'none'", "'none'");
+ shouldBe("document.body.scrollLeft = 54", "54");
+ shouldBe("oldBodyElement.scrollLeft", "0");
+ shouldBe("oldBodyElement.scrollTop", "0");
+ shouldBe("document.body.scrollLeft", "54");
+ shouldBe("document.body.scrollTop", "21");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "54");
+ shouldBe("window.scrollY", "21");
+
+ // Finally, let's clear the state to make this simpler for manual testing.
+ newMainBody.parentNode.removeChild(newMainBody);
+ window.scrollTo(0, 0);
+
+ finishJSTest();
+ }
+ </script>
+ </head>
+ <body onload="runTest()">
+ <div></div>
+ <script src="../../../resources/js-test-post.js"></script>
+ </body>
+</html>
</ins></span></pre></div>
<a id="trunkLayoutTestsfastdomElementbodyscrollTopbasicsquirksexpectedtxt"></a>
<div class="addfile"><h4>Added: trunk/LayoutTests/fast/dom/Element/body-scrollTop-basics-quirks-expected.txt (0 => 182677)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/fast/dom/Element/body-scrollTop-basics-quirks-expected.txt (rev 0)
+++ trunk/LayoutTests/fast/dom/Element/body-scrollTop-basics-quirks-expected.txt 2015-04-13 04:42:53 UTC (rev 182677)
</span><span class="lines">@@ -0,0 +1,134 @@
</span><ins>+Test the "scrollTop" properties on the body element in quirks mode.
+
+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
+
+
+Initial values
+PASS document.body.scrollLeft is 0
+PASS document.body.scrollTop is 0
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 0
+PASS window.scrollY is 0
+
+Let set the scrollTop value on the main body element (document.body), in quirks mode, this should change the main frame's scroll position.
+PASS document.body.scrollTop = 42 is 42
+PASS document.body.scrollLeft is 0
+PASS document.body.scrollTop is 42
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 0
+PASS window.scrollY is 42
+
+Change scrollX, this should have no effect on the vertical scrolling
+PASS window.scrollBy(21, 0) is undefined
+PASS document.body.scrollLeft is 21
+PASS document.body.scrollTop is 42
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 21
+PASS window.scrollY is 42
+
+Now we create a body element, completely detached from the DOM tree. This should not change the scroll position in any way.
+PASS floatingBody.scrollLeft is 0
+PASS floatingBody.scrollTop is 0
+PASS document.body.scrollLeft is 21
+PASS document.body.scrollTop is 42
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 21
+PASS window.scrollY is 42
+
+Setting the scroll position on that detached body element should not affect the main frame since it is not the first body element of the document.
+PASS floatingBody.scrollTop = 500 is 500
+PASS floatingBody.scrollLeft is 0
+PASS floatingBody.scrollTop is 0
+PASS document.body.scrollLeft is 21
+PASS document.body.scrollTop is 42
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 21
+PASS window.scrollY is 42
+
+We add a new body element after the first body element, this should have no impact on the state.
+PASS secondSiblingBody.scrollLeft is 0
+PASS secondSiblingBody.scrollTop is 0
+PASS document.body.scrollLeft is 21
+PASS document.body.scrollTop is 42
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 21
+PASS window.scrollY is 42
+
+Setting a "scrollTop" on secondSiblingBody should not affect the main frame.
+PASS secondSiblingBody.scrollTop = 321 is 321
+PASS secondSiblingBody.scrollLeft is 0
+PASS secondSiblingBody.scrollTop is 0
+PASS document.body.scrollLeft is 21
+PASS document.body.scrollTop is 42
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 21
+PASS window.scrollY is 42
+
+We add a new body element inside the first body element, this should have no impact on the state.
+PASS bodyChildOfMainBody.scrollLeft is 0
+PASS bodyChildOfMainBody.scrollTop is 0
+PASS document.body.scrollLeft is 21
+PASS document.body.scrollTop is 42
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 21
+PASS window.scrollY is 42
+
+Setting a "scrollTop" on bodyChildOfMainBody should not affect the main frame.
+PASS bodyChildOfMainBody.scrollTop = 57 is 57
+PASS bodyChildOfMainBody.scrollLeft is 0
+PASS bodyChildOfMainBody.scrollTop is 0
+PASS document.body.scrollLeft is 21
+PASS document.body.scrollTop is 42
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 21
+PASS window.scrollY is 42
+
+Now we insert a new body element *before* the existing one. It will replace the existing element as the first body element of the document.
+PASS document.body === newMainBody is true
+PASS document.body !== oldBodyElement is true
+Since we are in quirks mode, the new body element gets the scroll position from the frame, and the old one lose its scroll position.
+PASS oldBodyElement.scrollLeft is 0
+PASS oldBodyElement.scrollTop is 0
+PASS document.body.scrollLeft is 21
+PASS document.body.scrollTop is 42
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 21
+PASS window.scrollY is 42
+
+Scrolling that new body should work as expected.
+PASS document.body.scrollTop = 68 is 68
+PASS oldBodyElement.scrollTop = 894 is 894
+PASS oldBodyElement.scrollLeft is 0
+PASS oldBodyElement.scrollTop is 0
+PASS document.body.scrollLeft is 21
+PASS document.body.scrollTop is 68
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 21
+PASS window.scrollY is 68
+
+Scrolling the main frame should not require a renderer.
+PASS document.body.style.display = 'none' is 'none'
+PASS document.body.scrollTop = 54 is 54
+PASS oldBodyElement.scrollLeft is 0
+PASS oldBodyElement.scrollTop is 0
+PASS document.body.scrollLeft is 21
+PASS document.body.scrollTop is 54
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 21
+PASS window.scrollY is 54
+PASS successfullyParsed is true
+
+TEST COMPLETE
+
</ins></span></pre></div>
<a id="trunkLayoutTestsfastdomElementbodyscrollTopbasicsquirkshtml"></a>
<div class="addfile"><h4>Added: trunk/LayoutTests/fast/dom/Element/body-scrollTop-basics-quirks.html (0 => 182677)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/fast/dom/Element/body-scrollTop-basics-quirks.html (rev 0)
+++ trunk/LayoutTests/fast/dom/Element/body-scrollTop-basics-quirks.html 2015-04-13 04:42:53 UTC (rev 182677)
</span><span class="lines">@@ -0,0 +1,175 @@
</span><ins>+<html>
+ <head>
+ <style>
+ div {
+ height: 9999px;
+ width: 9999px;
+ }
+ </style>
+ <script src="../../../resources/js-test-pre.js"></script>
+ <script>
+ jsTestIsAsync = true;
+
+ function runTest() {
+ description('Test the "scrollTop" properties on the body element in quirks mode.');
+
+ debug("Initial values");
+ shouldBe("document.body.scrollLeft", "0");
+ shouldBe("document.body.scrollTop", "0");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "0");
+ shouldBe("window.scrollY", "0");
+
+ debug("");
+ debug("Let set the scrollTop value on the main body element (document.body), in quirks mode, this should change the main frame's scroll position.");
+ shouldBe("document.body.scrollTop = 42", "42");
+ shouldBe("document.body.scrollLeft", "0");
+ shouldBe("document.body.scrollTop", "42");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "0");
+ shouldBe("window.scrollY", "42");
+
+ debug("");
+ debug("Change scrollX, this should have no effect on the vertical scrolling");
+ shouldBe("window.scrollBy(21, 0)", "undefined");
+ shouldBe("document.body.scrollLeft", "21");
+ shouldBe("document.body.scrollTop", "42");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "21");
+ shouldBe("window.scrollY", "42");
+
+ debug("");
+ debug("Now we create a body element, completely detached from the DOM tree. This should not change the scroll position in any way.");
+ floatingBody = document.createElement('body');
+ shouldBe("floatingBody.scrollLeft", "0");
+ shouldBe("floatingBody.scrollTop", "0");
+ shouldBe("document.body.scrollLeft", "21");
+ shouldBe("document.body.scrollTop", "42");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "21");
+ shouldBe("window.scrollY", "42");
+
+ debug("");
+ debug("Setting the scroll position on that detached body element should not affect the main frame since it is not the first body element of the document.");
+ shouldBe("floatingBody.scrollTop = 500", "500");
+ shouldBe("floatingBody.scrollLeft", "0");
+ shouldBe("floatingBody.scrollTop", "0");
+ shouldBe("document.body.scrollLeft", "21");
+ shouldBe("document.body.scrollTop", "42");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "21");
+ shouldBe("window.scrollY", "42");
+
+ secondSiblingBody = document.createElement('body');
+ document.documentElement.appendChild(secondSiblingBody);
+ debug("");
+ debug("We add a new body element after the first body element, this should have no impact on the state.");
+ shouldBe("secondSiblingBody.scrollLeft", "0");
+ shouldBe("secondSiblingBody.scrollTop", "0");
+ shouldBe("document.body.scrollLeft", "21");
+ shouldBe("document.body.scrollTop", "42");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "21");
+ shouldBe("window.scrollY", "42");
+
+ debug("");
+ debug("Setting a \"scrollTop\" on secondSiblingBody should not affect the main frame.");
+ shouldBe("secondSiblingBody.scrollTop = 321", "321");
+ shouldBe("secondSiblingBody.scrollLeft", "0");
+ shouldBe("secondSiblingBody.scrollTop", "0");
+ shouldBe("document.body.scrollLeft", "21");
+ shouldBe("document.body.scrollTop", "42");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "21");
+ shouldBe("window.scrollY", "42");
+
+ bodyChildOfMainBody = document.createElement('body');
+ document.body.appendChild(bodyChildOfMainBody);
+ debug("");
+ debug("We add a new body element inside the first body element, this should have no impact on the state.");
+ shouldBe("bodyChildOfMainBody.scrollLeft", "0");
+ shouldBe("bodyChildOfMainBody.scrollTop", "0");
+ shouldBe("document.body.scrollLeft", "21");
+ shouldBe("document.body.scrollTop", "42");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "21");
+ shouldBe("window.scrollY", "42");
+
+ debug("");
+ debug("Setting a \"scrollTop\" on bodyChildOfMainBody should not affect the main frame.");
+ shouldBe("bodyChildOfMainBody.scrollTop = 57", "57");
+ shouldBe("bodyChildOfMainBody.scrollLeft", "0");
+ shouldBe("bodyChildOfMainBody.scrollTop", "0");
+ shouldBe("document.body.scrollLeft", "21");
+ shouldBe("document.body.scrollTop", "42");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "21");
+ shouldBe("window.scrollY", "42");
+
+ debug("");
+ debug("Now we insert a new body element *before* the existing one. It will replace the existing element as the first body element of the document.");
+ oldBodyElement = document.body;
+ newMainBody = document.createElement('body');
+ document.documentElement.insertBefore(newMainBody, oldBodyElement);
+ shouldBeTrue("document.body === newMainBody");
+ shouldBeTrue("document.body !== oldBodyElement");
+
+ debug("Since we are in quirks mode, the new body element gets the scroll position from the frame, and the old one lose its scroll position.");
+ shouldBe("oldBodyElement.scrollLeft", "0");
+ shouldBe("oldBodyElement.scrollTop", "0");
+ shouldBe("document.body.scrollLeft", "21");
+ shouldBe("document.body.scrollTop", "42");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "21");
+ shouldBe("window.scrollY", "42");
+
+ debug("");
+ debug("Scrolling that new body should work as expected.");
+ shouldBe("document.body.scrollTop = 68", "68");
+ shouldBe("oldBodyElement.scrollTop = 894", "894");
+
+ shouldBe("oldBodyElement.scrollLeft", "0");
+ shouldBe("oldBodyElement.scrollTop", "0");
+ shouldBe("document.body.scrollLeft", "21");
+ shouldBe("document.body.scrollTop", "68");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "21");
+ shouldBe("window.scrollY", "68");
+
+ debug("");
+ debug("Scrolling the main frame should not require a renderer.");
+ shouldBe("document.body.style.display = 'none'", "'none'");
+ shouldBe("document.body.scrollTop = 54", "54");
+ shouldBe("oldBodyElement.scrollLeft", "0");
+ shouldBe("oldBodyElement.scrollTop", "0");
+ shouldBe("document.body.scrollLeft", "21");
+ shouldBe("document.body.scrollTop", "54");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "21");
+ shouldBe("window.scrollY", "54");
+
+ // Finally, let's clear the state to make this simpler for manual testing.
+ newMainBody.parentNode.removeChild(newMainBody);
+ window.scrollTo(0, 0);
+
+ finishJSTest();
+ }
+ </script>
+ </head>
+ <body onload="runTest()">
+ <div></div>
+ <script src="../../../resources/js-test-post.js"></script>
+ </body>
+</html>
</ins></span></pre></div>
<a id="trunkLayoutTestsfastdomElementbodyscrollWidthbasicsquirksexpectedtxt"></a>
<div class="addfile"><h4>Added: trunk/LayoutTests/fast/dom/Element/body-scrollWidth-basics-quirks-expected.txt (0 => 182677)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/fast/dom/Element/body-scrollWidth-basics-quirks-expected.txt (rev 0)
+++ trunk/LayoutTests/fast/dom/Element/body-scrollWidth-basics-quirks-expected.txt 2015-04-13 04:42:53 UTC (rev 182677)
</span><span class="lines">@@ -0,0 +1,29 @@
</span><ins>+Test the "scrollWidth" properties on the body element in quirks mode.
+
+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
+
+
+Initial values
+PASS document.body.scrollWidth is document.documentElement.scrollWidth
+
+Now we create a body element, it should not have a scrollWidth since it is not the first body element of the document.
+PASS document.body.scrollWidth is document.documentElement.scrollWidth
+PASS floatingBody.scrollWidth is 0
+
+We add a new body element after the first body element, this should have no impact on the state.
+PASS document.body.scrollWidth is document.documentElement.scrollWidth
+PASS secondSiblingBody.scrollWidth is secondSiblingBody.clientWidth
+PASS document.body.scrollWidth is not secondSiblingBody.scrollWidth
+
+We add a new body element inside the first body element, this should have no impact on the state.
+PASS document.body.scrollWidth is document.documentElement.scrollWidth
+PASS bodyChildOfMainBody.scrollWidth is bodyChildOfMainBody.clientWidth
+PASS document.body.scrollWidth is not bodyChildOfMainBody.scrollWidth
+
+Getting the scrollWidth should not require a renderer.
+PASS document.body.style.display = 'none' is 'none'
+PASS document.body.scrollWidth is document.documentElement.scrollWidth
+PASS successfullyParsed is true
+
+TEST COMPLETE
+
</ins></span></pre></div>
<a id="trunkLayoutTestsfastdomElementbodyscrollWidthbasicsquirkshtml"></a>
<div class="addfile"><h4>Added: trunk/LayoutTests/fast/dom/Element/body-scrollWidth-basics-quirks.html (0 => 182677)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/fast/dom/Element/body-scrollWidth-basics-quirks.html (rev 0)
+++ trunk/LayoutTests/fast/dom/Element/body-scrollWidth-basics-quirks.html 2015-04-13 04:42:53 UTC (rev 182677)
</span><span class="lines">@@ -0,0 +1,58 @@
</span><ins>+<html>
+ <head>
+ <style>
+ div {
+ height: 9999px;
+ width: 9999px;
+ }
+ </style>
+ <script src="../../../resources/js-test-pre.js"></script>
+ <script>
+ jsTestIsAsync = true;
+
+ function runTest() {
+ description('Test the "scrollWidth" properties on the body element in quirks mode.');
+
+ debug("Initial values");
+ shouldBe("document.body.scrollWidth", "document.documentElement.scrollWidth");
+
+ debug("");
+ debug("Now we create a body element, it should not have a scrollWidth since it is not the first body element of the document.");
+ floatingBody = document.createElement('body');
+ shouldBe("document.body.scrollWidth", "document.documentElement.scrollWidth");
+ shouldBe("floatingBody.scrollWidth", "0");
+
+ secondSiblingBody = document.createElement('body');
+ document.documentElement.appendChild(secondSiblingBody);
+ debug("");
+ debug("We add a new body element after the first body element, this should have no impact on the state.");
+ shouldBe("document.body.scrollWidth", "document.documentElement.scrollWidth");
+ shouldBe("secondSiblingBody.scrollWidth", "secondSiblingBody.clientWidth");
+ shouldNotBe("document.body.scrollWidth", "secondSiblingBody.scrollWidth");
+
+ bodyChildOfMainBody = document.createElement('body');
+ document.body.appendChild(bodyChildOfMainBody);
+ debug("");
+ debug("We add a new body element inside the first body element, this should have no impact on the state.");
+ shouldBe("document.body.scrollWidth", "document.documentElement.scrollWidth");
+ shouldBe("bodyChildOfMainBody.scrollWidth", "bodyChildOfMainBody.clientWidth");
+ shouldNotBe("document.body.scrollWidth", "bodyChildOfMainBody.scrollWidth");
+
+ debug("");
+ debug("Getting the scrollWidth should not require a renderer.");
+ shouldBe("document.body.style.display = 'none'", "'none'");
+ shouldBe("document.body.scrollWidth", "document.documentElement.scrollWidth");
+
+ // Finally, let's clear the state to make this simpler for manual testing.
+ document.body.style.display = "block";
+ window.scrollTo(0, 0);
+
+ finishJSTest();
+ }
+ </script>
+ </head>
+ <body onload="runTest()">
+ <div></div>
+ <script src="../../../resources/js-test-post.js"></script>
+ </body>
+</html>
</ins></span></pre></div>
<a id="trunkLayoutTestsfastdomElementdetachedbodyelementdoesnotscrollmainframewithscrollLeftexpectedtxt"></a>
<div class="addfile"><h4>Added: trunk/LayoutTests/fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollLeft-expected.txt (0 => 182677)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollLeft-expected.txt (rev 0)
+++ trunk/LayoutTests/fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollLeft-expected.txt 2015-04-13 04:42:53 UTC (rev 182677)
</span><span class="lines">@@ -0,0 +1,32 @@
</span><ins>+Tests that, for Quirks mode, changing the scrollTop property of a fake body element does not scroll the viewport
+
+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
+
+
+
+Initial state.
+PASS detachedBody.scrollTop is 0
+PASS detachedBody.scrollLeft is 0
+PASS document.body.scrollTop = 0 is 0
+PASS document.body.scrollLeft is 0
+PASS document.body.scrollTop is 0
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 0
+PASS window.scrollY is 0
+
+Setting scrollTop on the detached body element should not change the main frame's scroll position.
+PASS detachedBody.scrollLeft = 542 is 542
+PASS detachedBody.scrollTop is 0
+PASS detachedBody.scrollLeft is 0
+PASS document.body.scrollTop = 0 is 0
+PASS document.body.scrollLeft is 0
+PASS document.body.scrollTop is 0
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 0
+PASS window.scrollY is 0
+PASS successfullyParsed is true
+
+TEST COMPLETE
+
</ins></span></pre></div>
<a id="trunkLayoutTestsfastdomElementdetachedbodyelementdoesnotscrollmainframewithscrollLefthtml"></a>
<div class="addfile"><h4>Added: trunk/LayoutTests/fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollLeft.html (0 => 182677)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollLeft.html (rev 0)
+++ trunk/LayoutTests/fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollLeft.html 2015-04-13 04:42:53 UTC (rev 182677)
</span><span class="lines">@@ -0,0 +1,51 @@
</span><ins>+<html>
+ <head>
+ <style>
+ div {
+ height: 9999px;
+ width: 9999px;
+ }
+ </style>
+ <script src="../../../resources/js-test-pre.js"></script>
+ <script>
+ jsTestIsAsync = true;
+
+ function runTest() {
+ description('Tests that, for Quirks mode, changing the scrollTop property of a fake body element does not scroll the viewport');
+
+ detachedBody = document.createElement('body');
+
+ debug("");
+ debug("Initial state.");
+ shouldBe("detachedBody.scrollTop", "0");
+ shouldBe("detachedBody.scrollLeft", "0");
+ shouldBe("document.body.scrollTop = 0", "0");
+ shouldBe("document.body.scrollLeft", "0");
+ shouldBe("document.body.scrollTop", "0");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "0");
+ shouldBe("window.scrollY", "0");
+
+ debug("");
+ debug("Setting scrollTop on the detached body element should not change the main frame's scroll position.");
+ shouldBe("detachedBody.scrollLeft = 542", "542");
+ shouldBe("detachedBody.scrollTop", "0");
+ shouldBe("detachedBody.scrollLeft", "0");
+ shouldBe("document.body.scrollTop = 0", "0");
+ shouldBe("document.body.scrollLeft", "0");
+ shouldBe("document.body.scrollTop", "0");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "0");
+ shouldBe("window.scrollY", "0");
+
+ finishJSTest();
+ }
+ </script>
+ </head>
+ <body onload="runTest()">
+ <div></div>
+ <script src="../../../resources/js-test-post.js"></script>
+ </body>
+</html>
</ins></span></pre></div>
<a id="trunkLayoutTestsfastdomElementdetachedbodyelementdoesnotscrollmainframewithscrollTopexpectedtxt"></a>
<div class="addfile"><h4>Added: trunk/LayoutTests/fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollTop-expected.txt (0 => 182677)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollTop-expected.txt (rev 0)
+++ trunk/LayoutTests/fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollTop-expected.txt 2015-04-13 04:42:53 UTC (rev 182677)
</span><span class="lines">@@ -0,0 +1,31 @@
</span><ins>+Tests that, for Quirks mode, changing the scrollTop property of a fake body element does not scroll the viewport
+
+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
+
+
+
+Initial state.
+PASS detachedBody.scrollTop is 0
+PASS detachedBody.scrollLeft is 0
+PASS document.body.scrollTop = 0 is 0
+PASS document.body.scrollLeft is 0
+PASS document.body.scrollTop is 0
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 0
+PASS window.scrollY is 0
+
+Setting scrollTop on the detached body element should not change the main frame's scroll position.
+PASS detachedBody.scrollTop = 500 is 500
+PASS document.body.scrollLeft is 0
+PASS document.body.scrollTop is 0
+PASS document.documentElement.scrollLeft is 0
+PASS document.documentElement.scrollTop is 0
+PASS window.scrollX is 0
+PASS window.scrollY is 0
+PASS detachedBody.scrollTop is 0
+PASS detachedBody.scrollLeft is 0
+PASS successfullyParsed is true
+
+TEST COMPLETE
+
</ins></span></pre></div>
<a id="trunkLayoutTestsfastdomElementdetachedbodyelementdoesnotscrollmainframewithscrollTophtml"></a>
<div class="addfile"><h4>Added: trunk/LayoutTests/fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollTop.html (0 => 182677)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollTop.html (rev 0)
+++ trunk/LayoutTests/fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollTop.html 2015-04-13 04:42:53 UTC (rev 182677)
</span><span class="lines">@@ -0,0 +1,50 @@
</span><ins>+<html>
+ <head>
+ <style>
+ div {
+ height: 9999px;
+ width: 9999px;
+ }
+ </style>
+ <script src="../../../resources/js-test-pre.js"></script>
+ <script>
+ jsTestIsAsync = true;
+
+ function runTest() {
+ description('Tests that, for Quirks mode, changing the scrollTop property of a fake body element does not scroll the viewport');
+
+ detachedBody = document.createElement('body');
+
+ debug("");
+ debug("Initial state.");
+ shouldBe("detachedBody.scrollTop", "0");
+ shouldBe("detachedBody.scrollLeft", "0");
+ shouldBe("document.body.scrollTop = 0", "0");
+ shouldBe("document.body.scrollLeft", "0");
+ shouldBe("document.body.scrollTop", "0");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "0");
+ shouldBe("window.scrollY", "0");
+
+ debug("");
+ debug("Setting scrollTop on the detached body element should not change the main frame's scroll position.");
+ shouldBe("detachedBody.scrollTop = 500", "500");
+ shouldBe("document.body.scrollLeft", "0");
+ shouldBe("document.body.scrollTop", "0");
+ shouldBe("document.documentElement.scrollLeft", "0");
+ shouldBe("document.documentElement.scrollTop", "0");
+ shouldBe("window.scrollX", "0");
+ shouldBe("window.scrollY", "0");
+ shouldBe("detachedBody.scrollTop", "0");
+ shouldBe("detachedBody.scrollLeft", "0");
+
+ finishJSTest();
+ }
+ </script>
+ </head>
+ <body onload="runTest()">
+ <div></div>
+ <script src="../../../resources/js-test-post.js"></script>
+ </body>
+</html>
</ins></span></pre></div>
<a id="trunkSourceWebCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/ChangeLog (182676 => 182677)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/ChangeLog 2015-04-13 01:29:59 UTC (rev 182676)
+++ trunk/Source/WebCore/ChangeLog 2015-04-13 04:42:53 UTC (rev 182677)
</span><span class="lines">@@ -1,3 +1,36 @@
</span><ins>+2015-04-12 Benjamin Poulain <benjamin@webkit.org>
+
+ Legacy scroll behavior on HTMLBodyElement should only apply to the first body element of a document
+ https://bugs.webkit.org/show_bug.cgi?id=143651
+
+ Reviewed by Sam Weinig.
+
+ WebKit has some very weird behaviors for the scroll methods on body. This patch
+ address the first bug: only the first body element should have the legacy behavior.
+
+ The relevant text in spec:
+ -http://dev.w3.org/csswg/cssom-view/#dom-element-scrollleft
+ -http://dev.w3.org/csswg/cssom-view/#dom-element-scrolltop
+ -http://dev.w3.org/csswg/cssom-view/#dom-element-scrollwidth
+ -http://dev.w3.org/csswg/cssom-view/#dom-element-scrollheight
+
+ Tests: fast/dom/Element/body-scrollHeight-basics-quirks.html
+ fast/dom/Element/body-scrollLeft-basics-quirks.html
+ fast/dom/Element/body-scrollTop-basics-quirks.html
+ fast/dom/Element/body-scrollWidth-basics-quirks.html
+ fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollLeft.html
+ fast/dom/Element/detached-body-element-does-not-scroll-main-frame-with-scrollTop.html
+
+ * html/HTMLBodyElement.cpp:
+ (WebCore::HTMLBodyElement::isFirstBodyElementOfDocument):
+ (WebCore::HTMLBodyElement::scrollLeft):
+ (WebCore::HTMLBodyElement::setScrollLeft):
+ (WebCore::HTMLBodyElement::scrollTop):
+ (WebCore::HTMLBodyElement::setScrollTop):
+ (WebCore::HTMLBodyElement::scrollHeight):
+ (WebCore::HTMLBodyElement::scrollWidth):
+ * html/HTMLBodyElement.h:
+
</ins><span class="cx"> 2015-04-12 Sungmann Cho <sungmann.cho@navercorp.com>
</span><span class="cx">
</span><span class="cx"> Fix trival typos related to the word "coordinate".
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlHTMLBodyElementcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/HTMLBodyElement.cpp (182676 => 182677)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/HTMLBodyElement.cpp 2015-04-13 01:29:59 UTC (rev 182676)
+++ trunk/Source/WebCore/html/HTMLBodyElement.cpp 2015-04-13 04:42:53 UTC (rev 182677)
</span><span class="lines">@@ -3,7 +3,7 @@
</span><span class="cx"> * (C) 1999 Antti Koivisto (koivisto@kde.org)
</span><span class="cx"> * (C) 2000 Simon Hausmann (hausmann@kde.org)
</span><span class="cx"> * (C) 2001 Dirk Mueller (mueller@kde.org)
</span><del>- * Copyright (C) 2004, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved.
</del><ins>+ * Copyright (C) 2004, 2006-2010, 2015 Apple Inc. All rights reserved.
</ins><span class="cx"> *
</span><span class="cx"> * This library is free software; you can redistribute it and/or
</span><span class="cx"> * modify it under the terms of the GNU Library General Public
</span><span class="lines">@@ -47,6 +47,13 @@
</span><span class="cx"> ASSERT(hasTagName(bodyTag));
</span><span class="cx"> }
</span><span class="cx">
</span><ins>+bool HTMLBodyElement::isFirstBodyElementOfDocument() const
+{
+ // By spec http://dev.w3.org/csswg/cssom-view/#the-html-body-element
+ // "The HTML body element is the first body HTML element child of the root HTML element html."
+ return document().body() == this;
+}
+
</ins><span class="cx"> Ref<HTMLBodyElement> HTMLBodyElement::create(Document& document)
</span><span class="cx"> {
</span><span class="cx"> return adoptRef(*new HTMLBodyElement(bodyTag, document));
</span><span class="lines">@@ -225,76 +232,94 @@
</span><span class="cx">
</span><span class="cx"> int HTMLBodyElement::scrollLeft()
</span><span class="cx"> {
</span><del>- document().updateLayoutIgnorePendingStylesheets();
- Frame* frame = document().frame();
- if (!frame)
- return 0;
- FrameView* view = frame->view();
- if (!view)
- return 0;
- return adjustForZoom(view->contentsScrollPosition().x(), *frame);
</del><ins>+ if (isFirstBodyElementOfDocument()) {
+ document().updateLayoutIgnorePendingStylesheets();
+ Frame* frame = document().frame();
+ if (!frame)
+ return 0;
+ FrameView* view = frame->view();
+ if (!view)
+ return 0;
+ return adjustForZoom(view->contentsScrollPosition().x(), *frame);
+ }
+ return HTMLElement::scrollLeft();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void HTMLBodyElement::setScrollLeft(int scrollLeft)
</span><span class="cx"> {
</span><del>- document().updateLayoutIgnorePendingStylesheets();
- Frame* frame = document().frame();
- if (!frame)
- return;
- FrameView* view = frame->view();
- if (!view)
- return;
- view->setScrollPosition(IntPoint(static_cast<int>(scrollLeft * frame->pageZoomFactor() * frame->frameScaleFactor()), view->scrollY()));
</del><ins>+ if (isFirstBodyElementOfDocument()) {
+ document().updateLayoutIgnorePendingStylesheets();
+ Frame* frame = document().frame();
+ if (!frame)
+ return;
+ FrameView* view = frame->view();
+ if (!view)
+ return;
+ view->setScrollPosition(IntPoint(static_cast<int>(scrollLeft * frame->pageZoomFactor() * frame->frameScaleFactor()), view->scrollY()));
+ }
+ HTMLElement::setScrollLeft(scrollLeft);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> int HTMLBodyElement::scrollTop()
</span><span class="cx"> {
</span><del>- document().updateLayoutIgnorePendingStylesheets();
- Frame* frame = document().frame();
- if (!frame)
- return 0;
- FrameView* view = frame->view();
- if (!view)
- return 0;
- return adjustForZoom(view->contentsScrollPosition().y(), *frame);
</del><ins>+ if (isFirstBodyElementOfDocument()) {
+ document().updateLayoutIgnorePendingStylesheets();
+ Frame* frame = document().frame();
+ if (!frame)
+ return 0;
+ FrameView* view = frame->view();
+ if (!view)
+ return 0;
+ return adjustForZoom(view->contentsScrollPosition().y(), *frame);
+ }
+ return HTMLElement::scrollTop();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void HTMLBodyElement::setScrollTop(int scrollTop)
</span><span class="cx"> {
</span><del>- document().updateLayoutIgnorePendingStylesheets();
- Frame* frame = document().frame();
- if (!frame)
- return;
- FrameView* view = frame->view();
- if (!view)
- return;
- view->setScrollPosition(IntPoint(view->scrollX(), static_cast<int>(scrollTop * frame->pageZoomFactor() * frame->frameScaleFactor())));
</del><ins>+ if (isFirstBodyElementOfDocument()) {
+ document().updateLayoutIgnorePendingStylesheets();
+ Frame* frame = document().frame();
+ if (!frame)
+ return;
+ FrameView* view = frame->view();
+ if (!view)
+ return;
+ view->setScrollPosition(IntPoint(view->scrollX(), static_cast<int>(scrollTop * frame->pageZoomFactor() * frame->frameScaleFactor())));
+ }
+ return HTMLElement::setScrollTop(scrollTop);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> int HTMLBodyElement::scrollHeight()
</span><span class="cx"> {
</span><del>- // Update the document's layout.
- document().updateLayoutIgnorePendingStylesheets();
- Frame* frame = document().frame();
- if (!frame)
- return 0;
- FrameView* view = frame->view();
- if (!view)
- return 0;
- return adjustForZoom(view->contentsHeight(), *frame);
</del><ins>+ if (isFirstBodyElementOfDocument()) {
+ // Update the document's layout.
+ document().updateLayoutIgnorePendingStylesheets();
+ Frame* frame = document().frame();
+ if (!frame)
+ return 0;
+ FrameView* view = frame->view();
+ if (!view)
+ return 0;
+ return adjustForZoom(view->contentsHeight(), *frame);
+ }
+ return HTMLElement::scrollHeight();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> int HTMLBodyElement::scrollWidth()
</span><span class="cx"> {
</span><del>- // Update the document's layout.
- document().updateLayoutIgnorePendingStylesheets();
- Frame* frame = document().frame();
- if (!frame)
- return 0;
- FrameView* view = frame->view();
- if (!view)
- return 0;
- return adjustForZoom(view->contentsWidth(), *frame);
</del><ins>+ if (isFirstBodyElementOfDocument()) {
+ // Update the document's layout.
+ document().updateLayoutIgnorePendingStylesheets();
+ Frame* frame = document().frame();
+ if (!frame)
+ return 0;
+ FrameView* view = frame->view();
+ if (!view)
+ return 0;
+ return adjustForZoom(view->contentsWidth(), *frame);
+ }
+ return HTMLElement::scrollWidth();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void HTMLBodyElement::addSubresourceAttributeURLs(ListHashSet<URL>& urls) const
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlHTMLBodyElementh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/HTMLBodyElement.h (182676 => 182677)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/HTMLBodyElement.h 2015-04-13 01:29:59 UTC (rev 182676)
+++ trunk/Source/WebCore/html/HTMLBodyElement.h 2015-04-13 04:42:53 UTC (rev 182677)
</span><span class="lines">@@ -39,6 +39,8 @@
</span><span class="cx"> private:
</span><span class="cx"> HTMLBodyElement(const QualifiedName&, Document&);
</span><span class="cx">
</span><ins>+ bool isFirstBodyElementOfDocument() const;
+
</ins><span class="cx"> virtual void parseAttribute(const QualifiedName&, const AtomicString&) override;
</span><span class="cx"> virtual bool isPresentationAttribute(const QualifiedName&) const override;
</span><span class="cx"> virtual void collectStyleForPresentationAttribute(const QualifiedName&, const AtomicString&, MutableStyleProperties&) override;
</span></span></pre>
</div>
</div>
</body>
</html>