PDF: remove unused SkPDFDevice::setDCTEncoder()

src/pdf/SkPDFDevice.cpp

Index: src/pdf/SkPDFDevice.cpp
diff --git a/src/pdf/SkPDFDevice.cpp b/src/pdf/SkPDFDevice.cpp
index a15571a5b306b72dd441b7341aeb3d1c7891f153..329d0971dd0878af13b8dac960b06c084e8394bf 100644
--- a/src/pdf/SkPDFDevice.cpp
+++ b/src/pdf/SkPDFDevice.cpp
@@ -727,7 +727,6 @@ SkPDFDevice::SkPDFDevice(const SkISize& pageSize, const SkISize& contentSize,
     , fLastContentEntry(NULL)
     , fLastMarginContentEntry(NULL)
     , fClipStack(NULL)
-    , fEncoder(NULL)

[mtklein, 2015/02/20 03:36:13]

So... this was just always NULL?

[hal.canary, 2015/02/20 12:52:57]

The constructor never sets it.  You had to call ->setDCTEncoder(), which in
practice no one ever did.  Then I landed http://crrev.com/935843007 , which took
away the possibility that they will in the future.

     , fRasterDpi(72.0f)
 {
     const SkImageInfo info = make_content_info(contentSize, &initialTransform);
@@ -760,7 +759,6 @@ SkPDFDevice::SkPDFDevice(const SkISize& layerSize,
     , fLastContentEntry(NULL)
     , fLastMarginContentEntry(NULL)
     , fClipStack(NULL)
-    , fEncoder(NULL)
     , fRasterDpi(72.0f)
 {
     fInitialTransform.reset();
@@ -2178,7 +2176,7 @@ void SkPDFDevice::internalDrawBitmap(const SkMatrix& origMatrix,
     }
 
     SkAutoTUnref<SkPDFObject> image(
-            SkPDFCreateImageObject(*bitmap, subset, fEncoder));
+            SkPDFCreateImageObject(*bitmap, subset));
     if (!image) {
         return;
     }