WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
66246
TransformState.move() should be negated in the unapply code path
https://bugs.webkit.org/show_bug.cgi?id=66246
Summary
TransformState.move() should be negated in the unapply code path
Simon Fraser (smfr)
Reported
2011-08-15 12:37:37 PDT
TransformState.move() should negate the arguments in the 'unapply' code path as it does for the 'apply' code path.
Attachments
Patch
(8.65 KB, patch)
2011-08-15 14:17 PDT
,
Simon Fraser (smfr)
cmarrin
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Simon Fraser (smfr)
Comment 1
2011-08-15 14:17:21 PDT
Created
attachment 103950
[details]
Patch
Chris Marrin
Comment 2
2011-08-15 15:10:10 PDT
Comment on
attachment 103950
[details]
Patch r=me
Simon Fraser (smfr)
Comment 3
2011-08-31 18:19:30 PDT
http://trac.webkit.org/changeset/94259
Ryosuke Niwa
Comment 4
2011-09-01 00:33:04 PDT
This patch appears to have caused the following tests to fail on Qt bots: fast/events/document-elementFromPoint.html plugins/mouse-events-fixedpos.html See
http://build.webkit.org/builders/Qt%20Linux%20Release/builds/37020
.
Ryosuke Niwa
Comment 5
2011-09-01 00:35:20 PDT
Those tests are also failing on Snow Leopard, Windows 7, and GTK:
http://build.webkit.org/results/SnowLeopard%20Intel%20Debug%20(Tests)/r94260%20(1946)/results.html
http://build.webkit.org/results/Windows%207%20Release%20(Tests)/r94273%20(15944)/results.html
http://build.webkit.org/results/GTK%20Linux%2032-bit%20Release/r94274%20(17041)/results.html
Simon Fraser (smfr)
Comment 6
2011-09-01 09:04:29 PDT
I already have
bug 67353
and will look at these other tests today.
Simon Fraser (smfr)
Comment 7
2011-09-01 11:48:58 PDT
Tests fixed in
http://trac.webkit.org/changeset/94324
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