Bug 144421

Summary: [iOS] tapping in the middle of a word in editable content should not select the word.
Product: WebKit Reporter: Enrica Casucci <enrica>
Component: WebKit2Assignee: Enrica Casucci <enrica>
Status: RESOLVED FIXED    
Severity: Normal CC: benjamin, sam, thorton
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch thorton: review+

Description Enrica Casucci 2015-04-29 17:26:13 PDT
We want to revert it to the original behavior.

rdar://problem/20710948
Comment 1 Enrica Casucci 2015-04-29 17:29:10 PDT
Created attachment 252013 [details]
Patch
Comment 2 Tim Horton 2015-04-29 17:33:23 PDT
Comment on attachment 252013 [details]
Patch

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

> Source/WebKit2/ChangeLog:9
> +        Reverting to the original behavior.

original before a specific revision?

> Source/WebKit2/WebProcess/WebPage/ios/WebPageIOS.mm:969
> +            result = wordRange->startPosition();

Can this range ever be null? I know Range likes to be null at inopportune times.
Comment 3 Enrica Casucci 2015-04-30 10:50:27 PDT
(In reply to comment #2)
> Comment on attachment 252013 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=252013&action=review
> 
> > Source/WebKit2/ChangeLog:9
> > +        Reverting to the original behavior.
> 
> original before a specific revision?
> 
> > Source/WebKit2/WebProcess/WebPage/ios/WebPageIOS.mm:969
> > +            result = wordRange->startPosition();
> 
> Can this range ever be null? I know Range likes to be null at inopportune
> times.

Yes. I'll add a check. Thanks!
Comment 4 Enrica Casucci 2015-04-30 11:22:31 PDT
Committed revision 183635.