ui: Eliminate allocating gfx::Canvas on the heap for every view.

ui/compositor/paint_recorder.cc

Index: ui/compositor/paint_recorder.cc
diff --git a/ui/compositor/paint_recorder.cc b/ui/compositor/paint_recorder.cc
index ee740c6faf5746999ad16945b98ceef0a8a6080f..d52427b05db8cdcca108207d81098d505d672b7e 100644
--- a/ui/compositor/paint_recorder.cc
+++ b/ui/compositor/paint_recorder.cc
@@ -9,30 +9,33 @@
 #include "third_party/skia/include/core/SkPictureRecorder.h"
 #include "ui/compositor/paint_cache.h"
 #include "ui/compositor/paint_context.h"
-#include "ui/gfx/canvas.h"
 #include "ui/gfx/skia_util.h"
 
 namespace ui {
 
 PaintRecorder::PaintRecorder(const PaintContext& context, PaintCache* cache)
-    : context_(context), canvas_(context.canvas_), cache_(cache) {
+    : context_(context),
+      owned_canvas_(
+          // If the |context| has a canvas, we'll just point to it in |canvas_|
+          // so use anything here to initialize the gfx::Canvas without any
+          // allocations.
+          // The SkCanvas reference returned by beginRecording is shared with
+          // the recorder_ so no  need to store a RefPtr to it on this class.
+          (context.canvas_
+               ? context.canvas_->sk_canvas()

[danakj, 2015/05/11 23:10:16]

This is all a little bit crazy, but will go away/get better when we eliminate
the canvas path. It's sad that this has to all be written in the initializer
like this but constructors /o\. I didn't want to add another constructor to
gfx::Canvas that allowed it to be null, that seemed slightly less good than some
temporary crazy edge cases for git cl format to figure out.

+               : skia::SharePtr(
+                     context.recorder_->beginRecording(
+                         gfx::RectToSkRect(context.bounds_),
+                         nullptr /* no SkRTreeFactory */,
+                         SkPictureRecorder::kComputeSaveLayerInfo_RecordFlag))
+                     .get()),
+          context.device_scale_factor_),
+      canvas_(context.canvas_ ? context.canvas_ : &owned_canvas_),
+      cache_(cache) {
 #if DCHECK_IS_ON()
   DCHECK(!context.inside_paint_recorder_);
   context.inside_paint_recorder_ = true;
 #endif
-
-  if (context.list_) {
-    SkRTreeFactory* no_factory = nullptr;
-    // This SkCancas is shared with the recorder_ so no need to store a RefPtr
-    // to it on this class.
-    skia::RefPtr<SkCanvas> skcanvas =
-        skia::SharePtr(context.recorder_->beginRecording(
-            gfx::RectToSkRect(context.bounds_), no_factory,
-            SkPictureRecorder::kComputeSaveLayerInfo_RecordFlag));
-    owned_canvas_ = make_scoped_ptr(gfx::Canvas::CreateCanvasWithoutScaling(
-        skcanvas.get(), context.device_scale_factor_));
-    canvas_ = owned_canvas_.get();
-  }
 }
 
 PaintRecorder::PaintRecorder(const PaintContext& context)