[Webkit-unassigned] [Bug 25444] SelectionStart, selectionEnd properties return wrong values when the selection is in a form input

[bugzilla-daemon at webkit.org]

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





--- Comment #14 from Julie Jeongeun Kim <jiyuluna at gmail.com>  2010-11-05 19:49:48 PST ---
Please check my inline comment.

(In reply to comment #13)
> (From update of attachment 73054 [details])
> Not need to call get().  You can do m_innerText->contains(n)
Yes. I'll update it.

> > WebCore/rendering/RenderTextControl.cpp:300
> > +    if (!indexPosition.node() || (indexPosition.node()->rootEditableElement() != m_innerText && !isSelectableElement(indexPosition.node())))
> 
> Can we also incorporate the condition indexPosition.node()->rootEditableElement() != m_innerText into isSelectableElement?  I think that'll improve the clarity here.
Sure, I'll do.

> > LayoutTests/fast/forms/selection-start-end-readonly-expected.txt:1
> > +https://bugs.webkit.org/show_bug.cgi?id=25444
> 
> I don't think we typically add a URL to bug because it's pretty clear which bug added this test if you see the svn log / trac.  But I'd like to see some descriptions on what this test is testing.  It's not clear at all from this output what output is expected.  If this test fails in the future, how do we know what is the correct behavior?
OK. I'll remove the link and add more detail description for test.

> > LayoutTests/fast/forms/selection-start-end-readonly-expected.txt:9
> > +Set Range: 0, 10
> > +[Result] Start: 0, End: 10
> > +
> > +Set Range: 0, 9
> > +[Result] Start: 0, End: 9
> 
> Could you make this a script test and output PASS/FAIL?
You mean the result from each test part? 
For instance,
+Set Range: 0, 10
+[Result] Start: 0, End: 10
+PASS
+Set Range: 0, 9
+[Result] Start: 0, End: 9
+FAIL
Could you explain what you expect?

Thanks.

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