cc: Partial tile update for one-copy raster.

cc/resources/picture_pile_impl_unittest.cc

 // Copyright 2013 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 "base/memory/scoped_ptr.h"
 #include "cc/test/fake_picture_pile_impl.h"
 #include "cc/test/skia_common.h"
 #include "skia/ext/refptr.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "third_party/skia/include/core/SkPixelRef.h"
@@ skipping 327 matching lines @@
       // up to one pixel outside the content rect is guaranteed to be opaque.
       // Outside of that is undefined.
       gfx::Rect canvas_rect(content_rect);
       canvas_rect.Inset(0, 0, -1, -1);
 
       SkBitmap bitmap;
       bitmap.allocN32Pixels(canvas_rect.width(), canvas_rect.height());
       SkCanvas canvas(bitmap);
       canvas.clear(SK_ColorTRANSPARENT);
 
-      pile->PlaybackToCanvas(&canvas, canvas_rect, contents_scale);
+      pile->PlaybackToCanvas(&canvas, canvas_rect, canvas_rect, contents_scale);
 
       SkColor* pixels = reinterpret_cast<SkColor*>(bitmap.getPixels());
       int num_pixels = bitmap.width() * bitmap.height();
       bool all_white = true;
       for (int i = 0; i < num_pixels; ++i) {
         EXPECT_EQ(SkColorGetA(pixels[i]), 255u);
         all_white &= (SkColorGetR(pixels[i]) == 255);
         all_white &= (SkColorGetG(pixels[i]) == 255);
         all_white &= (SkColorGetB(pixels[i]) == 255);
       }
@@ skipping 24 matching lines @@
   gfx::Size content_bounds(
       gfx::ToCeiledSize(gfx::ScaleSize(layer_bounds, contents_scale)));
 
   gfx::Rect canvas_rect(content_bounds);
   canvas_rect.Inset(0, 0, -1, -1);
 
   SkBitmap bitmap;
   bitmap.allocN32Pixels(canvas_rect.width(), canvas_rect.height());
   SkCanvas canvas(bitmap);
 
-  pile->PlaybackToCanvas(&canvas, canvas_rect, contents_scale);
+  pile->PlaybackToCanvas(&canvas, canvas_rect, canvas_rect, contents_scale);
 
   SkColor* pixels = reinterpret_cast<SkColor*>(bitmap.getPixels());
   int num_pixels = bitmap.width() * bitmap.height();
   for (int i = 0; i < num_pixels; ++i) {
     EXPECT_EQ(SkColorGetA(pixels[i]), 0u);
   }
 }
 
+// XXX ADD RASTER PARTIAL TILE TESTS HERE XXX

[vmpstr, 2015/05/15 20:54:21]

Is this a TODO or are you going to do this before landing?

[danakj, 2015/05/15 21:52:14]

I'm going to do this before landing.

+
 class OverlapTest : public ::testing::TestWithParam<float> {
  public:
   static float MinContentsScale() { return 1.f / 4.f; }
 };
 
 TEST_P(OverlapTest, NoOverlap) {
   gfx::Size tile_size(10, 10);
   gfx::Size layer_bounds(30, 30);
   gfx::Size bigger_than_layer_bounds(300, 300);
   float contents_scale = GetParam();
@@ skipping 18 matching lines @@
 
   scoped_refptr<FakePicturePileImpl> pile =
       FakePicturePileImpl::CreateFromPile(recording_source.get(), nullptr);
   gfx::Size content_bounds(
       gfx::ToCeiledSize(gfx::ScaleSize(layer_bounds, contents_scale)));
 
   SkBitmap bitmap;
   bitmap.allocN32Pixels(content_bounds.width(), content_bounds.height());
   SkCanvas canvas(bitmap);
 
-  pile->PlaybackToCanvas(&canvas, gfx::Rect(content_bounds), contents_scale);
+  pile->PlaybackToCanvas(&canvas, gfx::Rect(content_bounds),
+                         gfx::Rect(content_bounds), contents_scale);
 
   for (int y = 0; y < bitmap.height(); y++) {
     for (int x = 0; x < bitmap.width(); x++) {
       SkColor color = bitmap.getColor(x, y);
       EXPECT_EQ(SkColorGetR(test_color), SkColorGetR(color)) << "x: " << x
                                                              << ", y: " << y;
       EXPECT_EQ(SkColorGetG(test_color), SkColorGetG(color)) << "x: " << x
                                                              << ", y: " << y;
       EXPECT_EQ(SkColorGetB(test_color), SkColorGetB(color)) << "x: " << x
                                                              << ", y: " << y;
@@ skipping 89 matching lines @@
     EXPECT_TRUE(iterator);
     EXPECT_TRUE(*iterator == discardable_bitmap[1].pixelRef());
     EXPECT_TRUE(++iterator);
     EXPECT_TRUE(*iterator == discardable_bitmap[2].pixelRef());
     EXPECT_FALSE(++iterator);
   }
 }
 
 }  // namespace
 }  // namespace cc