Changeset 270119 in webkit


Ignore:
Timestamp:
Nov 20, 2020 9:47:33 AM (3 years ago)
Author:
Antti Koivisto
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):

Location:
trunk/Source/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r270117 r270119  
     12020-11-20  Antti Koivisto  <antti@apple.com>
     2
     3        [LFC][Integration] Hit test runs in reverse order
     4        https://bugs.webkit.org/show_bug.cgi?id=219205
     5
     6        Reviewed by Zalan Bujtas.
     7
     8        In overlap case the latter boxes should be hit first.
     9
     10        * layout/integration/LayoutIntegrationLineLayout.cpp:
     11        (WebCore::LayoutIntegration::LineLayout::hitTest):
     12
    1132020-11-20  Youenn Fablet  <youenn@apple.com>
    214
  • trunk/Source/WebCore/layout/integration/LayoutIntegrationLineLayout.cpp

    r270109 r270119  
    450450
    451451    // FIXME: This should do something efficient to find the run range.
    452     for (auto& run : inlineContent.runs) {
     452    for (auto& run : WTF::makeReversedRange(inlineContent.runs)) {
    453453        auto runRect = Layout::toLayoutRect(run.rect());
    454454        runRect.moveBy(accumulatedOffset);
Note: See TracChangeset for help on using the changeset viewer.