Make non-rectangular clip region apply in the correct location....

skia/ext/platform_canvas_unittest.cc

 // Copyright (c) 2006-2008 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.
 
 // TODO(awalker): clean up the const/non-const reference handling in this test
 
 #include "build/build_config.h"
 
 #if !defined(OS_WIN)
 #include <unistd.h>
@@ skipping 38 matching lines @@
         // Outside the square should be canvas_color
         if ((*bitmap.getAddr32(cur_x, cur_y) | alpha_mask) !=
             (canvas_color | alpha_mask))
           return false;
       }
     }
   }
   return true;
 }
 
+bool IsOfColor(const SkBitmap& bitmap, int x, int y, uint32_t color) {
+  // For masking out the alpha values.
+  static uint32_t alpha_mask = 0xFF << SK_A32_SHIFT;
+  return (*bitmap.getAddr32(x, y) | alpha_mask) == (color | alpha_mask);
+}
+
+// Return true if canvas has something that passes for a rounded-corner
+// rectangle. Basically, we're just checking to make sure that the pixels in the
+// middle are of rect_color and pixels in the corners are of canvas_color.
+bool VerifyRoundedRect(const PlatformCanvas& canvas,
+                       uint32_t canvas_color, uint32_t rect_color,
+                       int x, int y, int w, int h) {
+  PlatformDevice& device = canvas.getTopPlatformDevice();
+  const SkBitmap& bitmap = device.accessBitmap(false);
+  SkAutoLockPixels lock(bitmap);
+
+  // Check corner points first. They should be of canvas_color.
+  if (!IsOfColor(bitmap, x, y, canvas_color)) return false;
+  if (!IsOfColor(bitmap, x + w, y, canvas_color)) return false;
+  if (!IsOfColor(bitmap, x, y + h, canvas_color)) return false;
+  if (!IsOfColor(bitmap, x + w, y, canvas_color)) return false;
+
+  // Check middle points. They should be of rect_color.
+  if (!IsOfColor(bitmap, (x + w / 2), y, rect_color)) return false;
+  if (!IsOfColor(bitmap, x, (y + h / 2), rect_color)) return false;
+  if (!IsOfColor(bitmap, x + w, (y + h / 2), rect_color)) return false;
+  if (!IsOfColor(bitmap, (x + w / 2), y + h, rect_color)) return false;
+
+  return true;
+}
+
 // Checks whether there is a white canvas with a black square at the given
 // location in pixels (not in the canvas coordinate system).
 bool VerifyBlackRect(const PlatformCanvas& canvas, int x, int y, int w, int h) {
   return VerifyRect(canvas, SK_ColorWHITE, SK_ColorBLACK, x, y, w, h);
 }
 
 // Check that every pixel in the canvas is a single color.
 bool VerifyCanvasColor(const PlatformCanvas& canvas, uint32_t canvas_color) {
   return VerifyRect(canvas, canvas_color, 0, 0, 0, 0, 0);
 }
@@ skipping 76 matching lines @@
 const int kLayerY = 3;
 const int kLayerW = 9;
 const int kLayerH = 7;
 
 // Size used by some tests to draw a rectangle inside the layer.
 const int kInnerX = 4;
 const int kInnerY = 5;
 const int kInnerW = 2;
 const int kInnerH = 3;
 
+// Radius used by some tests to draw a rounded-corner rectangle.
+const SkScalar kRadius = 2.0;
+
 }
 
 // This just checks that our checking code is working properly, it just uses
 // regular skia primitives.
 TEST(PlatformCanvas, SkLayer) {
   // Create the canvas initialized to opaque white.
   PlatformCanvas canvas(16, 16, true);
   canvas.drawColor(SK_ColorWHITE);
 
   // Make a layer and fill it completely to make sure that the bounds are
@@ skipping 141 matching lines @@
   canvas.restore();
   EXPECT_TRUE(VerifyBlackRect(canvas, kInnerX + 1, kInnerY + 1,
                               kInnerW, kInnerH));
 
   // Translate both before and after, and have a clip.
   canvas.drawColor(SK_ColorWHITE);
   canvas.save();
   canvas.translate(1, 1);
   {
     LayerSaver layer(canvas, kLayerX, kLayerY, kLayerW, kLayerH);
+    canvas.drawColor(SK_ColorWHITE);
     canvas.translate(1, 1);
-    AddClip(canvas, kInnerX, kInnerY, kInnerW, kInnerH);
+    AddClip(canvas, kInnerX + 1, kInnerY + 1, kInnerW - 1, kInnerH - 1);
     DrawNativeRect(canvas, 0, 0, 100, 100);
 #if defined(OS_WIN)
-    canvas.getTopPlatformDevice().makeOpaque(kInnerX, kInnerY,
-                                             kInnerW, kInnerH);
+    canvas.getTopPlatformDevice().makeOpaque(kLayerX, kLayerY,
+                                             kLayerW, kLayerH);
 #endif
   }
   canvas.restore();
-  EXPECT_TRUE(VerifyBlackRect(canvas, kInnerX + 2, kInnerY + 2,
-                              kInnerW, kInnerH));
+  EXPECT_TRUE(VerifyBlackRect(canvas, kInnerX + 3, kInnerY + 3,
+                              kInnerW - 1, kInnerH - 1));
+
+  // Translate both before and after, and have a path clip.
+  canvas.drawColor(SK_ColorWHITE);
+  canvas.save();
+  canvas.translate(1, 1);
+  {
+    LayerSaver layer(canvas, kLayerX, kLayerY, kLayerW, kLayerH);
+    canvas.drawColor(SK_ColorWHITE);
+    canvas.translate(1, 1);
+
+    SkPath path;
+    SkRect rect;
+    rect.iset(kInnerX - 1, kInnerY - 1,
+              kInnerX + kInnerW, kInnerY + kInnerH);
+    path.addRoundRect(rect, kRadius, kRadius);
+    canvas.clipPath(path);
+
+    DrawNativeRect(canvas, 0, 0, 100, 100);
+#if defined(OS_WIN)
+    canvas.getTopPlatformDevice().makeOpaque(kLayerX, kLayerY,
+                                             kLayerW, kLayerH);
+#endif
+  }
+  canvas.restore();
+  EXPECT_TRUE(VerifyRoundedRect(canvas, SK_ColorWHITE, SK_ColorBLACK,
+                                kInnerX + 1, kInnerY + 1, kInnerW, kInnerH));
 }
 
 }  // namespace skia