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

Unified Diff: third_party/WebKit/Source/platform/fonts/GlyphPage.h

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/GlyphPage.h
diff --git a/third_party/WebKit/Source/platform/fonts/GlyphPage.h b/third_party/WebKit/Source/platform/fonts/GlyphPage.h
index 236c336f7b509e06af35357e4cb7bf79b042b8f7..8ab68e940376fe9de33f47b86c18d47a1cd2861b 100644
--- a/third_party/WebKit/Source/platform/fonts/GlyphPage.h
+++ b/third_party/WebKit/Source/platform/fonts/GlyphPage.h
@@ -46,8 +46,8 @@ namespace blink {
class SimpleFontData;
class GlyphPageTreeNodeBase;
-// Holds the glyph index and the corresponding SimpleFontData information for a given
-// character.
+// Holds the glyph index and the corresponding SimpleFontData information for a
+// given character.
struct GlyphData {
DISALLOW_NEW();
GlyphData(Glyph g = 0, const SimpleFontData* f = 0) : glyph(g), fontData(f) {}
@@ -147,7 +147,8 @@ class PLATFORM_EXPORT GlyphPage : public RefCounted<GlyphPage> {
if (hasPerGlyphFontData()) {
m_perGlyphFontData[index] = fontData;
} else {
- // A single-font GlyphPage already assigned m_fontDataForAllGlyphs in the constructor.
+ // A single-font GlyphPage already assigned m_fontDataForAllGlyphs in the
+ // constructor.
ASSERT(!glyph || fontData == m_fontDataForAllGlyphs);
}
}
@@ -173,7 +174,8 @@ class PLATFORM_EXPORT GlyphPage : public RefCounted<GlyphPage> {
}
void removePerGlyphFontData(const SimpleFontData* fontData) {
- // This method should only be called on the mixed page, which is never single-font.
+ // This method should only be called on the mixed page, which is never
+ // single-font.
ASSERT(hasPerGlyphFontData());
for (size_t i = 0; i < size; ++i) {
if (m_perGlyphFontData[i] == fontData) {
@@ -216,7 +218,8 @@ class PLATFORM_EXPORT GlyphPage : public RefCounted<GlyphPage> {
RefPtr<CustomDataPage> m_customFontToLoad;
Glyph m_glyphs[size];
- // NOTE: This array has (GlyphPage::size) elements if m_fontDataForAllGlyphs is null.
+ // NOTE: This array has (GlyphPage::size) elements if m_fontDataForAllGlyphs
+ // is null.
const SimpleFontData* m_perGlyphFontData[0];
};

Powered by Google App Engine
This is Rietveld 408576698