Title: [270119] trunk/Source/WebCore
Revision
270119
Author
[email protected]
Date
2020-11-20 09:47:33 -0800 (Fri, 20 Nov 2020)

Log Message

[LFC][Integration] Hit test runs in reverse order
https://bugs.webkit.org/show_bug.cgi?id=219205

Reviewed by Zalan Bujtas.

In overlap case the latter boxes should be hit first.

* layout/integration/LayoutIntegrationLineLayout.cpp:
(WebCore::LayoutIntegration::LineLayout::hitTest):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (270118 => 270119)


--- trunk/Source/WebCore/ChangeLog	2020-11-20 17:46:27 UTC (rev 270118)
+++ trunk/Source/WebCore/ChangeLog	2020-11-20 17:47:33 UTC (rev 270119)
@@ -1,3 +1,15 @@
+2020-11-20  Antti Koivisto  <[email protected]>
+
+        [LFC][Integration] Hit test runs in reverse order
+        https://bugs.webkit.org/show_bug.cgi?id=219205
+
+        Reviewed by Zalan Bujtas.
+
+        In overlap case the latter boxes should be hit first.
+
+        * layout/integration/LayoutIntegrationLineLayout.cpp:
+        (WebCore::LayoutIntegration::LineLayout::hitTest):
+
 2020-11-20  Youenn Fablet  <[email protected]>
 
         Make webrtc/captureCanvas-webrtc-software-h264-*.html tests use codecs in WebProcess

Modified: trunk/Source/WebCore/layout/integration/LayoutIntegrationLineLayout.cpp (270118 => 270119)


--- trunk/Source/WebCore/layout/integration/LayoutIntegrationLineLayout.cpp	2020-11-20 17:46:27 UTC (rev 270118)
+++ trunk/Source/WebCore/layout/integration/LayoutIntegrationLineLayout.cpp	2020-11-20 17:47:33 UTC (rev 270119)
@@ -449,7 +449,7 @@
     auto& inlineContent = *m_inlineContent;
 
     // FIXME: This should do something efficient to find the run range.
-    for (auto& run : inlineContent.runs) {
+    for (auto& run : WTF::makeReversedRange(inlineContent.runs)) {
         auto runRect = Layout::toLayoutRect(run.rect());
         runRect.moveBy(accumulatedOffset);
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to