Revert of change all factories to return their base-class

tests/DrawPathTest.cpp

 /*
  * Copyright 2012 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkBitmap.h"
 #include "SkCanvas.h"
 #include "SkDashPathEffect.h"
@@ skipping 174 matching lines @@
 
      *  A different (better) fix might be to rewrite dashing to do all of its
      *  length/phase/measure math using double, but this may need to be
      *  coordinated with SkPathMeasure, to be consistent between the two.
 
      <path stroke="mintcream" stroke-dasharray="27734 35660 2157846850 247"
            stroke-dashoffset="-248.135982067">
      */
 
     const SkScalar vals[] = { 27734, 35660, 2157846850.0f, 247 };
-    SkAutoTUnref<SkPathEffect> dontAssert(SkDashPathEffect::Create(vals, 4, -248.135982067f));
+    SkAutoTUnref<SkDashPathEffect> dontAssert(SkDashPathEffect::Create(vals, 4, -248.135982067f));
 }
 
 static void test_crbug_124652() {
     /*
         http://code.google.com/p/chromium/issues/detail?id=124652
         This particular test/bug only applies to the float case, where
         large values can "swamp" small ones.
      */
     SkScalar intervals[2] = {837099584, 33450};
-    SkAutoTUnref<SkPathEffect> dash(SkDashPathEffect::Create(intervals, 2, -10));
+    SkAutoTUnref<SkDashPathEffect> dash(SkDashPathEffect::Create(intervals, 2, -10));
 }
 
 static void test_bigcubic() {
     SkPath path;
     path.moveTo(64, 3);
     path.cubicTo(-329936, -100000000, -329936, 100000000, 1153, 330003);
 
     SkPaint paint;
     paint.setAntiAlias(true);
 
@@ skipping 19 matching lines @@
 // Extremely large path_length/dash_length ratios may cause infinite looping
 // in SkDashPathEffect::filterPath() due to single precision rounding.
 // The test is quite expensive, but it should get much faster after the fix
 // for http://crbug.com/165432 goes in.
 static void test_infinite_dash(skiatest::Reporter* reporter) {
     SkPath path;
     path.moveTo(0, 0);
     path.lineTo(5000000, 0);
 
     SkScalar intervals[] = { 0.2f, 0.2f };
-    SkAutoTUnref<SkPathEffect> dash(SkDashPathEffect::Create(intervals, 2, 0));
+    SkAutoTUnref<SkDashPathEffect> dash(SkDashPathEffect::Create(intervals, 2, 0));
 
     SkPath filteredPath;
     SkPaint paint;
     paint.setStyle(SkPaint::kStroke_Style);
     paint.setPathEffect(dash);
 
     paint.getFillPath(path, &filteredPath);
     // If we reach this, we passed.
     REPORTER_ASSERT(reporter, true);
 }
 
 // http://crbug.com/165432
 // Limit extreme dash path effects to avoid exhausting the system memory.
 static void test_crbug_165432(skiatest::Reporter* reporter) {
     SkPath path;
     path.moveTo(0, 0);
     path.lineTo(10000000, 0);
 
     SkScalar intervals[] = { 0.5f, 0.5f };
-    SkAutoTUnref<SkPathEffect> dash(SkDashPathEffect::Create(intervals, 2, 0));
+    SkAutoTUnref<SkDashPathEffect> dash(SkDashPathEffect::Create(intervals, 2, 0));
 
     SkPaint paint;
     paint.setStyle(SkPaint::kStroke_Style);
     paint.setPathEffect(dash);
 
     SkPath filteredPath;
     SkStrokeRec rec(paint);
     REPORTER_ASSERT(reporter, !dash->filterPath(&filteredPath, path, &rec, nullptr));
     REPORTER_ASSERT(reporter, filteredPath.isEmpty());
 }
 
 DEF_TEST(DrawPath, reporter) {
     test_giantaa();
     test_bug533();
     test_bigcubic();
     test_crbug_124652();
     test_crbug_140642();
     test_crbug_140803();
     test_inversepathwithclip();
     // why?
     if (false) test_crbug131181();
     test_infinite_dash(reporter);
     test_crbug_165432(reporter);
     test_big_aa_rect(reporter);
 }