Implements core logic for Pixel Canvas

ui/compositor/canvas_painter.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/compositor/canvas_painter.h"
 
 #include "cc/paint/display_item_list.h"
+#include "ui/compositor/compositor_switches.h"
 
 namespace ui {
 
 CanvasPainter::CanvasPainter(SkBitmap* output,
                              const gfx::Size& paint_size,
                              float raster_scale,
                              SkColor clear_color)
     : output_(output),
       paint_size_(paint_size),
       raster_scale_(raster_scale),
       clear_color_(clear_color),
       list_(new cc::DisplayItemList),
-      context_(list_.get(), raster_scale, gfx::Rect(paint_size_)) {}
+      context_(list_.get(),
+               raster_scale,
+               gfx::ScaleToRoundedRect(
+                   gfx::Rect(paint_size_),
+                   IsPixelCanvasRecordingEnabled() ? raster_scale : 1.f)) {}

[danakj, 2017/07/21 17:29:39]

Rather than reading the command line and trying to intercept that for tests, can
you just move the decision to use this higher and pass in the information this
class needs?

[malaykeshav, 2017/07/21 23:30:17]

Done

 
 CanvasPainter::~CanvasPainter() {
   gfx::Size pixel_size = gfx::ScaleToCeiledSize(paint_size_, raster_scale_);
   SkImageInfo info = SkImageInfo::MakeN32(
       pixel_size.width(), pixel_size.height(), kPremul_SkAlphaType);
   if (!output_->tryAllocPixels(info))
     return;
 
   SkCanvas canvas(*output_);
   canvas.clear(clear_color_);
 
   canvas.scale(raster_scale_, raster_scale_);
 
   list_->Finalize();
   list_->Raster(&canvas, nullptr);
 }
 
 }  // namespace ui