Bug 65841 - [Chromium] plugins/mouse-events-fixedpos.html should pass on win/mac/linux
Summary: [Chromium] plugins/mouse-events-fixedpos.html should pass on win/mac/linux
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: noel gordon
URL:
Keywords:
Depends on: 61721
Blocks:
  Show dependency treegraph
 
Reported: 2011-08-07 23:44 PDT by noel gordon
Modified: 2011-08-10 12:57 PDT (History)
2 users (show)

See Also:


Attachments
Screen-shot-for-posterity (68.33 KB, image/png)
2011-08-07 23:47 PDT, noel gordon
no flags Details
Patch (3.85 KB, patch)
2011-08-08 00:25 PDT, noel gordon
no flags Details | Formatted Diff | Diff
Patch for landing (4.50 KB, patch)
2011-08-08 01:35 PDT, noel gordon
no flags Details | Formatted Diff | Diff
test-plugin-click-origin.html (1.84 KB, text/html)
2011-08-09 19:34 PDT, noel gordon
no flags Details
Patch for landing - update ChangeLog (4.83 KB, patch)
2011-08-09 19:52 PDT, noel gordon
no flags Details | Formatted Diff | Diff
Patch for landing (5.29 KB, patch)
2011-08-10 12:29 PDT, Tony Chang
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Comment 1 noel gordon 2011-08-07 23:47:35 PDT
Created attachment 103206 [details]
Screen-shot-for-posterity
Comment 2 noel gordon 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.
Comment 3 noel gordon 2011-08-08 00:25:58 PDT
Created attachment 103213 [details]
Patch
Comment 4 noel gordon 2011-08-08 01:29:21 PDT
Add chromium-linux results.
Comment 5 noel gordon 2011-08-08 01:35:03 PDT
Created attachment 103220 [details]
Patch for landing
Comment 6 Tony Chang 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?
Comment 7 noel gordon 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.
Comment 8 noel gordon 2011-08-09 19:34:47 PDT
Created attachment 103431 [details]
test-plugin-click-origin.html
Comment 9 noel gordon 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
Comment 10 noel gordon 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.
Comment 11 noel gordon 2011-08-09 19:52:36 PDT
Created attachment 103433 [details]
Patch for landing - update ChangeLog
Comment 12 Tony Chang 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?
Comment 13 WebKit Review Bot 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
Comment 14 Tony Chang 2011-08-10 12:29:45 PDT
Created attachment 103517 [details]
Patch for landing
Comment 15 WebKit Review Bot 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>
Comment 16 WebKit Review Bot 2011-08-10 12:57:18 PDT
All reviewed patches have been landed.  Closing bug.