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

Unified Diff: Source/platform/fonts/GlyphPageTreeNode.cpp

Issue 509373002: This is code readability patch, No Layout Test case needed (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Comment fixes Created 6 years, 4 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: Source/platform/fonts/GlyphPageTreeNode.cpp
diff --git a/Source/platform/fonts/GlyphPageTreeNode.cpp b/Source/platform/fonts/GlyphPageTreeNode.cpp
index 052e90924004d4d8cc4e32c7d18521b0acc2042e..86ea3d84b21d745775820f6039329f3bf39e93d8 100644
--- a/Source/platform/fonts/GlyphPageTreeNode.cpp
+++ b/Source/platform/fonts/GlyphPageTreeNode.cpp
@@ -170,9 +170,9 @@ void GlyphPageTreeNode::initializePage(const FontData* fontData, unsigned pageNu
buffer[softHyphen] = zeroWidthSpace;
// \n, \t, and nonbreaking space must render as a space.
- buffer[(int)'\n'] = ' ';
- buffer[(int)'\t'] = ' ';
- buffer[noBreakSpace] = ' ';
+ buffer[newlineCharacter] = space;
+ buffer[tabulationCharacter] = space;
+ buffer[noBreakSpace] = space;
} else if (start == (leftToRightMark & ~(GlyphPage::size - 1))) {
// LRM, RLM, LRE, RLE, ZWNJ, ZWJ, and PDF must not render at all.
buffer[leftToRightMark - start] = zeroWidthSpace;

Powered by Google App Engine
This is Rietveld 408576698