Remove unused 'deepCopy' parameter

src/core/SkPicture.cpp

 
 /*
  * Copyright 2007 The Android Open Source Project
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 
 #include "SkPictureFlat.h"
@@ skipping 115 matching lines @@
 SkPicture::SkPicture()
     : fAccelData(NULL) {
     this->needsNewGenID();
     fPlayback = NULL;
     fWidth = fHeight = 0;
 }
 
 // This method makes a SkPicturePlayback object from an in-progress recording.
 // Unfortunately, it does not include the restoreToCount of a real endRecording
 // call.
 SkPicturePlayback* SkPicture::FakeEndRecording(const SkPicture* resourceSrc,

[scroggo, 2014/06/10 14:58:04]

As I asked in https://codereview.chromium.org/316063005/ , does it make sense to
move this into its only caller?

[robertphillips, 2014/06/10 15:00:09]

This entry point is going away entirely (replaced by a new SkPicture
constructor) so I don't think it is worth moving right now.

[scroggo, 2014/06/10 15:00:44]

sgtm. Thanks for the explanation.

-                                               const SkPictureRecord& record,
-                                               bool deepCopy) {
+                                               const SkPictureRecord& record) {
     SkPictInfo info;
     resourceSrc->createHeader(&info);
-    return SkNEW_ARGS(SkPicturePlayback, (resourceSrc, record, info, deepCopy));
+    return SkNEW_ARGS(SkPicturePlayback, (resourceSrc, record, info));
 }
 
 SkPicture::SkPicture(const SkPicture& src)
     : INHERITED()
     , fAccelData(NULL)
     , fContentInfo(src.fContentInfo) {
     this->needsNewGenID();
     fWidth = src.fWidth;
     fHeight = src.fHeight;
 
@@ skipping 413 matching lines @@
     } while (SK_InvalidGenID == genID);
     return genID;
 }
 
 uint32_t SkPicture::uniqueID() const {
     if (SK_InvalidGenID == fUniqueID) {
         fUniqueID = next_picture_generation_id();
     }
     return fUniqueID;
 }