[Webkit-unassigned] [Bug 48037] Triple click does not select whole line for mixed contenteditable regions

[bugzilla-daemon at webkit.org]

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





--- Comment #19 from kalman at chromium.org  2010-10-25 22:38:21 PST ---
(From update of attachment 71839)
View in context: https://bugs.webkit.org/attachment.cgi?id=71839&action=review

>> LayoutTests/editing/selection/triple-click-across-editability-boundaries.html:58
>> +    eventSender.mouseUp();
> 
> This might be too fast.  You may need to add some delays between each downs & ups.

Yeah I copied it from the other triple click test, seems to work?

>> WebCore/editing/visible_units.cpp:817
>> +        // We also do this in endOfParagraph, and might want to do it in other cases too.
> 
> Why FIXME?  Isn't new code correct?  I don't think we should add FIXME to indicate that we should fix other places.  And I note that there are other places in editing where we skip empty text nodes.  If you've found other code that may require skipping empty text nodes, then you should add FIXME to those places and not here.

Just copied the existing FIXME -- I don't know why that FIXME was added (http://trac.webkit.org/changeset/8786).
I took the "such as startOfParagraph" to mean that there might be other cases too besides startOfParagraph, but I will delete it if you want.  I'll definitely delete the one here though.

>> WebCore/editing/visible_units.h:37
>> +enum EditingBoundaryCrossingRule { CanCrossEditingBoundary, CannotCrossEditingBoundary };
> 
> Mn... you probably don't want to redefine enum here.

Yeah I was wondering about that.  So Position::EditingBoundaryCrossingRule etc everywhere?  Or define a new similarly-named enum?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.