RESOLVED FIXED 65841
[Chromium] plugins/mouse-events-fixedpos.html should pass on win/mac/linux
https://bugs.webkit.org/show_bug.cgi?id=65841
Summary [Chromium] plugins/mouse-events-fixedpos.html should pass on win/mac/linux
Attachments
Screen-shot-for-posterity (68.33 KB, image/png)
2011-08-07 23:47 PDT, noel gordon
no flags
Patch (3.85 KB, patch)
2011-08-08 00:25 PDT, noel gordon
no flags
Patch for landing (4.50 KB, patch)
2011-08-08 01:35 PDT, noel gordon
no flags
test-plugin-click-origin.html (1.84 KB, text/html)
2011-08-09 19:34 PDT, noel gordon
no flags
Patch for landing - update ChangeLog (4.83 KB, patch)
2011-08-09 19:52 PDT, noel gordon
no flags
Patch for landing (5.29 KB, patch)
2011-08-10 12:29 PDT, Tony Chang
no flags
noel gordon
Comment 1 2011-08-07 23:47:35 PDT
Created attachment 103206 [details] Screen-shot-for-posterity
noel gordon
Comment 2 2011-08-07 23:50:35 PDT
mac ports load plugins in windowless mode by default, chrome linux/win load them windowed mode by default. Need to add attribute windowPlugin="false" on the <embed> in plugins/mouse-events-fixedpos.html to force windowless plugin mode.
noel gordon
Comment 3 2011-08-08 00:25:58 PDT
noel gordon
Comment 4 2011-08-08 01:29:21 PDT
Add chromium-linux results.
noel gordon
Comment 5 2011-08-08 01:35:03 PDT
Created attachment 103220 [details] Patch for landing
Tony Chang
Comment 6 2011-08-08 09:28:29 PDT
Comment on attachment 103220 [details] Patch for landing View in context: https://bugs.webkit.org/attachment.cgi?id=103220&action=review Looks fine overall, just a small changelog nit. > LayoutTests/platform/chromium-linux/plugins/mouse-events-fixedpos-expected.txt:1 > +CONSOLE MESSAGE: line 39: PLUGIN: getFocusEvent Nit: Please mention in the ChangeLog how we differ from the Apple Mac port (ie., console messages in Apple Mac don't have line numbers). > LayoutTests/platform/chromium-win/plugins/mouse-events-fixedpos-expected.txt:2 > +CONSOLE MESSAGE: line 39: PLUGIN: mouseDown at (90, 90) Why does windows give different coordinates than mac and linux?
noel gordon
Comment 7 2011-08-09 19:33:56 PDT
> Why does windows give different coordinates than mac and linux? Because windows DRT uses client-window relative co-ords. We can write a quick test to see this.
noel gordon
Comment 8 2011-08-09 19:34:47 PDT
Created attachment 103431 [details] test-plugin-click-origin.html
noel gordon
Comment 9 2011-08-09 19:50:12 PDT
See the test, results within. Place a position:absolute plugin <embed> with top left corner at (100px,100px) and click on the page using DRT. If you click a (99,99), no plugin click is logged. But if you click at (100,100), the plugin click is logged at (0,0) in mac/linux (plugin relative co-ords), but at (100,100) on win (client window relative co-ords). See also the webkit win results: http://trac.webkit.org/browser/trunk/LayoutTests/platform/win/plugins/mouse-events-fixedpos-expected.txt
noel gordon
Comment 10 2011-08-09 19:51:16 PDT
(In reply to comment #6) > Nit: Please mention in the ChangeLog how we differ from the Apple Mac port (ie., console messages in Apple Mac don't have line numbers). Done.
noel gordon
Comment 11 2011-08-09 19:52:36 PDT
Created attachment 103433 [details] Patch for landing - update ChangeLog
Tony Chang
Comment 12 2011-08-10 09:44:13 PDT
(In reply to comment #7) > > Why does windows give different coordinates than mac and linux? > > Because windows DRT uses client-window relative co-ords. Is this a difference in DRT or in the plugin? Is it something we should try to make consistent across platforms or is this how NPAPI works?
WebKit Review Bot
Comment 13 2011-08-10 09:44:29 PDT
Comment on attachment 103433 [details] Patch for landing - update ChangeLog Rejecting attachment 103433 [details] from commit-queue. Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=ec2-cq-02', '--port..." exit_code: 2 Last 500 characters of output: s-fixedpos-expected.txt patching file LayoutTests/platform/chromium-win/plugins/mouse-events-fixedpos-expected.txt patching file LayoutTests/platform/chromium/test_expectations.txt Hunk #1 FAILED at 735. 1 out of 1 hunk FAILED -- saving rejects to file LayoutTests/platform/chromium/test_expectations.txt.rej patching file LayoutTests/plugins/mouse-events-fixedpos.html Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--reviewer', u'Tony Chang', u'--force']" exit_code: 1 Full output: http://queues.webkit.org/results/9332843
Tony Chang
Comment 14 2011-08-10 12:29:45 PDT
Created attachment 103517 [details] Patch for landing
WebKit Review Bot
Comment 15 2011-08-10 12:57:13 PDT
Comment on attachment 103517 [details] Patch for landing Clearing flags on attachment: 103517 Committed r92790: <http://trac.webkit.org/changeset/92790>
WebKit Review Bot
Comment 16 2011-08-10 12:57:18 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.