Bug 58284

Summary: REGRESSION(r81880): Paste menu is disabled for many edit fields (yahoo and google search, yahoo mail msg, forms, etc)
Product: WebKit Reporter: Jon Lee <jonlee>
Component: WebKit2Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, jberlin, jonlee, mjs
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Windows 7   
Attachments:
Description Flags
Patch
mjs: review+
Patch none

Description Jon Lee 2011-04-11 16:20:24 PDT
* SUMMARY
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.

* STEPS TO REPRODUCE
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

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

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

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

r=me
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]
Patch

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