LayoutTests/svg/custom/getBBox-ellipse-has-one-zero-value.html - Issue 208323007: Fix getBBox() returning (0,0) bug when width or height is zero

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: LayoutTests/svg/custom/getBBox-ellipse-has-one-zero-value.html

Issue 208323007: Fix getBBox() returning (0,0) bug when width or height is zero (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@myzbackup

Patch Set: Created 6 years, 9 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 | « LayoutTests/svg/custom/getBBox-circle-has-one-zero-value-expected.txt ('k') | LayoutTests/svg/custom/getBBox-ellipse-has-one-zero-value-expected.txt » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
(Empty)
	1 <!DOCTYPE HTML>

	2 <html>

	3 <!-- Test that svg bounding boxes are valid for a zero-width or zero-height elli pse -->

	4 <head>

	5 <script type="text/javascript">

	6 function checkBoundingBoxHasZeroValue() {

	7 if (window.testRunner)

	8 window.testRunner.dumpAsText();

	9 var ellipse1 = document.getElementById('ellipse1').getBBox();

	10 var ellipse2 = document.getElementById('ellipse2').getBBox();

	11 var ellipse3 = document.getElementById('ellipse3').getBBox();

	12 var ellipse4 = document.getElementById('ellipse4').getBBox();

	13 var results = "FAIL";

	14 if (ellipse1.height == 100 &&

	15 ellipse1.width == 0 &&

	16 ellipse2.height == 0 &&

	17 ellipse2.width == 100 &&

	18 ellipse3.height == 100 &&

	19 ellipse3.width == 0 &&

	20 ellipse4.height == 0 &&

	21 ellipse4.width == 100)

	22 results = "PASS";

	23 document.body.innerHTML = results + ", bounding boxes sizes are (" +

	24 ellipse1.width + ", " + ellipse1.height + "), (" +

	25 ellipse2.width + ", " + ellipse2.height + "), (" +

	26 ellipse3.width + ", " + ellipse3.height + ") and (" +

	27 ellipse4.width + ", " + ellipse4.height + ")";

	28 }

	29 </script>

	30 </head>

	31 <body onload="checkBoundingBoxHasZeroValue()">

	32 <svg>

	33 <ellipse id="ellipse1" cx="50" cy="100" rx="0" ry="50"/>

	34 <ellipse id="ellipse2" cx="50" cy="100" rx="50" ry="0"/>

	35 <ellipse id="ellipse3" vector-effect="non-scaling-stroke" cx="50" cy ="100" rx="0" ry="50"/>

	36 <ellipse id="ellipse4" vector-effect="non-scaling-stroke" cx="50" cy ="100" rx="50" ry="0"/>

	37 </svg>

	38 <body>

	39 </html>

OLD	NEW