reflow comments in platform/fonts

third_party/WebKit/Source/platform/fonts/mac/FontCacheMac.mm

 /*
  * Copyright (C) 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
  * Copyright (C) 2007 Nicholas Shanks <webkit@nickshanks.com>
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 128 matching lines @@
                                      inLanguage:nil];
   [string release];
 
   // FIXME: Remove this SPI usage: http://crbug.com/255122
   if (!substituteFont && codeUnitsLength == 1)
     substituteFont =
         [NSFont findFontLike:nsFont forCharacter:codeUnits[0] inLanguage:nil];
   if (!substituteFont)
     return nullptr;
 
-  // Use the family name from the AppKit-supplied substitute font, requesting the
-  // traits, weight, and size we want. One way this does better than the original
-  // AppKit request is that it takes synthetic bold and oblique into account.
-  // But it does create the possibility that we could end up with a font that
-  // doesn't actually cover the characters we need.
+  // Use the family name from the AppKit-supplied substitute font, requesting
+  // the traits, weight, and size we want. One way this does better than the
+  // original AppKit request is that it takes synthetic bold and oblique into
+  // account.  But it does create the possibility that we could end up with a
+  // font that doesn't actually cover the characters we need.
 
   NSFontManager* fontManager = [NSFontManager sharedFontManager];
 
   NSFontTraitMask traits;
   NSInteger weight;
   CGFloat size;
 
   if (nsFont) {
     traits = [fontManager traitsOfFont:nsFont];
     if (platformData.m_syntheticBold)
@@ skipping 27 matching lines @@
         substituteFont = bestVariation;
     }
   }
 
   substituteFont =
       useHinting() ? [substituteFont screenFont] : [substituteFont printerFont];
 
   substituteFontTraits = [fontManager traitsOfFont:substituteFont];
   substituteFontWeight = [fontManager weightOfFont:substituteFont];
 
-  // TODO(eae): Remove once skia supports bold emoji. See https://bugs.chromium.org/p/skia/issues/detail?id=4904
+  // TODO(eae): Remove once skia supports bold emoji. See
+  // https://bugs.chromium.org/p/skia/issues/detail?id=4904
   // Bold emoji look the same as normal emoji, so syntheticBold isn't needed.
   bool syntheticBold =
       isAppKitFontWeightBold(weight) &&
       !isAppKitFontWeightBold(substituteFontWeight) &&
       ![substituteFont.familyName isEqual:@"Apple Color Emoji"];
 
   FontPlatformData alternateFont(
       substituteFont, platformData.size(), syntheticBold,
       (traits & NSFontItalicTrait) &&
           !(substituteFontTraits & NSFontItalicTrait),
       platformData.orientation());
 
   return fontDataFromFontPlatformData(&alternateFont, DoNotRetain);
 }
 
 PassRefPtr<SimpleFontData> FontCache::getLastResortFallbackFont(
     const FontDescription& fontDescription,
     ShouldRetain shouldRetain) {
   DEFINE_STATIC_LOCAL(AtomicString, timesStr, ("Times"));
 
-  // FIXME: Would be even better to somehow get the user's default font here.  For now we'll pick
-  // the default that the user would get without changing any prefs.
+  // FIXME: Would be even better to somehow get the user's default font here.
+  // For now we'll pick the default that the user would get without changing
+  // any prefs.
   RefPtr<SimpleFontData> simpleFontData =
       getFontData(fontDescription, timesStr, false, shouldRetain);
   if (simpleFontData)
     return simpleFontData.release();
 
-  // The Times fallback will almost always work, but in the highly unusual case where
-  // the user doesn't have it, we fall back on Lucida Grande because that's
-  // guaranteed to be there, according to Nathan Taylor. This is good enough
-  // to avoid a crash at least.
+  // The Times fallback will almost always work, but in the highly unusual case
+  // where the user doesn't have it, we fall back on Lucida Grande because
+  // that's guaranteed to be there, according to Nathan Taylor. This is good
+  // enough to avoid a crash at least.
   DEFINE_STATIC_LOCAL(AtomicString, lucidaGrandeStr, ("Lucida Grande"));
   return getFontData(fontDescription, lucidaGrandeStr, false, shouldRetain);
 }
 
 std::unique_ptr<FontPlatformData> FontCache::createFontPlatformData(
     const FontDescription& fontDescription,
     const FontFaceCreationParams& creationParams,
     float fontSize) {
   NSFontTraitMask traits = fontDescription.style() ? NSFontItalicTrait : 0;
   float size = fontSize;
 
   NSFont* nsFont = MatchNSFontFamily(creationParams.family(), traits,
                                      fontDescription.weight(), size);
   if (!nsFont)
     return nullptr;
 
   NSFontManager* fontManager = [NSFontManager sharedFontManager];
   NSFontTraitMask actualTraits = 0;
   if (fontDescription.style())
     actualTraits = [fontManager traitsOfFont:nsFont];
   NSInteger actualWeight = [fontManager weightOfFont:nsFont];
 
   NSFont* platformFont =
       useHinting() ? [nsFont screenFont] : [nsFont printerFont];
   NSInteger appKitWeight = toAppKitFontWeight(fontDescription.weight());
 
-  // TODO(eae): Remove once skia supports bold emoji. See https://bugs.chromium.org/p/skia/issues/detail?id=4904
+  // TODO(eae): Remove once skia supports bold emoji. See
+  // https://bugs.chromium.org/p/skia/issues/detail?id=4904
   // Bold emoji look the same as normal emoji, so syntheticBold isn't needed.
   bool syntheticBold = [platformFont.familyName isEqual:@"Apple Color Emoji"]
                            ? false
                            : (isAppKitFontWeightBold(appKitWeight) &&
                               !isAppKitFontWeightBold(actualWeight)) ||
                                  fontDescription.isSyntheticBold();
 
   bool syntheticItalic =
       ((traits & NSFontItalicTrait) && !(actualTraits & NSFontItalicTrait)) ||
       fontDescription.isSyntheticItalic();
 
-  // FontPlatformData::typeface() is null in the case of Chromium out-of-process font loading failing.
-  // Out-of-process loading occurs for registered fonts stored in non-system locations.
-  // When loading fails, we do not want to use the returned FontPlatformData since it will not have
-  // a valid SkTypeface.
+  // FontPlatformData::typeface() is null in the case of Chromium out-of-process
+  // font loading failing.  Out-of-process loading occurs for registered fonts
+  // stored in non-system locations.  When loading fails, we do not want to use
+  // the returned FontPlatformData since it will not have a valid SkTypeface.
   std::unique_ptr<FontPlatformData> platformData = wrapUnique(
       new FontPlatformData(platformFont, size, syntheticBold, syntheticItalic,
                            fontDescription.orientation()));
   if (!platformData->typeface()) {
     return nullptr;
   }
   return platformData;
 }
 
 }  // namespace blink