Attachment 341146 Details for Bug 185926 – Patch

[patch] Patch

bug-185926-20180523164419.patch (text/plain), 5.44 KB, created by Megan Gardner on 2018-05-23 16:44:19 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Megan Gardner

Created: 2018-05-23 16:44:19 PDT

Size: 5.44 KB

patch

obsolete

>Subversion Revision: 231854
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index 51940422374a2719afe00b023c72a9af669ead17..649cd2db8afc102d831adfab6c29d1a8ca2f9b55 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,21 @@
>+2018-05-23  Megan Gardner  <megan_gardner@apple.com>
>+
>+        Fix Issues with Loupe Gesture
>+        https://bugs.webkit.org/show_bug.cgi?id=185926
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        The loupe gesture was not giving us the correct selection in some situations.
>+
>+        * UIProcess/ios/WKContentViewInteraction.mm:
>+        (-[WKContentView gestureRecognizer:canBePreventedByGestureRecognizer:]):
>+        (-[WKContentView gestureRecognizer:shouldRecognizeSimultaneouslyWithGestureRecognizer:]):
>+        (-[WKContentView setSelectedTextRange:]):
>+        * WebProcess/WebPage/WebPage.h:
>+        * WebProcess/WebPage/ios/WebPageIOS.mm:
>+        (WebKit::WebPage::selectWithGesture):
>+        (WebKit::WebPage::clearSelection):
>+
> 2018-05-16  Fujii Hironori  <Hironori.Fujii@sony.com>
> 
>         [Win] Implement WebPage::handleEditingKeyboardEvent
>diff --git a/Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm b/Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm
>index 28077747f84c901e437cb75d3992e30af569f6c1..192f3afaaccaef85b294b10017539afd72da0b73 100644
>--- a/Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm
>+++ b/Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm
>@@ -1320,6 +1320,11 @@ static NSValue *nsSizeForTapHighlightBorderRadius(WebCore::IntSize borderRadius,
> #if __IPHONE_OS_VERSION_MIN_REQUIRED >= 120000
>     isForcePressGesture = (preventingGestureRecognizer == _textSelectionAssistant.get().forcePressGesture);
> #endif
>+#if ENABLE(MINIMAL_SIMULATOR)
>+    if ((preventingGestureRecognizer == _textSelectionAssistant.get().loupeGesture) && (preventedGestureRecognizer == _highlightLongPressGestureRecognizer || preventedGestureRecognizer == _longPressGestureRecognizer || preventedGestureRecognizer == _textSelectionAssistant.get().forcePressGesture))
>+        return YES;
>+#endif
>+    
>     if ((preventingGestureRecognizer == _textSelectionAssistant.get().loupeGesture || isForcePressGesture || [_webSelectionAssistant isSelectionGestureRecognizer:preventingGestureRecognizer]) && (preventedGestureRecognizer == _highlightLongPressGestureRecognizer || preventedGestureRecognizer == _longPressGestureRecognizer))
>         return NO;
> 
>@@ -1344,6 +1349,10 @@ static inline bool isSamePair(UIGestureRecognizer *a, UIGestureRecognizer *b, UI
>     if (isSamePair(gestureRecognizer, otherGestureRecognizer, _highlightLongPressGestureRecognizer.get(), _webSelectionAssistant.get().selectionLongPressRecognizer))
>         return YES;
> #if __IPHONE_OS_VERSION_MIN_REQUIRED >= 120000
>+#if ENABLE(MINIMAL_SIMULATOR)
>+    if (isSamePair(gestureRecognizer, otherGestureRecognizer, _textSelectionAssistant.get().loupeGesture, _textSelectionAssistant.get().forcePressGesture))
>+        return YES;
>+#endif
>     if (isSamePair(gestureRecognizer, otherGestureRecognizer, _highlightLongPressGestureRecognizer.get(), _textSelectionAssistant.get().forcePressGesture))
>         return YES;
> #endif
>@@ -3196,6 +3205,10 @@ static void selectionChangedWithTouch(WKContentView *view, const WebCore::IntPoi
> 
> - (void)setSelectedTextRange:(UITextRange *)range
> {
>+#if ENABLE(MINIMAL_SIMULATOR)
>+    if (!range)
>+        [self clearSelection];
>+#endif
>     if (hasAssistedNode(_assistedNodeInformation) && !range)
>         [self clearSelection];
> }
>diff --git a/Source/WebKit/WebProcess/WebPage/WebPage.h b/Source/WebKit/WebProcess/WebPage/WebPage.h
>index 4da75aea52a35ef66cf1a9996084372dce3d22ef..d30e4ab380eab863c578e9baf601818bc24b7ee8 100644
>--- a/Source/WebKit/WebProcess/WebPage/WebPage.h
>+++ b/Source/WebKit/WebProcess/WebPage/WebPage.h
>@@ -1541,6 +1541,10 @@ private:
> #if PLATFORM(IOS)
>     bool m_allowsMediaDocumentInlinePlayback { false };
> #endif
>+    
>+#if PLATFORM(IOS)
>+    RefPtr<WebCore::Range> m_startingGestureRange;
>+#endif
> 
> #if ENABLE(FULLSCREEN_API)
>     RefPtr<WebFullScreenManager> m_fullScreenManager;
>diff --git a/Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm b/Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm
>index e57c86fad372ef0f8babef079c32ad80b1e2e7be..f74304afe9079c0fa5c8035b78e084f6e08fbbe0 100644
>--- a/Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm
>+++ b/Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm
>@@ -1141,7 +1141,16 @@ void WebPage::selectWithGesture(const IntPoint& point, uint32_t granularity, uin
>         if (position.rootEditableElement())
>             range = Range::create(*frame.document(), position, position);
>         else
>+#if ENABLE(MINIMAL_SIMULATOR)
>+            if (m_startingGestureRange) {
>+                range = Range::create(*frame.document(), m_startingGestureRange->startPosition(), position);
>+                if (range->collapsed())
>+                    range = Range::create(*frame.document(), position, m_startingGestureRange->startPosition());
>+            } else
>+                m_startingGestureRange = Range::create(*frame.document(), position, position);
>+#else
>             range = wordRangeFromPosition(position);
>+#endif
>         break;
> 
>     case GestureType::TapAndAHalf:
>@@ -1291,6 +1300,7 @@ static RefPtr<Range> rangeAtWordBoundaryForPosition(Frame* frame, const VisibleP
> }
> 
> void WebPage::clearSelection(){
>+    m_startingGestureRange = nullptr;
>     m_currentBlockSelection = nullptr;
>     m_page->focusController().focusedOrMainFrame().selection().clear();
> }

Actions: View | Formatted Diff | Diff

Attachments on bug 185926: 341146 | 341157 | 341197 | 341222