SkPDF: use sk_make_sp<T> when it makes sense.

src/pdf/SkPDFShader.cpp

Index: src/pdf/SkPDFShader.cpp
diff --git a/src/pdf/SkPDFShader.cpp b/src/pdf/SkPDFShader.cpp
index 8d5ee5a753697450285b73851c96e59612a3c324..a1e43a5e54418ec635149fbdfc1d028cbfb6e065 100644
--- a/src/pdf/SkPDFShader.cpp
+++ b/src/pdf/SkPDFShader.cpp
@@ -771,7 +771,7 @@ SkPDFFunctionShader* SkPDFFunctionShader::Create(
         return nullptr;
     }
 
-    sk_sp<SkPDFArray> domain(new SkPDFArray);
+    auto domain = sk_make_sp<SkPDFArray>();
     domain->reserve(4);
     domain->appendScalar(bbox.fLeft);
     domain->appendScalar(bbox.fRight);
@@ -799,7 +799,7 @@ SkPDFFunctionShader* SkPDFFunctionShader::Create(
         functionCode = codeFunction(*info, perspectiveInverseOnly);
     }
 
-    sk_sp<SkPDFDict> pdfShader(new SkPDFDict);
+    auto pdfShader = sk_make_sp<SkPDFDict>();
     pdfShader->insertInt("ShadingType", 1);
     pdfShader->insertName("ColorSpace", "DeviceRGB");
     pdfShader->insertObject("Domain", SkRef(domain.get()));
@@ -808,7 +808,7 @@ SkPDFFunctionShader* SkPDFFunctionShader::Create(
             make_ps_function(functionCode, domain.get()));
     pdfShader->insertObjRef("Function", function.release());
 
-    sk_sp<SkPDFFunctionShader> pdfFunctionShader(new SkPDFFunctionShader(autoState->detach()));
+    auto pdfFunctionShader = sk_make_sp<SkPDFFunctionShader>(autoState->detach());

[bungeman-skia, 2016/03/08 02:20:36]

This appears to be the reason for trying to make the constructor public. 

[hal.canary, 2016/03/08 11:58:48]

yep.  changed back.

 
     pdfFunctionShader->insertInt("PatternType", 2);
     pdfFunctionShader->insertObject("Matrix",