Change DebugCanvas API to not encourage memory leaks

src/utils/debugger/SkDebugCanvas.h

 
 /*
  * Copyright 2012 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 
 #ifndef SKDEBUGCANVAS_H_
@@ skipping 105 matching lines @@
     SK_ATTR_DEPRECATED("please use getDrawCommandAt and getSize instead")
     const SkTDArray<SkDrawCommand*>& getDrawCommands() const;
 
     /**
         Returns the vector of draw commands. Do not use this entry
         point - it is going away!
      */
     SkTDArray<SkDrawCommand*>& getDrawCommands();
 
     /**
-     * Returns the string vector of draw commands
+     * Fills the string vector with draw commands 
      */
-    SkTArray<SkString>* getDrawCommandsAsStrings() const;
+    void getDrawCommandsAsStrings(SkTArray<SkString>* outCommandStrings) const;
 
     /**
-     * Returns an array containing an offset (in the SkPicture) for each command
+     * Fills the size_t array with an offset (in the SkPicture) for each command
      */
-    SkTDArray<size_t>* getDrawCommandOffsets() const;
+    void getDrawCommandOffsets(SkTDArray<size_t>* outCommandOffsets) const;
 
     /**
         Returns length of draw command vector.
      */
     int getSize() const {
         return fCommandVector.count();
     }
 
     /**
         Toggles the visibility / execution of the draw command at index i with
@@ skipping 167 matching lines @@
     bool lastClipStackData(const SkPath& devPath);
     void outputConicPoints(const SkPoint* pts, SkScalar weight);
     void outputPoints(const SkPoint* pts, int count);
     void outputPointsCommon(const SkPoint* pts, int count);
     void outputScalar(SkScalar num);
 
     typedef SkCanvas INHERITED;
 };
 
 #endif