commit ef14cb397cca75607019e2cc401d4bb9969ed105
author caryclark <caryclark@google.com> Thu Jul 16 14:16:04 2015 -0700
committer Commit bot <commit-bot@chromium.org> Thu Jul 16 14:16:04 2015 -0700
tree 4af9b4ab7579235f79dc6cb5643240e75b2e51f4
parent 31b21f67190916b94c5345126ddab327a8feac57

make glyph_pos imageblur* largeglyphblur portable

R=reed@google.com,bungeman@google.com

Review URL: https://codereview.chromium.org/1243493003

gm/glyph_pos.cpp

diff --git a/gm/glyph_pos.cpp b/gm/glyph_pos.cpp
index 16cdfca..90329ed 100644
--- a/gm/glyph_pos.cpp
+++ b/gm/glyph_pos.cpp
@@ -48,9 +48,6 @@
     SkISize onISize() override { return SkISize::Make(800, 600); }
 
     void onDraw(SkCanvas* canvas) override {
-        if (!fProp) {
-            fProp.reset(sk_tool_utils::create_portable_typeface("Helvetica", SkTypeface::kNormal));
-        }
 
         // There's a black pixel at 40, 40 for reference.
         canvas->drawPoint(40.0f, 40.0f, SK_ColorBLACK);
@@ -108,8 +105,7 @@
         paint.setColor(SK_ColorBLACK);
         paint.setAntiAlias(true);
         paint.setTextSize(kTextHeight * textScale);
-        paint.setTypeface(fProp);
-        paint.setDevKernText(true);
+        sk_tool_utils::set_portable_typeface_always(&paint);
         paint.setStrokeWidth(fStrokeWidth);
         paint.setStyle(fStrokeStyle);
 
@@ -158,7 +154,6 @@
     }
 
 private:
-    SkAutoTUnref<SkTypeface> fProp;
     SkScalar fStrokeWidth;
     SkPaint::Style fStrokeStyle;
 

gm/imageblur.cpp

diff --git a/gm/imageblur.cpp b/gm/imageblur.cpp
index c7139b4..d0f1fd1 100644
--- a/gm/imageblur.cpp
+++ b/gm/imageblur.cpp
@@ -41,11 +41,11 @@
         SkRandom rand;
         SkPaint textPaint;
         textPaint.setAntiAlias(true);
-        sk_tool_utils::set_portable_typeface(&textPaint);
+        sk_tool_utils::set_portable_typeface_always(&textPaint);
         for (int i = 0; i < 25; ++i) {
             int x = rand.nextULessThan(WIDTH);
             int y = rand.nextULessThan(HEIGHT);
-            textPaint.setColor(rand.nextBits(24) | 0xFF000000);
+            textPaint.setColor(sk_tool_utils::color_to_565(rand.nextBits(24) | 0xFF000000));
             textPaint.setTextSize(rand.nextRangeScalar(0, 300));
             canvas->drawText(str, strlen(str), SkIntToScalar(x),
                              SkIntToScalar(y), textPaint);

gm/imageblurtiled.cpp

diff --git a/gm/imageblurtiled.cpp b/gm/imageblurtiled.cpp
index ed4ae6c..5b00b17 100644
--- a/gm/imageblurtiled.cpp
+++ b/gm/imageblurtiled.cpp
@@ -50,7 +50,7 @@
                 };
                 SkPaint textPaint;
                 textPaint.setAntiAlias(true);
-                sk_tool_utils::set_portable_typeface(&textPaint);
+                sk_tool_utils::set_portable_typeface_always(&textPaint);
                 textPaint.setTextSize(SkIntToScalar(100));
                 int posY = 0;
                 for (unsigned i = 0; i < SK_ARRAY_COUNT(str); i++) {

gm/largeglyphblur.cpp

diff --git a/gm/largeglyphblur.cpp b/gm/largeglyphblur.cpp
index 8723824..9d9ff41 100644
--- a/gm/largeglyphblur.cpp
+++ b/gm/largeglyphblur.cpp
@@ -32,7 +32,7 @@
         const char text[] = "Hamburgefons";
 
         SkPaint paint;
-        sk_tool_utils::set_portable_typeface(&paint);
+        sk_tool_utils::set_portable_typeface_always(&paint);
         paint.setTextSize(256);
         paint.setAntiAlias(true);