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

Unified Diff: third_party/WebKit/Source/platform/fonts/shaping/HarfBuzzShaper.cpp

Issue 2386333002: reflow comments in platform/fonts (Closed)
Patch Set: comments 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/Source/platform/fonts/shaping/HarfBuzzShaper.cpp
diff --git a/third_party/WebKit/Source/platform/fonts/shaping/HarfBuzzShaper.cpp b/third_party/WebKit/Source/platform/fonts/shaping/HarfBuzzShaper.cpp
index 48c2b8031d1df8b316564da22e8cf290b0780d69..fa3a91a4d9211375ff325624ceab6c19c71cb0f7 100644
--- a/third_party/WebKit/Source/platform/fonts/shaping/HarfBuzzShaper.cpp
+++ b/third_party/WebKit/Source/platform/fonts/shaping/HarfBuzzShaper.cpp
@@ -601,7 +601,8 @@ PassRefPtr<ShapeResult> HarfBuzzShaper::shapeResult() {
Vector<UChar32> fallbackCharsHint;
// TODO: Check whether this treatAsZerowidthspace from the previous script
- // segmentation plays a role here, does the new scriptRuniterator handle that correctly?
+ // segmentation plays a role here, does the new scriptRuniterator handle that
+ // correctly?
while (runSegmenter.consume(&segmentRange)) {
RefPtr<FontFallbackIterator> fallbackIterator =
m_font->createFontFallbackIterator(segmentRange.fontFallbackPriority);
@@ -662,9 +663,10 @@ PassRefPtr<ShapeResult> HarfBuzzShaper::shapeResult() {
.get()
: currentFontDataForRangeSet->fontData();
- // Compatibility with SimpleFontData approach of keeping a flag for overriding drawing direction.
- // TODO: crbug.com/506224 This should go away in favor of storing that information elsewhere, for example in
- // ShapeResult.
+ // Compatibility with SimpleFontData approach of keeping a flag for
+ // overriding drawing direction.
+ // TODO: crbug.com/506224 This should go away in favor of storing that
+ // information elsewhere, for example in ShapeResult.
const SimpleFontData* directionAndSmallCapsAdjustedFont =
fontDataAdjustedForOrientation(
smallcapsAdjustedFont, m_font->getFontDescription().orientation(),

Powered by Google App Engine
This is Rietveld 408576698