Title: [187145] trunk/Source/WebCore
Revision
187145
Author
commit-qu...@webkit.org
Date
2015-07-21 18:46:00 -0700 (Tue, 21 Jul 2015)

Log Message

Web Inspector: Node highlights are wrong when the page is zoomed
https://bugs.webkit.org/show_bug.cgi?id=147177

Patch by Joseph Pecoraro <pecor...@apple.com> on 2015-07-21
Reviewed by Simon Fraser.

* inspector/InspectorOverlay.cpp:
(WebCore::InspectorOverlay::update):
Remove scaling that appears to no longer be needed, it was double
scaling the overlay content and misplacing it in the process.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (187144 => 187145)


--- trunk/Source/WebCore/ChangeLog	2015-07-22 01:39:17 UTC (rev 187144)
+++ trunk/Source/WebCore/ChangeLog	2015-07-22 01:46:00 UTC (rev 187145)
@@ -1,3 +1,15 @@
+2015-07-21  Joseph Pecoraro  <pecor...@apple.com>
+
+        Web Inspector: Node highlights are wrong when the page is zoomed
+        https://bugs.webkit.org/show_bug.cgi?id=147177
+
+        Reviewed by Simon Fraser.
+
+        * inspector/InspectorOverlay.cpp:
+        (WebCore::InspectorOverlay::update):
+        Remove scaling that appears to no longer be needed, it was double
+        scaling the overlay content and misplacing it in the process.
+
 2015-07-21  Dean Jackson  <d...@apple.com>
 
         Default media controls use a serif font, which seems wrong

Modified: trunk/Source/WebCore/inspector/InspectorOverlay.cpp (187144 => 187145)


--- trunk/Source/WebCore/inspector/InspectorOverlay.cpp	2015-07-22 01:39:17 UTC (rev 187144)
+++ trunk/Source/WebCore/inspector/InspectorOverlay.cpp	2015-07-22 01:46:00 UTC (rev 187145)
@@ -316,8 +316,6 @@
     FrameView* overlayView = overlayPage()->mainFrame().view();
     IntSize viewportSize = view->unscaledVisibleContentSizeIncludingObscuredArea();
     IntSize frameViewFullSize = view->unscaledVisibleContentSizeIncludingObscuredArea(ScrollableArea::IncludeScrollbars);
-    overlayPage()->setPageScaleFactor(m_page.pageScaleFactor(), IntPoint());
-    frameViewFullSize.scale(m_page.pageScaleFactor());
     overlayView->resize(frameViewFullSize);
 
     // Clear canvas and paint things.
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to