Bug 78031 - SVG root element's inspector highlight has different dimensions than the element when zooming the page
Summary: SVG root element's inspector highlight has different dimensions than the elem...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Max Vujovic
URL:
Keywords:
Depends on: 78037
Blocks:
  Show dependency tree / graph
 
Reported: 2012-02-07 13:47 PST by Max Vujovic
Modified: 2012-10-10 09:40 PDT (History)
4 users (show)

See Also:

Attachments
Reproduction (474 bytes, text/html)
2012-02-07 13:47 PST, Max Vujovic 		no flags Details
Patch (3.67 KB, patch)
2012-10-08 15:22 PDT, Max Vujovic 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Max Vujovic 2012-02-07 13:47:44 PST 
Created attachment 125919 [details]
Reproduction

When zooming into the page, web inspector will show a SVG root element's highlight as larger than the element.
When zooming out of the page, web inspector will show a SVG root element's highlight as smaller than the element.
In both cases, the highlight's dimensions should match the element's dimensions.

Please see the attached reproduction for an example.
Comment 1 Max Vujovic 2012-03-19 08:36:47 PDT 
This should be fixed after bug 78037, but I think it deserves a regression test. I'll make a test and upload a patch. Then, we can mark it fixed.
Comment 2 Max Vujovic 2012-10-08 15:22:27 PDT 
Created attachment 167627 [details]
Patch

This bug was fixed in bug 78037. I've uploaded a patch which just contains a regression test for this bug.
Comment 3 WebKit Review Bot 2012-10-10 07:22:47 PDT 
Comment on attachment 167627 [details]
Patch

Clearing flags on attachment: 167627

Committed r130899: <http://trac.webkit.org/changeset/130899>
Comment 4 WebKit Review Bot 2012-10-10 07:22:50 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 5 Max Vujovic 2012-10-10 09:40:37 PDT 
Thanks for the review!