<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Mar 12, 2013 at 6:48 AM, Ryosuke Niwa <span dir="ltr">&lt;<a href="mailto:rniwa@webkit.org" target="_blank">rniwa@webkit.org</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks for the clarification.<br><br><div class="gmail_quote"><div class="im">On Mon, Mar 11, 2013 at 9:35 AM, Rouslan Solomakhin <span dir="ltr">&lt;<a href="mailto:rouslan+webkit@chromium.org" target="_blank">rouslan+webkit@chromium.org</a>&gt;</span> wrote:<br>


</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="im"><div>On Sun, Mar 10, 2013 at 10:47 PM, Ryosuke Niwa <span dir="ltr">&lt;<a href="mailto:rniwa@webkit.org" target="_blank">rniwa@webkit.org</a>&gt;</span> wrote:<br>


</div></div><div class="gmail_extra"><div class="gmail_quote"><div class="im"><div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>On Thu, Mar 7, 2013 at 2:58 PM, Rouslan Solomakhin <span dir="ltr">&lt;<a href="mailto:rouslan+webkit@chromium.org" target="_blank">rouslan+webkit@chromium.org</a>&gt;</span> wrote:</div>



<div class="gmail_quote"><div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div dir="ltr"><div>The contextual spellcheck in Chromium needs to identify the spellcheck request that added spelling markers to the document.</div></div></blockquote><div><br></div></div><div>What is contextual spellcheck?</div>



</div></blockquote><div><br></div></div></div><div class="im"><div>Contextual spellcheck in Chromium takes a sentence like &quot;The duck says quick&quot; and marks the word &quot;quick&quot; as contextually misspelled. The spellcheck would suggest &quot;quack&quot; instead of &quot;quick&quot;. Although all words are spelled correctly, the word &quot;quick&quot; makes less sense than &quot;quack&quot; in the context of that sentence.</div>


</div><div>
<div> <span style="color:rgb(80,0,80)"> </span></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote"><div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



<div dir="ltr"><div>This information will be used to provide a feedback loop to the contextial spellcheck. I was thinking about adding an ID field to DocumentMarkerDescription detail. Is this a correct approach?</div>

</div></blockquote><div><br></div></div><div class="im"><div>I&#39;m with morrita on this one but it&#39;s hard to answer this kind of a design question without understanding what you&#39;re trying to achieve.</div></div>
</div></blockquote><div>


<br></div></div><div class="im"><div>What we&#39;re trying to achieve is provide a feedback loop for the spellcheck, so it can improve the suggestions over time. For the feedback loop to work, we&#39;re going to assign unique identifiers to spellcheck results and store the identifiers in markers. When the user performs any operation on a spellcheck marker, the identifier from the marker will be used in the feedback. User operations may include &quot;Add to dictionary&quot;, &quot;Always ignore&quot;, &quot;Select suggestion X&quot;, etc.</div>


</div></div></div></div></blockquote><div><br></div><div>That sounds like a useful feature in general.</div><div><br></div></div><div>On Mon, Mar 11, 2013 at 9:41 AM, Rouslan Solomakhin <span dir="ltr">&lt;<a href="mailto:rouslan+webkit@chromium.org" target="_blank">rouslan+webkit@chromium.org</a>&gt;</span> wrote:<div>
<div class="h5"><br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div>On Sun, Mar 10, 2013 at 10:40 PM, Hajime Morrita <span dir="ltr">&lt;<a href="mailto:morrita@chromium.org" target="_blank">morrita@chromium.org</a>&gt;</span> wrote:<br>


</div><div class="gmail_extra"><div class="gmail_quote"><div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">


<div dir="ltr">There are DocumentMarker, or DocumentMakerDetails. Which do you have in your mind?</div></blockquote><div><br></div></div><div>I had DocumentMarkerDetails in mind. It seems to be the standard location of auxiliary information for a document marker. </div>


<div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">Or you can possibly introduce another DocumentMakerDetails for the &quot;contextual  marker&quot;. This approach might be cleaner. By introducing the new subclass, it won&#39;t interfere existing marker implementations. One caveat of this approach is that this needs extra heap allocations since DocumentMarkerDetails with unique IDs cannot be shared and will be allocated per marker.</div>


</blockquote><div><br></div></div><div>I like the approach with another DocumentMarkerDetails class. I would like to have both description and identifier in my DocumentMarkerDetails. Would it be OK to inherit from DocumentMarkerDescription?</div>


</div></div></div></blockquote><div><br></div></div></div><div>I&#39;d rather not add another subclass. That&#39;s even worse than modifying DocumentMarkerDetails IMO because this new subclass of DocumentMarkerDetails can&#39;t be shared among multiple markers. </div>
</div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>

<div><br></div><div>It seems okay for each DocumentMarker to directly have an identifier although we might want to if-def it depending on how big that identifier is.</div></div></blockquote><div style><br></div><div style>
On the other hand, if the ID is assigned per spellcheck request, not per word, it can be shared in the requested sentence. So the overhead won&#39;t that big. Also, API surface on DocumentMarkerController will be suffered by the change, especially if we ifdef it. Considering that the contextual spell checking (not grammatical checking) is virtually a chromium only feature, it might be better to let the complexity cost small by just extending DocumentMarkerDetails.<br>
</div><div><br></div><div style>I&#39;m fine with either way though. There isn&#39;t big difference here.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div><br></div><div>- R. Niwa</div></div><div><br>

</div>
<br>_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br>
<a href="https://lists.webkit.org/mailman/listinfo/webkit-dev" target="_blank">https://lists.webkit.org/mailman/listinfo/webkit-dev</a><br>
<br></blockquote></div><br></div></div>