Changeset 190919 in webkit

Timestamp:

Oct 12, 2015 7:11:06 PM (9 years ago)

Author:

jonlee@apple.com

Message:

Add missing paint objects for arc and arcTo fills
​https://bugs.webkit.org/show_bug.cgi?id=150075
<rdar://problem/23082233>

Reviewed by Dean Jackson.

• Animometer/tests/simple/resources/simple-canvas-paths.js: Add missing paint objects

for arcTo and arc segment fills.
(CanvasArcToSegmentFill):
(CanvasArcToSegmentFill.prototype.draw):
(CanvasArcSegmentFill):
(CanvasArcSegmentFill.prototype.draw):

Location:

trunk/PerformanceTests

Files:

• Animometer/tests/simple/resources/simple-canvas-paths.js (modified) (2 diffs)
• ChangeLog (modified) (1 diff)

trunk/PerformanceTests/Animometer/tests/simple/resources/simple-canvas-paths.js

@@ -105 +105 @@
 };
 
+function CanvasArcToSegmentFill(stage) {
+    CanvasArcToSegment.call(this, stage);
+};
+CanvasArcToSegmentFill.prototype.draw = function(context) {
+    context.fillStyle = this._color;
+    context.beginPath();
+    context.moveTo(this._point1.x, this._point1.y);
+    context.arcTo(this._point2.x, this._point2.y, this._point3.x, this._point3.y, this._radius);
+    context.fill();
+};
+
 function CanvasArcSegment(stage) {
     var maxSize = stage.randomInt(20, 200);
@@ -123 +134 @@
     context.arc(this._point.x, this._point.y, this._radius, this._startAngle, this._endAngle, this._counterclockwise);
     context.stroke();
+};
+
+function CanvasArcSegmentFill(stage) {
+    CanvasArcSegment.call(this, stage);
+};
+CanvasArcSegmentFill.prototype.draw = function(context) {
+    context.fillStyle = this._color;
+    context.beginPath();
+    context.arc(this._point.x, this._point.y, this._radius, this._startAngle, this._endAngle, this._counterclockwise);
+    context.fill();
 };
 

trunk/PerformanceTests/ChangeLog

@@ +1 @@
+2015-10-12  Jon Lee  <jonlee@apple.com>
+
+        Add missing paint objects for arc and arcTo fills
+        https://bugs.webkit.org/show_bug.cgi?id=150075
+        <rdar://problem/23082233>
+
+        Reviewed by Dean Jackson.
+
+        * Animometer/tests/simple/resources/simple-canvas-paths.js: Add missing paint objects
+        for arcTo and arc segment fills.
+        (CanvasArcToSegmentFill):
+        (CanvasArcToSegmentFill.prototype.draw):
+        (CanvasArcSegmentFill):
+        (CanvasArcSegmentFill.prototype.draw):
+
 2015-10-12  Jon Lee  <jonlee@apple.com>