[Webkit-unassigned] [Bug 25639] Spellcheck disabling does not disable context menu

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed May 12 02:31:57 PDT 2010


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


Tony Chang (Google) <tony at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |tony at chromium.org




--- Comment #2 from Tony Chang (Google) <tony at chromium.org>  2010-05-12 02:31:56 PST ---
(In reply to comment #1)
> Created an attachment (id=55819)
 --> (https://bugs.webkit.org/attachment.cgi?id=55819) [details]
> Patch

This is not for review because it has no tests.  But I wanted some suggestions on how to test.  We can't use a pixel test because context menus aren't shown.  I considered adding a method to eventSender, but there's no method in the WebKit api for getting the menu items.  It's probably possible for me to crawl the NSMenu of a showing context menu, but is that too much of a hack?

The other option is to just write a manual layout test.  Suggestions welcome, otherwise I'll probably try to crawl the NSMenu tomorrow.

-- 
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