Add 'printing' flag to PaintInfo

Source/core/page/PrintContextTest.cpp

Index: Source/core/page/PrintContextTest.cpp
diff --git a/Source/core/page/PrintContextTest.cpp b/Source/core/page/PrintContextTest.cpp
index cb49b96c035e43d518f3c6c048670477b05762e8..950e1214ee9c8b81f86b67a9b79052264124b677 100644
--- a/Source/core/page/PrintContextTest.cpp
+++ b/Source/core/page/PrintContextTest.cpp
@@ -94,19 +94,15 @@ protected:
     void printSinglePage(SkCanvas& canvas)
     {
         IntRect pageRect(0, 0, kPageWidth, kPageHeight);
+        printContext().begin(pageRect.width(), pageRect.height());
         document().view()->updateAllLifecyclePhases();
-        document().setPrinting(true);

[Julien - ping for review, 2015/07/15 17:14:25]

Should we remove Document::setPrinting if we're not testing this configuration
anymore? Or do you reserve this to a follow-up change (in which case a TODO is
in order)?

[fs, 2015/07/15 18:25:58]

Document::setPrinting(...) is still called, only via printContext().begin(...),
so this call was made redundant by rearranging the code (which now more like the
"real" printing code.) The flag is still used during layout AFAIK. Maybe we can
eventually drop the GC::setPrinting call below though.

[Julien - ping for review, 2015/07/16 14:32:01]

I am just concerned about having several sites holding the same information,
potentially with discrepancies between them. You're bringing down the number of
places where we store the painting information so that's a great change.

         SkPictureBuilder pictureBuilder(pageRect);
         GraphicsContext& context = pictureBuilder.context();
         context.setPrinting(true);
-        DeprecatedPaintLayer& rootLayer = *document().view()->layoutView()->layer();
-        DeprecatedPaintLayerPaintingInfo paintingInfo(&rootLayer, LayoutRect(pageRect), PaintBehaviorNormal, LayoutSize());
-        DeprecatedPaintLayerPainter(rootLayer).paintLayerContents(&context, paintingInfo, PaintLayerPaintingCompositingAllPhases);
-        printContext().begin(kPageWidth, kPageHeight);
-        printContext().end();
+        document().view()->paintContents(&context, pageRect);
         pictureBuilder.endRecording()->playback(&canvas);
         printContext().outputLinkedDestinations(&canvas, pageRect);
-        document().setPrinting(false);
+        printContext().end();
     }
 
     static String absoluteBlockHtmlForLink(int x, int y, int width, int height, const char* url, const char* children = nullptr)