Log Message
Merge r207274. rdar://problem/28849629
Modified Paths
- branches/safari-602-branch/LayoutTests/ChangeLog
- branches/safari-602-branch/Source/WebCore/ChangeLog
- branches/safari-602-branch/Source/WebCore/platform/graphics/cg/PathCG.cpp
Added Paths
Diff
Modified: branches/safari-602-branch/LayoutTests/ChangeLog (207745 => 207746)
--- branches/safari-602-branch/LayoutTests/ChangeLog 2016-10-24 07:18:39 UTC (rev 207745)
+++ branches/safari-602-branch/LayoutTests/ChangeLog 2016-10-24 07:18:43 UTC (rev 207746)
@@ -1,5 +1,19 @@
2016-10-20 Matthew Hanson <matthew_han...@apple.com>
+ Merge r207274. rdar://problem/28849629
+
+ 2016-10-12 Simon Fraser <simon.fra...@apple.com>
+
+ polygonPathFromPoints calls uncheckedAppend, but assertion size() < capacity() fails
+ https://bugs.webkit.org/show_bug.cgi?id=163276
+
+ Reviewed by Sam Weinig.
+
+ * css3/masking/large-clip-path-expected.txt: Added.
+ * css3/masking/large-clip-path.html: Added.
+
+2016-10-20 Matthew Hanson <matthew_han...@apple.com>
+
Merge r207159. rdar://problem/28857481
2016-10-11 Daniel Bates <daba...@apple.com>
Added: branches/safari-602-branch/LayoutTests/css3/masking/large-clip-path-expected.txt (0 => 207746)
--- branches/safari-602-branch/LayoutTests/css3/masking/large-clip-path-expected.txt (rev 0)
+++ branches/safari-602-branch/LayoutTests/css3/masking/large-clip-path-expected.txt 2016-10-24 07:18:43 UTC (rev 207746)
@@ -0,0 +1,3 @@
+Test passes if there is no ASSERT in debug.
+
+
Added: branches/safari-602-branch/LayoutTests/css3/masking/large-clip-path.html (0 => 207746)
--- branches/safari-602-branch/LayoutTests/css3/masking/large-clip-path.html (rev 0)
+++ branches/safari-602-branch/LayoutTests/css3/masking/large-clip-path.html 2016-10-24 07:18:43 UTC (rev 207746)
@@ -0,0 +1,22 @@
+<!DOCTYPE html>
+
+<html>
+<head>
+ <style>
+ .box {
+ height: 100px;
+ width: 100px;
+ background-color: silver;
+ -webkit-clip-path: polygon(26px 52px,21.938px 51.959px,18.314px 51.821px,15.105px 51.561px,12.286px 51.157px,9.832px 50.583px,7.72px 49.814px,5.925px 48.827px,4.422px 47.598px,3.187px 46.103px,2.196px 44.317px,1.424px 42.216px,.847px 39.777px,.441px 36.974px,.181px 33.785px,.043px 30.184px,0 26.148px,.039px 22.182px,.173px 18.626px,.426px 15.459px,.824px 12.66px,1.392px 10.207px,2.155px 8.08px,3.137px 6.257px,4.365px 4.716px,5.863px 3.437px,7.655px 2.397px,9.767px 1.575px,12.224px .951px,15.051px .503px,18.273px .209px,21.915px .048px,26px 0,29.939px .057px,33.472px .24px,36.619px .567px,39.4px 1.056px,41.838px 1.725px,43.953px 2.592px,45.765px 3.676px,47.298px 4.994px,48.571px 6.565px,49.607px 8.406px,50.425px 10.536px,51.047px 12.973px,51.494px 15.734px,51.787px 18.838px,51.947px 22.303px,52px 26.148px,51.971px 30.087px,51.865px 33.615px,51.651px 36.754px,51.302px 39.526px,50.788px 41.953px,50.08px 44.056px,49.15px 45.856px,47.969px 47.376px,46.508px 48.636px,4
4.737px 49.657px,42.629px 50.464px,40.152px 51.074px,37.28px 51.512px,33.983px 51.798px,30.233px 51.954px,26px 52px,26px 52px,26px 52px,26px 52px,26px 52px,26px 52px,26px 52px,26px 52px,26px 52px,26px 52px,26px 52px,26px 52px,26px 52px,26px 52px,26px 52px,26px 52px,26px 52px);
+ }
+ </style>
+ <script>
+ if (window.testRunner)
+ testRunner.dumpAsText();
+ </script>
+</head>
+<body>
+<p>Test passes if there is no ASSERT in debug.</p>
+<div class="box"></div>
+</body>
+</html>
Modified: branches/safari-602-branch/Source/WebCore/ChangeLog (207745 => 207746)
--- branches/safari-602-branch/Source/WebCore/ChangeLog 2016-10-24 07:18:39 UTC (rev 207745)
+++ branches/safari-602-branch/Source/WebCore/ChangeLog 2016-10-24 07:18:43 UTC (rev 207746)
@@ -1,5 +1,24 @@
2016-10-20 Matthew Hanson <matthew_han...@apple.com>
+ Merge r207274. rdar://problem/28849629
+
+ 2016-10-12 Simon Fraser <simon.fra...@apple.com>
+
+ Crash when using megaplan.ru
+ https://bugs.webkit.org/show_bug.cgi?id=163276
+ rdar://problem/28446672
+
+ Reviewed by Sam Weinig.
+
+ Make sure we allocate enough space in the vector of CGPoints that we use for path building.
+
+ Test: css3/masking/large-clip-path.html
+
+ * platform/graphics/cg/PathCG.cpp:
+ (WebCore::Path::polygonPathFromPoints):
+
+2016-10-20 Matthew Hanson <matthew_han...@apple.com>
+
Merge r207159. rdar://problem/28857481
2016-10-11 Daniel Bates <daba...@apple.com>
Modified: branches/safari-602-branch/Source/WebCore/platform/graphics/cg/PathCG.cpp (207745 => 207746)
--- branches/safari-602-branch/Source/WebCore/platform/graphics/cg/PathCG.cpp 2016-10-24 07:18:39 UTC (rev 207745)
+++ branches/safari-602-branch/Source/WebCore/platform/graphics/cg/PathCG.cpp 2016-10-24 07:18:43 UTC (rev 207746)
@@ -80,7 +80,7 @@
return path;
Vector<CGPoint, 32> cgPoints;
- cgPoints.reserveInitialCapacity(points.size() - 1);
+ cgPoints.reserveInitialCapacity(points.size());
for (size_t i = 0; i < points.size(); ++i)
cgPoints.uncheckedAppend(points[i]);
_______________________________________________ webkit-changes mailing list webkit-changes@lists.webkit.org https://lists.webkit.org/mailman/listinfo/webkit-changes