Bug 134992

Summary: REGRESSION (iOS WebKit2): Cannot scroll while dragging a selection
Product: WebKit Reporter: Enrica Casucci <enrica>
Component: WebKit2Assignee: Enrica Casucci <enrica>
Status: RESOLVED FIXED    
Severity: Normal Keywords: InRadar
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: iPhone / iPad   
OS: Unspecified   
Attachments:
Description Flags
Patch benjamin: review+

Description Enrica Casucci 2014-07-16 15:59:16 PDT
<rdar://problem/17528020>
Comment 1 Enrica Casucci 2014-07-16 16:05:31 PDT
Created attachment 235032 [details]
Patch
Comment 2 Benjamin Poulain 2014-07-16 16:39:44 PDT
Comment on attachment 235032 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=235032&action=review

rs=me

> Source/WebKit2/ChangeLog:14
> +        This patch exposes the scroller and the visible content rect so that
> +        UIKit can implement autoscroll when dragging the selections.
> +        It also changes that way we do hit testing to allow hit test outside
> +        the clipping region and fixes the way we compute the selection rectangle
> +        for the block selection, ensuring that we consider also non text elements
> +        like images.

I think this would have been better as 3 patches.

> Source/WebKit2/UIProcess/ios/WKContentViewInteraction.mm:352
> +- (UIScrollView *)scroller

Shouldn't we just override UIKit's _scroller method?

> Source/WebKit2/UIProcess/ios/WKContentViewInteraction.mm:357
> +- (CGRect)visibleRect

I would prefer keeping the name unobscuredRect. Having [WKContentView visibleRect] returning the unobscuredRect seems very confusing.

> Source/WebKit2/WebProcess/WebPage/ios/WebPageIOS.mm:718
> +    for (unsigned i = 0; i < size; i++) {

WebKit style:
i++ -> ++i
Comment 3 Enrica Casucci 2014-07-16 17:01:03 PDT
Committed revision 171159.