Title: [265474] branches/safari-610.1.25.10-branch/Source/WebKitLegacy/ios
- Revision
- 265474
- Author
- [email protected]
- Date
- 2020-08-10 16:48:07 -0700 (Mon, 10 Aug 2020)
Log Message
Cherry-pick r265424. rdar://problem/66803405
REGRESSION (r265176): Cannot make or extend selections in iOS WebKitLegacy
https://bugs.webkit.org/show_bug.cgi?id=215319
<rdar://problem/66688560>
Reviewed by Devin Rousso.
* WebCoreSupport/WebVisiblePosition.mm:
(+[DOMRange rangeForFirstPosition:second:]):
We want to ensure that firstPosition is *before* secondPosition, so
the change in r265176 got the condition backwards. Swap it.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@265424 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Modified Paths
Diff
Modified: branches/safari-610.1.25.10-branch/Source/WebKitLegacy/ios/ChangeLog (265473 => 265474)
--- branches/safari-610.1.25.10-branch/Source/WebKitLegacy/ios/ChangeLog 2020-08-10 23:48:05 UTC (rev 265473)
+++ branches/safari-610.1.25.10-branch/Source/WebKitLegacy/ios/ChangeLog 2020-08-10 23:48:07 UTC (rev 265474)
@@ -1,5 +1,35 @@
2020-08-10 Alan Coon <[email protected]>
+ Cherry-pick r265424. rdar://problem/66803405
+
+ REGRESSION (r265176): Cannot make or extend selections in iOS WebKitLegacy
+ https://bugs.webkit.org/show_bug.cgi?id=215319
+ <rdar://problem/66688560>
+
+ Reviewed by Devin Rousso.
+
+ * WebCoreSupport/WebVisiblePosition.mm:
+ (+[DOMRange rangeForFirstPosition:second:]):
+ We want to ensure that firstPosition is *before* secondPosition, so
+ the change in r265176 got the condition backwards. Swap it.
+
+ git-svn-id: https://svn.webkit.org/repository/webkit/trunk@265424 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+ 2020-08-10 Tim Horton <[email protected]>
+
+ REGRESSION (r265176): Cannot make or extend selections in iOS WebKitLegacy
+ https://bugs.webkit.org/show_bug.cgi?id=215319
+ <rdar://problem/66688560>
+
+ Reviewed by Devin Rousso.
+
+ * WebCoreSupport/WebVisiblePosition.mm:
+ (+[DOMRange rangeForFirstPosition:second:]):
+ We want to ensure that firstPosition is *before* secondPosition, so
+ the change in r265176 got the condition backwards. Swap it.
+
+2020-08-10 Alan Coon <[email protected]>
+
Cherry-pick r265255. rdar://problem/66644016
Try to fix Catalyst build
Modified: branches/safari-610.1.25.10-branch/Source/WebKitLegacy/ios/WebCoreSupport/WebVisiblePosition.mm (265473 => 265474)
--- branches/safari-610.1.25.10-branch/Source/WebKitLegacy/ios/WebCoreSupport/WebVisiblePosition.mm 2020-08-10 23:48:05 UTC (rev 265473)
+++ branches/safari-610.1.25.10-branch/Source/WebKitLegacy/ios/WebCoreSupport/WebVisiblePosition.mm 2020-08-10 23:48:07 UTC (rev 265474)
@@ -478,7 +478,7 @@
{
auto firstVP = [first _visiblePosition];
auto secondVP = [second _visiblePosition];
- if (firstVP < secondVP)
+ if (secondVP < firstVP)
std::swap(firstVP, secondVP);
return kit(makeSimpleRange(firstVP, secondVP));
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
