Bug 46726 - Right-clicking on windowless Flash plugin in WebKit2 makes a context menu appear in the bottom-right corner of the screen
Summary: Right-clicking on windowless Flash plugin in WebKit2 makes a context menu app...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Plug-ins (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Windows XP
: P2 Normal
Assignee: Nobody
URL: http://www.worldofwarcraft.com/index.xml
Keywords: InRadar, PlatformOnly
Depends on:
Blocks:
 
Reported: 2010-09-28 08:13 PDT by Adam Roben (:aroben)
Modified: 2010-10-01 09:17 PDT (History)
3 users (show)

See Also:

Attachments
Implement NPN_GetValue(NPNVnetscapeWindow) (12.40 KB, patch)
2010-10-01 09:11 PDT, Adam Roben (:aroben) 		andersca: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Roben (:aroben) 2010-09-28 08:13:18 PDT 
To reproduce:

1. Go to http://www.worldofwarcraft.com/index.xml
2. Right-click on the animation near the top of the page

A context menu for Flash appears in the bottom-right corner of the screen. It should appear where you clicked.
Comment 1 Adam Roben (:aroben) 2010-09-28 08:13:47 PDT 
<rdar://problem/8486319>
Comment 2 Adam Roben (:aroben) 2010-10-01 09:11:29 PDT 
Created attachment 69472 [details]
Implement NPN_GetValue(NPNVnetscapeWindow)
Comment 3 WebKit Review Bot 2010-10-01 09:12:35 PDT 
Attachment 69472 [details] did not pass style-queue:

Failed to run "['WebKitTools/Scripts/check-webkit-style']" exit_code: 1
WebKitTools/DumpRenderTree/TestNetscapePlugIn/PluginTest.cpp:78:  PluginTest::NPN_GetValue is incorrectly named. Don't use underscores in your identifier names.  [readability/naming] [4]
WebKitTools/DumpRenderTree/TestNetscapePlugIn/Tests/win/GetValueNetscapeWindow.cpp:26:  Found other header before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted.  [build/include_order] [4]
WebKitTools/DumpRenderTree/TestNetscapePlugIn/Tests/win/GetValueNetscapeWindow.cpp:28:  Alphabetical sorting problem.  [build/include_order] [4]
Total errors found: 3 in 12 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 4 Adam Roben (:aroben) 2010-10-01 09:17:26 PDT 
Committed r68891: <http://trac.webkit.org/changeset/68891>