Bug 66315 - [chromium] SVG pan updates have wrong vertical sense
Summary: [chromium] SVG pan updates have wrong vertical sense
Status: RESOLVED DUPLICATE of bug 65800
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-16 10:18 PDT by W. James MacLean
Modified: 2011-08-16 11:28 PDT (History)
2 users (show)

See Also:


Attachments
Proposed fix + test (6.56 KB, patch)
2011-08-16 10:19 PDT, W. James MacLean
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description W. James MacLean 2011-08-16 10:18:26 PDT
When a .svg file is is panned in chromium (using shift+right-click+drag), the vertical motion is reversed from the mouse motion. This operation works as expected on Safari, so a mechanism is needed to specify which sense to use for the different ports.
Comment 1 W. James MacLean 2011-08-16 10:19:16 PDT
Created attachment 104063 [details]
Proposed fix + test
Comment 2 Tim Horton 2011-08-16 10:23:15 PDT
My understanding is that we already have a mechanism for converting between the coordinate spaces (windowToContents). Check out the patch in https://bugs.webkit.org/show_bug.cgi?id=65800 which makes use of this, and if you have a chance, build it with Chromium and see why it doesn't pass the test there.

I'd prefer that patch to this one because it seems unfortunate to keep track of this information in different places, especially because it's different between different platforms *and* frontends.
Comment 3 Tim Horton 2011-08-16 10:23:48 PDT
(In reply to comment #0)
> When a .svg file is is panned in chromium (using shift+right-click+drag), the vertical motion is reversed from the mouse motion. This operation works as expected on Safari, so a mechanism is needed to specify which sense to use for the different ports.

I will note that it no longer works correctly in the latest shipping version of Safari on OS X.
Comment 4 W. James MacLean 2011-08-16 11:25:52 PDT
(In reply to comment #2)
> My understanding is that we already have a mechanism for converting between the coordinate spaces (windowToContents). Check out the patch in https://bugs.webkit.org/show_bug.cgi?id=65800 which makes use of this, and if you have a chance, build it with Chromium and see why it doesn't pass the test there.
> 
> I'd prefer that patch to this one because it seems unfortunate to keep track of this information in different places, especially because it's different between different platforms *and* frontends.

I was unaware of windowToContents, but the patch you mention does resolve the issue. Presumably we should mark this bug as a duplicate, and proceed with https://bugs.webkit.org/show_bug.cgi?id=65800.
Comment 5 W. James MacLean 2011-08-16 11:26:27 PDT

*** This bug has been marked as a duplicate of bug 65800 ***
Comment 6 Tim Horton 2011-08-16 11:28:16 PDT
(In reply to comment #4)
> I was unaware of windowToContents, but the patch you mention does resolve the issue. Presumably we should mark this bug as a duplicate, and proceed with https://bugs.webkit.org/show_bug.cgi?id=65800.

Seems like the right thing to do, to me!

If you could run the test with pixel results on and send me the chromium result, I'd love to see why it differs, but haven't had the time to set up a Chromium build.