⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Changeset 203317 in webkit


Ignore:
Timestamp:
Jul 15, 2016, 6:43:53 PM (10 years ago)
Author:
Simon Fraser
Message:

Repaints rects drawn incorrectly when inspecting a WebView on a Retina display
https://bugs.webkit.org/show_bug.cgi?id=159824
rdar://problem/27376305

Reviewed by Brian Burg.

InspectorOverlayPage.js set up the canvases with a deviceScaleFactor passed into
reset(), which comes from the overlay's m_page.deviceScaleFactor(). However, updatePaintRects()
used window.devicePixelRatio which was always 1.

Fix by setting the deviceScaleFactor on the m_overlayPage.

  • inspector/InspectorOverlay.cpp:

(WebCore::InspectorOverlay::overlayPage):

Location:
trunk/Source/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r203314 r203317  
     12016-07-15  Simon Fraser  <simon.fraser@apple.com>
     2
     3        Repaints rects drawn incorrectly when inspecting a WebView on a Retina display
     4        https://bugs.webkit.org/show_bug.cgi?id=159824
     5        rdar://problem/27376305
     6
     7        Reviewed by Brian Burg.
     8
     9        InspectorOverlayPage.js set up the canvases with a deviceScaleFactor passed into
     10        reset(), which comes from the overlay's m_page.deviceScaleFactor(). However, updatePaintRects()
     11        used window.devicePixelRatio which was always 1.
     12
     13        Fix by setting the deviceScaleFactor on the m_overlayPage.
     14
     15        * inspector/InspectorOverlay.cpp:
     16        (WebCore::InspectorOverlay::overlayPage):
     17
    1182016-07-15  Myles C. Maxfield  <mmaxfield@apple.com>
    219
  • trunk/Source/WebCore/inspector/InspectorOverlay.cpp

    r203244 r203317  
    868868    fillWithEmptyClients(pageConfiguration);
    869869    m_overlayPage = std::make_unique<Page>(WTFMove(pageConfiguration));
     870    m_overlayPage->setDeviceScaleFactor(m_page.deviceScaleFactor());
    870871
    871872    Settings& settings = m_page.settings();
Note: See TracChangeset for help on using the changeset viewer.