Changeset 181721 in webkit

Timestamp:

Mar 18, 2015, 7:16:12 PM (10 years ago)

Author:

Simon Fraser

Message:

Don't paint PDFs on every scroll
​https://bugs.webkit.org/show_bug.cgi?id=142857
rdar://problem/20130207

Reviewed by Tim Horton.

Don't push a scale factor onto the PDF layer controller if it hasn't
changed, since doing so causes a repaint.

• WebProcess/Plugins/PDF/PDFPlugin.mm:

(WebKit::PDFPlugin::updatePageAndDeviceScaleFactors):

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• WebProcess/Plugins/PDF/PDFPlugin.mm (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog ¶

@@ +1 @@
+2015-03-18  Simon Fraser  <simon.fraser@apple.com>
+
+        Don't paint PDFs on every scroll
+        https://bugs.webkit.org/show_bug.cgi?id=142857
+        rdar://problem/20130207
+
+        Reviewed by Tim Horton.
+        
+        Don't push a scale factor onto the PDF layer controller if it hasn't
+        changed, since doing so causes a repaint.
+
+        * WebProcess/Plugins/PDF/PDFPlugin.mm:
+        (WebKit::PDFPlugin::updatePageAndDeviceScaleFactors):
+
 2015-03-18  Anders Carlsson  <andersca@apple.com>
 

trunk/Source/WebKit2/WebProcess/Plugins/PDF/PDFPlugin.mm ¶

@@ -1031 +1031 @@
         newScaleFactor *= webFrame()->page()->pageScaleFactor();
 
-    [m_pdfLayerController setDeviceScaleFactor:newScaleFactor];
+    if (newScaleFactor != [m_pdfLayerController deviceScaleFactor])
+        [m_pdfLayerController setDeviceScaleFactor:newScaleFactor];
 }