Title: [254471] trunk/Source/WebCore
Revision
254471
Author
[email protected]
Date
2020-01-13 16:28:39 -0800 (Mon, 13 Jan 2020)

Log Message

Unreviewed build fix for PlayStation and clang-cl.

clang for Windows (< v10.0.0) cannot destructure a const class. See also r249524.

* rendering/InlineTextBox.cpp:
(WebCore::InlineTextBox::collectMarkedTextsForHighlights const):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (254470 => 254471)


--- trunk/Source/WebCore/ChangeLog	2020-01-14 00:15:03 UTC (rev 254470)
+++ trunk/Source/WebCore/ChangeLog	2020-01-14 00:28:39 UTC (rev 254471)
@@ -1,3 +1,12 @@
+2020-01-13  Ross Kirsling  <[email protected]>
+
+        Unreviewed build fix for PlayStation and clang-cl.
+
+        clang for Windows (< v10.0.0) cannot destructure a const class. See also r249524.
+
+        * rendering/InlineTextBox.cpp:
+        (WebCore::InlineTextBox::collectMarkedTextsForHighlights const):
+
 2020-01-13  Truitt Savell  <[email protected]>
 
         Unreviewed, rolling out r254463.

Modified: trunk/Source/WebCore/rendering/InlineTextBox.cpp (254470 => 254471)


--- trunk/Source/WebCore/rendering/InlineTextBox.cpp	2020-01-14 00:15:03 UTC (rev 254470)
+++ trunk/Source/WebCore/rendering/InlineTextBox.cpp	2020-01-14 00:28:39 UTC (rev 254471)
@@ -1039,11 +1039,11 @@
     Vector<MarkedText> markedTexts;
     auto& parentRenderer = parent()->renderer();
     auto& parentStyle = parentRenderer.style();
-    for (auto& [highlightKey, highlightGroup] : renderer().document().highlightMap().map()) {
-        auto renderStyle = parentRenderer.getUncachedPseudoStyle({ PseudoId::Highlight, highlightKey }, &parentStyle);
+    for (auto& highlight : renderer().document().highlightMap().map()) {
+        auto renderStyle = parentRenderer.getUncachedPseudoStyle({ PseudoId::Highlight, highlight.key }, &parentStyle);
         if (!renderStyle)
             continue;
-        for (auto& staticRange : highlightGroup->ranges()) {
+        for (auto& staticRange : highlight.value->ranges()) {
             Position startPos = createLegacyEditingPosition(staticRange->startContainer(), staticRange->startOffset());
             Position endPos = createLegacyEditingPosition(staticRange->endContainer(), staticRange->endOffset());
 
@@ -1059,7 +1059,7 @@
                 highlightData.setContext({startRenderer, endRenderer, static_cast<unsigned>(startOffset), static_cast<unsigned>(endOffset)});
                 auto [highlightStart, highlightEnd] = highlightStartEnd(highlightData);
                 if (highlightStart < highlightEnd)
-                    markedTexts.append({ highlightStart, highlightEnd, MarkedText::Highlight, nullptr, highlightKey });
+                    markedTexts.append({ highlightStart, highlightEnd, MarkedText::Highlight, nullptr, highlight.key });
             }
         }
     }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to