Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(39)

Unified Diff: third_party/WebKit/LayoutTests/fast/canvas/canvas-text-baseline-tiny-fonts.html

Issue 2392033002: Correcting text baseline for tiny fonts (Closed)
Patch Set: Rebaseline. Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/fast/canvas/canvas-text-baseline-tiny-fonts.html
diff --git a/third_party/WebKit/LayoutTests/fast/canvas/canvas-text-baseline-tiny-fonts.html b/third_party/WebKit/LayoutTests/fast/canvas/canvas-text-baseline-tiny-fonts.html
new file mode 100644
index 0000000000000000000000000000000000000000..486b840e553a8462e54b77277df8cfe1daed74ba
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/fast/canvas/canvas-text-baseline-tiny-fonts.html
@@ -0,0 +1,37 @@
+<!DOCTYPE html>
+<canvas id="canvas1" width=200 height=250></canvas>
+<canvas id="canvas2" width=250 height=250></canvas>
+<canvas id="canvas3" width=250 height=250></canvas>
+<canvas id="canvas4" width=250 height=250></canvas>
+
+<script>
+if (window.testRunner)
+ window.testRunner.dumpAsTextWithPixelResults();
+
+function draw() {
+ // The code works fine with 1px font size, however testing that makes
+ // the test flaky due to pixel differences.
+ for (i = 2; i <= 4; i++) {
+ var ctx = document.getElementById("canvas" + i).getContext("2d");
+ ctx.scale(25, 25);
+ ctx.lineWidth = 0.1;
+
+ ctx.beginPath();
+ ctx.moveTo(0, 5);
+ ctx.lineTo(20, 5);
+ ctx.stroke();
+
+ ctx.font = i + "px sans-serif";
+ ctx.textBaseline = "top";
+ ctx.strokeText("A", 1, 5);
+ ctx.textBaseline = "middle"
+ ctx.strokeText("B", 3, 5);
+ ctx.textBaseline = "alphabetic"
+ ctx.strokeText("C", 5, 5);
+ ctx.textBaseline = "bottom";
+ ctx.strokeText("D", 7, 5);
+ }
+}
+
+window.onload = draw;
+</script>

Powered by Google App Engine
This is Rietveld 408576698