Bug 58284 - REGRESSION(r81880): Paste menu is disabled for many edit fields (yahoo and google search, yahoo mail msg, forms, etc)
Summary: REGRESSION(r81880): Paste menu is disabled for many edit fields (yahoo and go...
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Windows 7
: P2 Normal
Assignee: Nobody
Keywords: InRadar
: 58209 (view as bug list)
Depends on:
Reported: 2011-04-11 16:20 PDT by Jon Lee
Modified: 2011-04-14 09:26 PDT (History)
4 users (show)

See Also:

Patch (2.13 KB, patch)
2011-04-11 16:22 PDT, Jon Lee
mjs: review+
Details | Formatted Diff | Diff
Patch (4.92 KB, patch)
2011-04-13 12:02 PDT, Jon Lee
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jon Lee 2011-04-11 16:20:24 PDT
Paste is disabled for many edit fields (yahoo and google search, yahoo mail msg, etc). Accelerator Ctrl-V will paster into same field from clipboard.
Tested on W7. 
Not reproducible on SL.

1.copy some text for clipboard from third party (I used Character Map on windows)
2. Open yahoo.com or google.com 
3. While cursor is in search field on the page, select Edit>Paste

Paste option is disabled, but text can be pasted via Ctrl-V.
Comment 1 Jon Lee 2011-04-11 16:20:47 PDT
Comment 2 Jon Lee 2011-04-11 16:22:02 PDT
Created attachment 89117 [details]
Comment 3 Maciej Stachowiak 2011-04-11 18:24:40 PDT
Comment on attachment 89117 [details]

Is it possible to create a regression test for this? If so, please add one, if not, r=me
Comment 4 Jon Lee 2011-04-12 10:17:52 PDT
I spoke with Enrica about this, and she told me that because the bug was in the menu and not keyboard shortcuts, that this is not something that can be tested via DumpRenderTree.
Comment 5 Jon Lee 2011-04-13 12:02:30 PDT
Created attachment 89425 [details]

New patch that avoids the #ifdef PLATFORM() call and moves it to a runtime check in EditingBehavior.h
Comment 6 Maciej Stachowiak 2011-04-13 14:22:00 PDT
Comment on attachment 89425 [details]

Comment 7 WebKit Commit Bot 2011-04-13 22:37:41 PDT
The commit-queue encountered the following flaky tests while processing attachment 89425 [details]:

http/tests/misc/favicon-loads-with-icon-loading-override.html bug 58412 (author: alice.liu@apple.com)
The commit-queue is continuing to process your patch.
Comment 8 WebKit Commit Bot 2011-04-13 22:40:27 PDT
Comment on attachment 89425 [details]

Clearing flags on attachment: 89425

Committed r83814: <http://trac.webkit.org/changeset/83814>
Comment 9 WebKit Commit Bot 2011-04-13 22:40:30 PDT
All reviewed patches have been landed.  Closing bug.
Comment 10 Jessie Berlin 2011-04-14 09:26:33 PDT
*** Bug 58209 has been marked as a duplicate of this bug. ***