Canvas: add clipping path support.

components/canvas/canvas_msg.rs

@@ -24,8 +24,9 @@ pub enum Canvas2dMsg {
     DrawImageSelf(Size2D<f64>, Rect<f64>, Rect<f64>, bool),
     BeginPath,
     BezierCurveTo(Point2D<f32>, Point2D<f32>, Point2D<f32>),
-    ClosePath,
     ClearRect(Rect<f32>),
+    Clip,
+    ClosePath,
     Fill,
     FillRect(Rect<f32>),
     GetImageData(Rect<f64>, Size2D<f64>, Sender<Vec<u8>>),

components/canvas/canvas_paint_task.rs

@@ -212,6 +212,7 @@ impl<'a> CanvasPaintTask<'a> {
                             Canvas2dMsg::ClosePath => painter.close_path(),
                             Canvas2dMsg::Fill => painter.fill(),
                             Canvas2dMsg::Stroke => painter.stroke(),
+                            Canvas2dMsg::Clip => painter.clip(),
                             Canvas2dMsg::DrawImage(imagedata, image_size, dest_rect, source_rect, smoothing_enabled) => {
                                 painter.draw_image(imagedata, image_size, dest_rect, source_rect, smoothing_enabled)
                             }
@@ -271,6 +272,7 @@ impl<'a> CanvasPaintTask<'a> {
         if let Some(state) = self.saved_states.pop() {
             mem::replace(&mut self.state, state);
             self.drawtarget.set_transform(&self.state.transform);
+            self.drawtarget.pop_clip();
         }
     }
 
@@ -325,6 +327,10 @@ impl<'a> CanvasPaintTask<'a> {
         };
     }
 
+    fn clip(&self) {
+        self.drawtarget.push_clip(&self.path_builder.finish());
+    }
+
     fn draw_image(&self, image_data: Vec<u8>, image_size: Size2D<f64>,
                   dest_rect: Rect<f64>, source_rect: Rect<f64>, smoothing_enabled: bool) {
         // We round up the floating pixel values to draw the pixels

components/script/dom/canvasrenderingcontext2d.rs

@@ -454,6 +454,7 @@ impl<'a> CanvasRenderingContext2DMethods for JSRef<'a, CanvasRenderingContext2D>
 
     // https://html.spec.whatwg.org/multipage/#dom-context-2d-fill
     fn Fill(self, _: CanvasWindingRule) {
+        // TODO: Process winding rule
         self.renderer.send(CanvasMsg::Canvas2d(Canvas2dMsg::Fill)).unwrap();
     }
 
@@ -462,6 +463,12 @@ impl<'a> CanvasRenderingContext2DMethods for JSRef<'a, CanvasRenderingContext2D>
         self.renderer.send(CanvasMsg::Canvas2d(Canvas2dMsg::Stroke)).unwrap();
     }
 
+    // https://html.spec.whatwg.org/multipage/#dom-context-2d-clip
+    fn Clip(self, _: CanvasWindingRule) {
+        // TODO: Process winding rule
+        self.renderer.send(CanvasMsg::Canvas2d(Canvas2dMsg::Clip)).unwrap();
+    }
+
     // https://html.spec.whatwg.org/multipage/#dom-context-2d-drawimage
     fn DrawImage(self, image: HTMLImageElementOrHTMLCanvasElementOrCanvasRenderingContext2D,
                  dx: f64, dy: f64) -> Fallible<()> {

components/script/dom/webidls/CanvasRenderingContext2D.webidl

@@ -90,7 +90,7 @@ interface CanvasRenderingContext2D {
   //boolean drawCustomFocusRing(Path2D path, Element element);
   //void scrollPathIntoView();
   //void scrollPathIntoView(Path2D path);
-  //void clip(optional CanvasWindingRule fillRule = "nonzero");
+  void clip(optional CanvasWindingRule fillRule = "nonzero");
   //void clip(Path2D path, optional CanvasWindingRule fillRule = "nonzero");
   //void resetClip();
   //boolean isPointInPath(unrestricted double x, unrestricted double y, optional CanvasWindingRule fillRule = "nonzero");

tests/wpt/metadata/2dcontext/compositing/2d.composite.clip.copy.html.ini

@@ -1,5 +0,0 @@
-[2d.composite.clip.copy.html]
-  type: testharness
-  [fill() does not affect pixels outside the clip region.]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/compositing/2d.composite.clip.destination-atop.html.ini

@@ -1,5 +0,0 @@
-[2d.composite.clip.destination-atop.html]
-  type: testharness
-  [fill() does not affect pixels outside the clip region.]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/compositing/2d.composite.clip.destination-in.html.ini

@@ -1,5 +0,0 @@
-[2d.composite.clip.destination-in.html]
-  type: testharness
-  [fill() does not affect pixels outside the clip region.]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/compositing/2d.composite.clip.destination-out.html.ini

@@ -1,5 +0,0 @@
-[2d.composite.clip.destination-out.html]
-  type: testharness
-  [fill() does not affect pixels outside the clip region.]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/compositing/2d.composite.clip.destination-over.html.ini

@@ -1,5 +0,0 @@
-[2d.composite.clip.destination-over.html]
-  type: testharness
-  [fill() does not affect pixels outside the clip region.]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/compositing/2d.composite.clip.lighter.html.ini

@@ -1,5 +0,0 @@
-[2d.composite.clip.lighter.html]
-  type: testharness
-  [fill() does not affect pixels outside the clip region.]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/compositing/2d.composite.clip.source-atop.html.ini

@@ -1,5 +0,0 @@
-[2d.composite.clip.source-atop.html]
-  type: testharness
-  [fill() does not affect pixels outside the clip region.]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/compositing/2d.composite.clip.source-in.html.ini

@@ -1,5 +0,0 @@
-[2d.composite.clip.source-in.html]
-  type: testharness
-  [fill() does not affect pixels outside the clip region.]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/compositing/2d.composite.clip.source-out.html.ini

@@ -1,5 +0,0 @@
-[2d.composite.clip.source-out.html]
-  type: testharness
-  [fill() does not affect pixels outside the clip region.]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/compositing/2d.composite.clip.source-over.html.ini

@@ -1,5 +0,0 @@
-[2d.composite.clip.source-over.html]
-  type: testharness
-  [fill() does not affect pixels outside the clip region.]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/compositing/2d.composite.clip.xor.html.ini

@@ -1,5 +0,0 @@
-[2d.composite.clip.xor.html]
-  type: testharness
-  [fill() does not affect pixels outside the clip region.]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/drawing-images-to-the-canvas/2d.drawImage.clip.html.ini

@@ -1,5 +0,0 @@
-[2d.drawImage.clip.html]
-  type: testharness
-  [Canvas test: 2d.drawImage.clip]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/drawing-rectangles-to-the-canvas/2d.clearRect.clip.html.ini

@@ -1,5 +0,0 @@
-[2d.clearRect.clip.html]
-  type: testharness
-  [clearRect is affected by clipping regions]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/drawing-rectangles-to-the-canvas/2d.fillRect.clip.html.ini

@@ -1,5 +0,0 @@
-[2d.fillRect.clip.html]
-  type: testharness
-  [fillRect is affected by clipping regions]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/drawing-rectangles-to-the-canvas/2d.strokeRect.clip.html.ini

@@ -1,5 +0,0 @@
-[2d.strokeRect.clip.html]
-  type: testharness
-  [strokeRect is affected by clipping regions]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/path-objects/2d.path.clip.basic.1.html.ini

@@ -1,5 +0,0 @@
-[2d.path.clip.basic.1.html]
-  type: testharness
-  [Canvas test: 2d.path.clip.basic.1]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/path-objects/2d.path.clip.basic.2.html.ini

@@ -1,5 +0,0 @@
-[2d.path.clip.basic.2.html]
-  type: testharness
-  [Canvas test: 2d.path.clip.basic.2]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/path-objects/2d.path.clip.empty.html.ini

@@ -1,5 +0,0 @@
-[2d.path.clip.empty.html]
-  type: testharness
-  [Canvas test: 2d.path.clip.empty]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/path-objects/2d.path.clip.intersect.html.ini

@@ -1,5 +0,0 @@
-[2d.path.clip.intersect.html]
-  type: testharness
-  [Canvas test: 2d.path.clip.intersect]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/path-objects/2d.path.clip.winding.1.html.ini

@@ -1,5 +0,0 @@
-[2d.path.clip.winding.1.html]
-  type: testharness
-  [Canvas test: 2d.path.clip.winding.1]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/path-objects/2d.path.clip.winding.2.html.ini

@@ -1,5 +0,0 @@
-[2d.path.clip.winding.2.html]
-  type: testharness
-  [Canvas test: 2d.path.clip.winding.2]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/shadows/2d.shadow.clip.2.html.ini

@@ -1,5 +0,0 @@
-[2d.shadow.clip.2.html]
-  type: testharness
-  [Shadows are not drawn outside the clipping region]
-    expected: FAIL
-

tests/wpt/metadata/2dcontext/the-canvas-state/2d.state.saverestore.clip.html.ini

@@ -1,5 +0,0 @@
-[2d.state.saverestore.clip.html]
-  type: testharness
-  [save()/restore() affects the clipping path]
-    expected: FAIL
-

tests/wpt/metadata/html/dom/interfaces.html.ini

@@ -6924,9 +6924,6 @@
   [CanvasRenderingContext2D interface: operation clip(CanvasFillRule)]
     expected: FAIL
 
-  [CanvasRenderingContext2D interface: operation clip(Path2D,CanvasFillRule)]
-    expected: FAIL
-
   [CanvasRenderingContext2D interface: operation resetClip()]
     expected: FAIL
 
@@ -7086,15 +7083,6 @@
   [CanvasRenderingContext2D interface: calling scrollPathIntoView(Path2D) on document.createElement("canvas").getContext("2d") with too few arguments must throw TypeError]
     expected: FAIL
 
-  [CanvasRenderingContext2D interface: document.createElement("canvas").getContext("2d") must inherit property "clip" with the proper type (39)]
-    expected: FAIL
-
-  [CanvasRenderingContext2D interface: calling clip(CanvasFillRule) on document.createElement("canvas").getContext("2d") with too few arguments must throw TypeError]
-    expected: FAIL
-
-  [CanvasRenderingContext2D interface: document.createElement("canvas").getContext("2d") must inherit property "clip" with the proper type (40)]
-    expected: FAIL
-
   [CanvasRenderingContext2D interface: calling clip(Path2D,CanvasFillRule) on document.createElement("canvas").getContext("2d") with too few arguments must throw TypeError]
     expected: FAIL
 

tests/wpt/metadata/html/semantics/embedded-content/the-canvas-element/initial.reset.clip.html.ini

@@ -1,5 +0,0 @@
-[initial.reset.clip.html]
-  type: testharness
-  [Resetting the canvas state resets the current clip region]
-    expected: FAIL
-