Remove GraphicsContext::applyDeviceScaleFactor()

Source/web/PageWidgetDelegate.cpp

Index: Source/web/PageWidgetDelegate.cpp
diff --git a/Source/web/PageWidgetDelegate.cpp b/Source/web/PageWidgetDelegate.cpp
index b43aba571d5b73f799685c7eb9b36073f7d942b9..c92d29c0d59ca623ec97389bd7172077977f196d 100644
--- a/Source/web/PageWidgetDelegate.cpp
+++ b/Source/web/PageWidgetDelegate.cpp
@@ -67,8 +67,9 @@ void PageWidgetDelegate::paint(Page& page, PageOverlayList* overlays, WebCanvas*
         return;
     GraphicsContext gc(canvas);
     gc.setCertainlyOpaque(background == Opaque);
-    gc.applyDeviceScaleFactor(page.deviceScaleFactor());
-    gc.setDeviceScaleFactor(page.deviceScaleFactor());

[f(malita), 2014/11/14 03:24:56]

This looks weird: the device scale factor is supposed to convey a supplemental
scale not reflected in the CTM; scaling AND setting a DSF doesn't smell right.

Just a thought, probably better to not stir it up :)

[Stephen Chennney, 2014/11/14 15:26:22]

It's does what it was designed to do, but that may not be right. The code needs
to both apply the factor, and tell GraphicsContext what that factor was.
GraphicsContext then uses the deviceScaleFactior in two other drawing ops.

The interesting question is: why use that scale factor in other ops if it is
already baked into the transform.

I think the GraphicsContext::drawPicture usage is just plain wrong (because you
still can't determine if it will be rasterized at 1-1 size, and even if you
ignore that, you should be dividing by the scale, not multiplying).

The marker drawing code that uses it is correct in that it makes use of the
device scale to generate higher res markers, then it unapplies the
deviceScaleFactor before drawing. Effectively it is trying to draw something the
same visual size but with higher resolution if the device scale factor is
higher. This code will break if we ever have high device scale factors.

+    float scaleFactor = page.deviceScaleFactor();
+    gc.scale(scaleFactor, scaleFactor);
+    gc.setDeviceScaleFactor(scaleFactor);
     IntRect dirtyRect(rect);
     gc.save(); // Needed to save the canvas, not the GraphicsContext.
     FrameView* view = root.view();