WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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
noel gordon
Reported
2011-08-07 23:44:10 PDT
Sea of red.
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=mouse-events-fixedpos.html
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
Show Obsolete
(3)
View All
Add attachment
proposed patch, testcase, etc.
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
Created
attachment 103213
[details]
Patch
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.
Top of Page
Format For Printing
XML
Clone This Bug