Remove PaintInfo's paintingRoot

third_party/WebKit/Source/core/frame/LocalFrame.cpp

 /*
  * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
  *                     1999 Lars Knoll <knoll@kde.org>
  *                     1999 Antti Koivisto <koivisto@kde.org>
  *                     2000 Simon Hausmann <hausmann@kde.org>
  *                     2000 Stefan Schimanski <1Stein@gmx.de>
  *                     2001 George Staikos <staikos@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2005 Alexey Proskuryakov <ap@nypop.com>
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
@@ skipping 42 matching lines @@
 #include "core/inspector/InstrumentingAgents.h"
 #include "core/layout/HitTestResult.h"
 #include "core/layout/LayoutView.h"
 #include "core/layout/compositing/PaintLayerCompositor.h"
 #include "core/loader/FrameLoadRequest.h"
 #include "core/loader/FrameLoaderClient.h"
 #include "core/loader/NavigationScheduler.h"
 #include "core/page/FocusController.h"
 #include "core/page/Page.h"
 #include "core/page/scrolling/ScrollingCoordinator.h"
+#include "core/paint/ObjectPainter.h"
+#include "core/paint/PaintInfo.h"
 #include "core/paint/PaintLayer.h"
 #include "core/paint/TransformRecorder.h"
 #include "core/svg/SVGDocumentExtensions.h"
 #include "platform/DragImage.h"
 #include "platform/PluginScriptForbiddenScope.h"
 #include "platform/RuntimeEnabledFeatures.h"
 #include "platform/ScriptForbiddenScope.h"
 #include "platform/graphics/GraphicsContext.h"
 #include "platform/graphics/StaticBitmapImage.h"
 #include "platform/graphics/paint/ClipRecorder.h"
@@ skipping 21 matching lines @@
     {
         ASSERT(!node || node->layoutObject());
         if (node)
             node->layoutObject()->updateDragState(true);
     }
 
     ~ScopedFramePaintingState()
     {
         if (node && node->layoutObject())
             node->layoutObject()->updateDragState(false);
-        frame->view()->setNodeToDraw(0);
     }
 
     RawPtrWillBeMember<LocalFrame> frame;
     RawPtrWillBeMember<Node> node;
 };
 
 inline float parentPageZoomFactor(LocalFrame* frame)
 {
     Frame* parent = frame->tree().parent();
     if (!parent || !parent->isLocalFrame())
@@ skipping 531 matching lines @@
 }
 
 PassOwnPtr<DragImage> LocalFrame::nodeImage(Node& node)
 {
     if (!node.layoutObject())
         return nullptr;
 
     const ScopedFramePaintingState state(this, &node);
 
     m_view->updateAllLifecyclePhases();
-
-    m_view->setNodeToDraw(&node); // Enable special sub-tree drawing mode.
-
-    // Document::updateLayout may have blown away the original LayoutObject.
     LayoutObject* layoutObject = node.layoutObject();
     if (!layoutObject)
         return nullptr;
 
-    IntRect rect;
+    // Directly paint boxes as if they are a stacking context.
+    if (layoutObject->isBox() && layoutObject->container()) {
+        IntRect boundingBox = layoutObject->absoluteBoundingBoxRectIncludingDescendants();
+        LayoutPoint paintOffset = boundingBox.location();
+        paintOffset -= layoutObject->offsetFromContainer(layoutObject->container(), LayoutPoint());
 
+        float deviceScaleFactor = m_host->deviceScaleFactor();
+        boundingBox.scale(deviceScaleFactor);
+        SkPictureBuilder pictureBuilder(SkRect::MakeIWH(boundingBox.width(), boundingBox.height()));
+        {
+            PaintInfo paintInfo(pictureBuilder.context(), boundingBox, PaintPhase::PaintPhaseForeground, GlobalPaintFlattenCompositingLayers, 0);
+
+            AffineTransform transform;
+            transform.scale(deviceScaleFactor, deviceScaleFactor);
+            transform.translate(-boundingBox.x(), -boundingBox.y());
+            TransformRecorder transformRecorder(pictureBuilder.context(), *this, transform);

[chrishtr, 2016/01/19 18:29:21]

The code here looks like it duplicates paintIntoDragImage. Refactor into a
helper.

[pdr., 2016/01/19 23:29:21]

Good idea. This required some invasive refactoring because there are two ways
the context is used. I think the code ended up in a nicer place.

+
+            ObjectPainter(*layoutObject).paintAsPseudoStackingContext(paintInfo, LayoutPoint(paintOffset));
+        }
+        RefPtr<const SkPicture> recording = pictureBuilder.endRecording();
+        RefPtr<SkImage> skImage = adoptRef(SkImage::NewFromPicture(recording.get(),
+            SkISize::Make(boundingBox.width(), boundingBox.height()), nullptr, nullptr));
+        RefPtr<Image> image = StaticBitmapImage::create(skImage.release());
+        return DragImage::create(image.get(), LayoutObject::shouldRespectImageOrientation(layoutObject), deviceScaleFactor, InterpolationHigh);
+    }
+
+    // FIXME(pdr): This will also paint the background if the object contains transparency. We can
+    // directly call layoutObject->paint(...) (see: ObjectPainter::paintAsPseudoStackingContext) but
+    // painters are inconsistent about which transform space they expect (see: svg, inlines, etc.)
     return paintIntoDragImage(*layoutObject, LayoutObject::shouldRespectImageOrientation(layoutObject),
-        GlobalPaintFlattenCompositingLayers, layoutObject->paintingRootRect(rect));
+        GlobalPaintFlattenCompositingLayers, layoutObject->absoluteBoundingBoxRectIncludingDescendants());
 }
 
 PassOwnPtr<DragImage> LocalFrame::dragImageForSelection(float opacity)
 {
     if (!selection().isRange())
         return nullptr;
 
     const ScopedFramePaintingState state(this, 0);
     m_view->updateAllLifecyclePhases();
 
@@ skipping 210 matching lines @@
 {
     m_frame->disableNavigation();
 }
 
 FrameNavigationDisabler::~FrameNavigationDisabler()
 {
     m_frame->enableNavigation();
 }
 
 } // namespace blink