Cleanup FontMac.mm
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 7 Nov 2013 00:00:14 +0000 (00:00 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 7 Nov 2013 00:00:14 +0000 (00:00 +0000)
https://bugs.webkit.org/show_bug.cgi?id=123928

Reviewed by Andy Estes.

* platform/graphics/mac/FontMac.mm:
(WebCore::showGlyphsWithAdvances): Inline the value of variable isVertical as we
reference it exactly once and its value is sufficiently clear.
(WebCore::Font::drawGlyphs): Remove default case in switch block so that the compiler
checks that all cases are covered. Move definition of platformData to the top of the
function and use it whenever we want to access the platform font data

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@158800 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/mac/FontMac.mm

index 0736fdd..261f0be 100644 (file)
@@ -1,3 +1,17 @@
+2013-11-06  Daniel Bates  <dabates@apple.com>
+
+        Cleanup FontMac.mm
+        https://bugs.webkit.org/show_bug.cgi?id=123928
+
+        Reviewed by Andy Estes.
+
+        * platform/graphics/mac/FontMac.mm:
+        (WebCore::showGlyphsWithAdvances): Inline the value of variable isVertical as we
+        reference it exactly once and its value is sufficiently clear.
+        (WebCore::Font::drawGlyphs): Remove default case in switch block so that the compiler
+        checks that all cases are covered. Move definition of platformData to the top of the
+        function and use it whenever we want to access the platform font data
+
 2013-11-06  Brent Fulgham  <bfulgham@apple.com>
 
         [WebGL] We should not allow generateMipMap on compressed textures
index ff80cfb..5e4d9aa 100644 (file)
@@ -71,8 +71,7 @@ static void showGlyphsWithAdvances(const FloatPoint& point, const SimpleFontData
             positions[i].y = positions[i - 1].y + advance.height;
         }
     }
-    bool isVertical = font->platformData().orientation() == Vertical;
-    if (isVertical) {
+    if (platformData.orientation() == Vertical) {
         CGAffineTransform savedMatrix;
         CGAffineTransform rotateLeftTransform = CGAffineTransformMake(0, -1, 1, 0, 0, 0);
         savedMatrix = CGContextGetTextMatrix(context);
@@ -109,13 +108,14 @@ static void showGlyphsWithAdvances(const FloatPoint& point, const SimpleFontData
 
 void Font::drawGlyphs(GraphicsContext* context, const SimpleFontData* font, const GlyphBuffer& glyphBuffer, int from, int numGlyphs, const FloatPoint& point) const
 {
-    if (!font->platformData().size())
+    const FontPlatformData& platformData = font->platformData();
+    if (!platformData.size())
         return;
 
     CGContextRef cgContext = context->platformContext();
 
-    bool shouldSmoothFonts = true;
-    bool changeFontSmoothing = false;
+    bool shouldSmoothFonts;
+    bool changeFontSmoothing;
     
     switch(fontDescription().fontSmoothing()) {
     case Antialiased: {
@@ -137,11 +137,10 @@ void Font::drawGlyphs(GraphicsContext* context, const SimpleFontData* font, cons
         break;
     }
     case AutoSmoothing: {
-        // For the AutoSmooth case, don't do anything! Keep the default settings.
-        break; 
+        shouldSmoothFonts = true;
+        changeFontSmoothing = false;
+        break;
     }
-    default: 
-        ASSERT_NOT_REACHED();
     }
     
     if (!shouldUseSmoothing()) {
@@ -155,7 +154,6 @@ void Font::drawGlyphs(GraphicsContext* context, const SimpleFontData* font, cons
         CGContextSetShouldSmoothFonts(cgContext, shouldSmoothFonts);
     }
 
-    const FontPlatformData& platformData = font->platformData();
     NSFont* drawFont;
     if (!isPrinterFont()) {
         drawFont = [platformData.font() screenFont];
@@ -179,7 +177,7 @@ void Font::drawGlyphs(GraphicsContext* context, const SimpleFontData* font, cons
     matrix.d = -matrix.d;
     if (platformData.m_syntheticOblique) {
         static float obliqueSkew = tanf(SYNTHETIC_OBLIQUE_ANGLE * piFloat / 180);
-        if (font->platformData().orientation() == Vertical)
+        if (platformData.orientation() == Vertical)
             matrix = CGAffineTransformConcat(matrix, CGAffineTransformMake(1, obliqueSkew, 0, 1, 0, 0));
         else
             matrix = CGAffineTransformConcat(matrix, CGAffineTransformMake(1, 0, -obliqueSkew, 1, 0, 0));