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

Unified Diff: LayoutTests/fast/text/font-ligature-letter-spacing.html

Issue 325673002: Fixing handling of Ligature when letter-spacing is present (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Code Issue fix Created 6 years, 6 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
« no previous file with comments | « no previous file | LayoutTests/fast/text/font-ligature-letter-spacing-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/fast/text/font-ligature-letter-spacing.html
diff --git a/LayoutTests/fast/text/font-ligature-letter-spacing.html b/LayoutTests/fast/text/font-ligature-letter-spacing.html
new file mode 100644
index 0000000000000000000000000000000000000000..df295aebea8deb041f8849d665401480b722c760
--- /dev/null
+++ b/LayoutTests/fast/text/font-ligature-letter-spacing.html
@@ -0,0 +1,56 @@
+<!DOCTYPE html>
+<html>
+<head>
+<meta http-equiv="Content-Type" content="text/html;charset=utf-8">
+<title>Letter spacing and Ligature</title>
+
+<style type="text/css">
+@font-face {
+ font-family: megalopolis;
+ src: url(../../third_party/MEgalopolis/MEgalopolisExtra.woff) format("woff");
+}
+
+.dligDiv {
+-moz-font-feature-settings:"frac" 1, "dlig" 1;
+-moz-font-feature-settings:"frac=1, dlig=1";
+-ms-font-feature-settings:"frac" 1, "dlig" 1;
+-o-font-feature-settings:"frac" 1, "dlig" 1;
+-webkit-font-feature-settings:"frac" 1, "dlig" 1;
+font-feature-settings:"frac" 1, "dlig" 1;
+}
+
+.common {
+font-size: 24px;
+line-height: 100%;
+padding: 0px;
+letter-spacing:20px;
+font-family: megalopolis;
+}
+
+p { font-family: serif; font-style: italic; }
+</style>
+
+<script src="../../resources/testharness.js"></script>
+<script src="../../resources/testharnessreport.js"></script>
+<script>
+ setup({ explicit_done: true });
+ function testLetterSpaceAndLigature() {
+ var elementWidthWithLigatureAndLetterSpacing = document.getElementsByClassName("dligSpan")[0].getBoundingClientRect().width;
+ var elementWidthWithLetterSpacing = document.getElementsByClassName("letterSpace")[0].getBoundingClientRect().width;
+ test(function() {
+ assert_equals(elementWidthWithLigatureAndLetterSpacing, elementWidthWithLetterSpacing, "Ligature not applied due to letter spacing.");
+ }, "Ligature expected not to be applied due to letter spacing.");
+ done();
+ }
+</script>
+</head>
+<body onload="testLetterSpaceAndLigature();">
+<div class="dligDiv common">
+ <span class="dligSpan">CACACACA</span>
+</div>
+
+<div class="common">
+ <span class="letterSpace">CACACACA</span>
+</div>
+</body>
+</html>
« no previous file with comments | « no previous file | LayoutTests/fast/text/font-ligature-letter-spacing-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698