third_party/WebKit/LayoutTests/fast/canvas/winding-enumeration.html - Issue 2311223002: Convert fast/canvas Layouttests to testharness

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: third_party/WebKit/LayoutTests/fast/canvas/winding-enumeration.html

Issue 2311223002: Convert fast/canvas Layouttests to testharness (Closed)

Patch Set: Convert fast/canvas Layouttests to testharness Created 4 years, 3 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch

« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/fast/canvas/winding-enumeration-expected.txt » ('j') | third_party/WebKit/LayoutTests/fast/canvas/zero-size-fill-rect.html » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 <!DOCTYPE HTML>	1 <!DOCTYPE html>

2 <html>	2 <title>Test CanvasWindingRule enumeration</title>

3 <head>

4 <script src="../../resources/js-test.js"></script>

5 </head>

6 <body>

7	3

	4 <script src="../../resources/testharness.js"></script>

	5 <script src="../../resources/testharnessreport.js"></script>

	6

	7 <canvas></canvas>
	fs 2016/09/08 10:13:10 Probably good the way it was (w/ createElement.) Probably good the way it was (w/ createElement.) Abhishek Kanike 2016/09/14 04:33:57 Done. Show quoted text On 2016/09/08 10:13:10, fs wrote: > Probably good the way it was (w/ createElement.) Done.
8 <script>	8 <script>

9 var canvas = document.createElement("canvas");	9 test(function() {

	10 var canvas = document.querySelector('canvas');
	fs 2016/09/08 10:13:10 Indent 2 or 4 spaces within the test-function body Indent 2 or 4 spaces within the test-function body. Abhishek Kanike 2016/09/14 04:33:57 Done. Show quoted text On 2016/09/08 10:13:10, fs wrote: > Indent 2 or 4 spaces within the test-function body. Done.
10 var context = canvas.getContext("2d");	11 var context = canvas.getContext("2d");

11	12

12 shouldNotThrow("context.fill()");	13 context.fill();

13 shouldNotThrow("context.fill('nonzero')");	14 context.fill('nonzero');

14 shouldThrow("context.fill('randomstring')");	15 assert_throws(new TypeError(), function() { context.fill('randomstring'); });

15	16

16 shouldNotThrow("context.clip()");	17 context.clip();

17 shouldNotThrow("context.clip('nonzero')");	18 context.clip('nonzero');

18 shouldThrow("context.clip('randomstring')");	19 assert_throws(new TypeError(), function() { context.clip('randomstring'); });

19	20

20 shouldNotThrow("context.isPointInPath(0, 0)");	21 context.isPointInPath(0, 0);

21 shouldNotThrow("context.isPointInPath(0, 0, 'nonzero')");	22 context.isPointInPath(0, 0, 'nonzero');

22 shouldThrow("context.isPointInPath(0, 0, 'randomstring')");	23 assert_throws(new TypeError(), function() { context.isPointInPath(0, 0, 'randoms tring'); });

23 </script>	24 });

24	25 </script>

25 </body>

26

OLD	NEW