[Webkit-unassigned] [Bug 149264] IFrame scrolling=yes is ignored in iOS Safari

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Apr 28 00:27:09 PDT 2017


https://bugs.webkit.org/show_bug.cgi?id=149264

--- Comment #9 from Frédéric Wang (:fredw) <fred.wang at free.fr> ---
Created attachment 308506

  --> https://bugs.webkit.org/attachment.cgi?id=308506&action=review

Part 1 - Do not flatten iframe

This is a preliminary patch to remove flattening for iframe. However, it's not enough to make scrolling work (I'm still investigating this...).

In particular, we have an issue similar to bug 163911. A quick workaround to make scrolling work with the Find operation is:

--- a/Source/WebCore/editing/Editor.cpp
+++ b/Source/WebCore/editing/Editor.cpp
@@ -3122,2 +3122,2 @@ bool Editor::findString(const String& target, FindOptions options)
-    if (!(options & DoNotRevealSelection))
-        m_frame.selection().revealSelection();
+    // if (!(options & DoNotRevealSelection))
+    m_frame.selection().revealSelection();
--- a/Source/WebKit2/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp
+++ b/Source/WebKit2/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp
@@ -1370,3 +1370,4 @@ void WebFrameLoaderClient::transitionToCommittedForNewPage()
 #if PLATFORM(IOS)
-    m_frame->coreFrame()->view()->setDelegatesScrolling(true);
+    if (m_frame->coreFrame()->isMainFrame())
+        m_frame->coreFrame()->view()->setDelegatesScrolling(true);
 #endif

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20170428/18909222/attachment.html>


More information about the webkit-unassigned mailing list