Attachment 370337 Details for Bug 198009 – Patch

[patch] Patch

bug-198009-20190521162942.patch (text/plain), 5.39 KB, created by Ali Juma on 2019-05-21 13:29:43 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Ali Juma

Created: 2019-05-21 13:29:43 PDT

Size: 5.39 KB

patch

obsolete

>Subversion Revision: 245569
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 48edb2b1c23095081c423b679d51a45fb19f1dc9..622500fcbb10d89991ec794fed10a7170c4276a0 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,18 @@
>+2019-05-21  Ali Juma  <ajuma@chromium.org>
>+
>+        Intersection Observer: bounding client rect is wrong for an inline element
>+        https://bugs.webkit.org/show_bug.cgi?id=198009
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Make target rect computation for inline elements equivalent to the logic used
>+        when computing bounding client rects.
>+
>+        Test: imported/w3c/web-platform-tests/intersection-observer/inline-with-block-child-client-rect.html
>+
>+        * dom/Document.cpp:
>+        (WebCore::computeIntersectionState):
>+
> 2019-05-21  Antti Koivisto  <antti@apple.com>
> 
>         RTL/overflow scroll tests fail with async overflow enabled
>diff --git a/Source/WebCore/dom/Document.cpp b/Source/WebCore/dom/Document.cpp
>index 92605852d261272f53fbd52638f111cabd9c64c6..d30d89d09a66a8d870b010e08a4ac016f1c0c3e5 100644
>--- a/Source/WebCore/dom/Document.cpp
>+++ b/Source/WebCore/dom/Document.cpp
>@@ -7349,9 +7349,14 @@ static Optional<IntersectionObservationState> computeIntersectionState(FrameView
>     LayoutRect localTargetBounds;
>     if (is<RenderBox>(*targetRenderer))
>         localTargetBounds = downcast<RenderBox>(targetRenderer)->borderBoundingBox();
>-    else if (is<RenderInline>(targetRenderer))
>-        localTargetBounds = downcast<RenderInline>(targetRenderer)->linesBoundingBox();
>-    else if (is<RenderLineBreak>(targetRenderer))
>+    else if (is<RenderInline>(targetRenderer)) {
>+        Vector<FloatQuad> absoluteTargetQuads;
>+        targetRenderer->absoluteQuads(absoluteTargetQuads);
>+        FloatRect absoluteTargetBounds;
>+        for (auto quad : absoluteTargetQuads)
>+            absoluteTargetBounds.unite(quad.boundingBox());
>+        localTargetBounds = enclosingLayoutRect(targetRenderer->absoluteToLocalQuad(absoluteTargetBounds).boundingBox());
>+    } else if (is<RenderLineBreak>(targetRenderer))
>         localTargetBounds = downcast<RenderLineBreak>(targetRenderer)->linesBoundingBox();
> 
>     Optional<LayoutRect> rootLocalTargetRect;
>diff --git a/LayoutTests/imported/w3c/ChangeLog b/LayoutTests/imported/w3c/ChangeLog
>index b9152e70ec7431af4639ac248704d180f93bffd1..38e3f46f77c0f1fb59d28512ba4a92aa5289b464 100644
>--- a/LayoutTests/imported/w3c/ChangeLog
>+++ b/LayoutTests/imported/w3c/ChangeLog
>@@ -1,3 +1,16 @@
>+2019-05-21  Ali Juma  <ajuma@chromium.org>
>+
>+        Intersection Observer: bounding client rect is wrong for an inline element
>+        https://bugs.webkit.org/show_bug.cgi?id=198009
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Import test added upstream in
>+        https://github.com/web-platform-tests/wpt/commit/cf40205c68f739ae23429439de13d125e3df9f7a
>+
>+        * web-platform-tests/intersection-observer/inline-with-block-child-client-rect-expected.txt: Added.
>+        * web-platform-tests/intersection-observer/inline-with-block-child-client-rect.html: Added.
>+
> 2019-05-17  Rob Buis  <rbuis@igalia.com>
> 
>         Implement imagesrcset and imagesizes attributes on link rel=preload
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/intersection-observer/inline-with-block-child-client-rect-expected.txt b/LayoutTests/imported/w3c/web-platform-tests/intersection-observer/inline-with-block-child-client-rect-expected.txt
>new file mode 100644
>index 0000000000000000000000000000000000000000..17c97453febc411ce1962d5f27cabc551c21a6ca
>--- /dev/null
>+++ b/LayoutTests/imported/w3c/web-platform-tests/intersection-observer/inline-with-block-child-client-rect-expected.txt
>@@ -0,0 +1,5 @@
>+ 
>+
>+PASS Inline target containing a block child 
>+PASS First rAF 
>+
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/intersection-observer/inline-with-block-child-client-rect.html b/LayoutTests/imported/w3c/web-platform-tests/intersection-observer/inline-with-block-child-client-rect.html
>new file mode 100644
>index 0000000000000000000000000000000000000000..81a8fd125691bbfca73ff73e861e80ecf69818f8
>--- /dev/null
>+++ b/LayoutTests/imported/w3c/web-platform-tests/intersection-observer/inline-with-block-child-client-rect.html
>@@ -0,0 +1,46 @@
>+<!DOCTYPE html>
>+<script src="/resources/testharness.js"></script>
>+<script src="/resources/testharnessreport.js"></script>
>+<script src="./resources/intersection-observer-test-utils.js"></script>
>+
>+<style>
>+pre, #log {
>+  position: absolute;
>+  top: 120px;
>+  left: 0;
>+}
>+#target {
>+  display: inline;
>+}
>+</style>
>+
>+<div id="target">
>+  <div>
>+    <img width=100 height=100 />
>+  </div>
>+</div>
>+
>+<script>
>+var vw = document.documentElement.clientWidth;
>+var vh = document.documentElement.clientHeight;
>+var entries = [];
>+var target;
>+
>+runTestCycle(function() {
>+  target = document.getElementById("target");
>+  assert_true(!!target, "target exists");
>+  var observer = new IntersectionObserver(function(changes) {
>+    entries = entries.concat(changes)
>+  });
>+  observer.observe(target);
>+  entries = entries.concat(observer.takeRecords());
>+  assert_equals(entries.length, 0, "No initial notifications.");
>+  runTestCycle(step0, "First rAF");
>+}, "Inline target containing a block child");
>+
>+function step0() {
>+  assert_equals(entries.length, 1);
>+  checkRect(entries[0].boundingClientRect, clientBounds(target),
>+            "entry.boundingClientRect == target.getBoundingClientRect()");
>+}
>+</script>

Actions: View | Formatted Diff | Diff

Attachments on bug 198009: 370170 | 370337 | 370396 | 370402 | 370403 | 370405 | 370411 | 370413 | 370420