<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - AX: new lines in content editable elements don't notify accessibility"
href="https://bugs.webkit.org/show_bug.cgi?id=153361#c37">Comment # 37</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - AX: new lines in content editable elements don't notify accessibility"
href="https://bugs.webkit.org/show_bug.cgi?id=153361">bug 153361</a>
from <span class="vcard"><a class="email" href="mailto:d_russell@apple.com" title="Doug Russell <d_russell@apple.com>"> <span class="fn">Doug Russell</span></a>
</span></b>
<pre>
<span class="quote">> > We should really use the version of indexForVisiblePosition the rest of
> > editing code uses.
> > The one that takes VisiblePosition as the first argument and the scope as
> > the second argument.
> > The variant you're using here won't work in many cases.
>
> Good to know, this was some of the code I was hoping to get feedback on.
>
> This version? int indexForVisiblePosition(const VisiblePosition&
> visiblePosition, RefPtr<ContainerNode>& scope)
>
> How do I determine what to pass to scope?
> </span >
I was misreading the method signature. Good on this.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>