[Webkit-unassigned] [Bug 47689] REGRESSION (r69548): Autocorrections are applied even after typing further characters in the word

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Oct 14 16:23:41 PDT 2010


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





--- Comment #6 from jpu at apple.com  2010-10-14 16:23:41 PST ---
(In reply to comment #5)
> (From update of attachment 70794 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=70794&action=review
> 

I will generate a new patch to correct some of following issues.

> It’s not great to name a test after the Apple Radar bug number for the problem. Since people outside Apple have limited or no access to Radar we try to keep the amount of Radar references in the bugs and tests to a minimum. But I suppose we can live with it for a Mac-specific test.

I will use webkit bug number or more descriptive name in the future.

> 
> > LayoutTests/platform/mac-tiger/Skipped:213
> > +platform/mac/editing/spelling/radar8552250.html
> >  \ No newline at end of file
> 
> Should include a newline.
> 
> > LayoutTests/platform/mac/editing/spelling/radar8552250.html:38
> > +    setTimeout("keepTyping()", 1000);
> 
> It’s really bad to have a 1 second delay in a regression test. We want the layout tests to run really quickly. There is a debugging machinery for making time move forward quickly: the eventSender.leapForward function. However, if the timing check is in code outside WebKit perhaps that won’t work. You need to at least give that a try.

leapForward() didn't work in this case, because the timer is inside WebCore.

> 
> > LayoutTests/platform/mac/editing/spelling/radar8552250.html:46
> > +<div>You should see phrase 'the collapsing' without underline.</div>
> 
> How does the test output indicate the lack of the underline? Is there something in the render tree dump that would be there if the underline was present?

The underline is drawn based on marker value "CorrectionIndicator", which seems to be contained in the dump.

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


More information about the webkit-unassigned mailing list