[Webkit-unassigned] [Bug 110447] REGRESSION(r143470): editing/spelling/spelling-changed-text.html fails on Qt, GTK, EFL

[bugzilla-daemon at webkit.org]

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


Tony Chang <tony at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #189612|review?                     |review+
               Flag|                            |




--- Comment #8 from Tony Chang <tony at chromium.org>  2013-02-21 15:40:27 PST ---
(From update of attachment 189612)
View in context: https://bugs.webkit.org/attachment.cgi?id=189612&action=review

The change to editing/spelling/spelling-changed-text.html looks good.

> LayoutTests/platform/efl/TestExpectations:1824
> -webkit.org/b/108370 editing/spelling/spelling-with-underscore-selection.html [ Skip ]
> -webkit.org/b/108370 editing/spelling/spelling-with-whitespace-selection.html [ Skip ]
> +webkit.org/b/108370 editing/spelling/spelling-double-clicked-word.html [ Failure ]
> +webkit.org/b/108370 editing/spelling/spelling-double-clicked-word-with-underscores.html [ Failure ]
> +webkit.org/b/108370 editing/spelling/spelling-exactly-selected-multiple-words.html [ Failure ]

I would make the Skip -> Failure change in a separate patch.  My only worry is that some of these tests are timing out rather than failing, and in that case, we probably don't want to run the test.

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