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

Unified Diff: third_party/WebKit/Source/core/css/resolver/FontBuilder.cpp

Issue 2399633003: reflow comments in core/css/resolver (Closed)
Patch Set: 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/core/css/resolver/FontBuilder.cpp
diff --git a/third_party/WebKit/Source/core/css/resolver/FontBuilder.cpp b/third_party/WebKit/Source/core/css/resolver/FontBuilder.cpp
index c2a4bbd31e12d085eb08465fa6d6795883e3e263..958592ff0f51f64b83773dd5293ce516018f72e3 100644
--- a/third_party/WebKit/Source/core/css/resolver/FontBuilder.cpp
+++ b/third_party/WebKit/Source/core/css/resolver/FontBuilder.cpp
@@ -1,6 +1,7 @@
/*
* Copyright (C) 1999 Lars Knoll (knoll@kde.org)
- * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
+ * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc.
+ * All rights reserved.
* Copyright (C) 2013 Google Inc. All rights reserved.
* Copyright (C) 2015 Collabora Ltd. All rights reserved.
*
@@ -208,8 +209,8 @@ void FontBuilder::setSize(FontDescription& fontDescription,
set(PropertySetFlag::Size);
- // Overly large font sizes will cause crashes on some platforms (such as Windows).
- // Cap font size here to make sure that doesn't happen.
+ // Overly large font sizes will cause crashes on some platforms (such as
+ // Windows). Cap font size here to make sure that doesn't happen.
specifiedSize = std::min(maximumAllowedFontSize, specifiedSize);
fontDescription.setKeywordSize(size.keyword);
@@ -311,8 +312,9 @@ void FontBuilder::updateAdjustedSize(FontDescription& fontDescription,
return;
// We need to create a temporal Font to get xHeight of a primary font.
- // The aspect value is based on the xHeight of the font for the computed font size,
- // so we need to reset the adjustedSize to computedSize. See FontDescription::effectiveFontSize.
+ // The aspect value is based on the xHeight of the font for the computed font
+ // size, so we need to reset the adjustedSize to computedSize. See
+ // FontDescription::effectiveFontSize.
fontDescription.setAdjustedSize(fontDescription.computedSize());
Font font(fontDescription);
« no previous file with comments | « third_party/WebKit/Source/core/css/resolver/FontBuilder.h ('k') | third_party/WebKit/Source/core/css/resolver/MatchRequest.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698