Make CanvasPainter raster directly to an SkBitmap

ui/views/drag_utils.cc

Index: ui/views/drag_utils.cc
diff --git a/ui/views/drag_utils.cc b/ui/views/drag_utils.cc
index f9e6c8be5ab607d7122bca1385181057e1780f2a..8952624610e5954c8d512cfd26474b8111effb53 100644
--- a/ui/views/drag_utils.cc
+++ b/ui/views/drag_utils.cc
@@ -10,7 +10,7 @@
 #include "ui/gfx/geometry/size.h"
 #include "ui/views/widget/widget.h"
 
-namespace {
+namespace views {
 
 float GetDeviceScaleForNativeView(views::Widget* widget) {

[tapted, 2017/02/14 23:34:07]

nit: reorder to match .h  (and remove views:: prefix from Widget)

[enne (OOO), 2017/02/15 00:02:05]

You might want to consider a presubmit like
https://cs.chromium.org/chromium/src/cc/PRESUBMIT.py?q=cc/PRES+package:%5Echr...

[tapted, 2017/02/15 00:41:23]

ooh - indeed :) also for gfx::.. just came up in another review

   float device_scale = 1.0f;
@@ -23,10 +23,6 @@ float GetDeviceScaleForNativeView(views::Widget* widget) {
   return device_scale;
 }

[tapted, 2017/02/14 23:34:07]

random orthogonal thought: I wonder if we should have something like
display::Screen::GetMaxDisplayScaleFactor() -- i.e. return the device_scale from
the Display with the highest scale factor. Since presumably you could always
drag this asset to that display, and it would look a bit rubbish if it was drawn
at a smaller scale. (maybe even this method could just always return 2.0f
without breaking anything..).

[enne (OOO), 2017/02/15 00:02:05]

Yeah, that would make a lot of sense.  You could still get color space wrong,
but at least it wouldn't be fuzzy.

 
-}  // namespace
-
-namespace views {
-
 gfx::Canvas* GetCanvasForDragImage(views::Widget* widget,
                                    const gfx::Size& canvas_size) {
   float device_scale = GetDeviceScaleForNativeView(widget);