Rearrange SkRecord public API to fit better with cc/resources/picture

include/record/SkRecording.h

 /*
  * Copyright 2014 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef SkRecording_DEFINED
 #define SkRecording_DEFINED
 
-#include "SkCanvas.h"  // SkCanvas
-#include "SkTypes.h"   // SkNoncopyable
+#include "SkCanvas.h"     // SkCanvas
+#include "SkRefCnt.h"     // SkAutoTUnref
+#include "SkTemplates.h"  // SkAutoTDelete
+#include "SkTypes.h"      // SkNoncopyable
 
 // These are intentionally left opaque.
 class SkRecord;
 class SkRecorder;
 
 namespace EXPERIMENTAL {
 
 /** Easy mode interface to SkRecord-based SkCanvas recording.
  *
- *  SkRecording* recording = SkRecording::Create(1920, 1080);
+ *  scoped_ptr<SkRecording> recording(new SkRecording(1920, 1080));

[reed1, 2014/04/23 21:53:30]

its not wrong, but I don't think our pseudo code in dox needs to use these
scoping helpers.

[mtklein, 2014/04/23 22:05:54]

It's intended for Chrome readers.  Want me to rewrite it in terms of
SkAutoTUnref and co?

+ *  skia::RefPtr<SkCanvas> canvas(skia::SharePtr(recording->canvas()));
  *
- *  SkCanvas* canvas = recording->canvas();
  *  canvas->drawThis();
  *  canvas->clipThat();
  *  ...
  *
- *  scoped_ptr<const SkPlayback> playback(SkRecording::Delete(recording));
+ *  canvas.clear();  // You must deref the canvas before you may call releasePlayback().
+ *  scoped_ptr<const SkPlayback> playback(recording->releasePlayback());
  *  playback->draw(&someCanvas);
  *  playback->draw(&someOtherCanvas);
  *
  *  SkPlayback is thread safe; SkRecording is not.
  */
 
 class SkPlayback : SkNoncopyable {
 public:
     // Remember, if you've got an SkPlayback*, you probably own it.  Don't forget to delete it!
     ~SkPlayback();
 
     // Draw recorded commands into a canvas.
     void draw(SkCanvas*) const;
 
 private:
     explicit SkPlayback(const SkRecord*);
 
-    const SkRecord* fRecord;
+    SkAutoTDelete<const SkRecord> fRecord;
 
     friend class SkRecording;
 };
 
 class SkRecording : SkNoncopyable {
 public:
-    // Result must be returned via SkRecording::Delete.
-    static SkRecording* Create(int width, int height);
-
-    // Caller takes ownership of SkPlayback.
-    static const SkPlayback* Delete(SkRecording*);
-
-    // Draws issued to this canvas will be replayed by SkPlayback::draw().
-    // This pointer is owned by the SkRecording; the caller must not take ownership.
-    SkCanvas* canvas();
-
-private:
     SkRecording(int width, int height);
     ~SkRecording();
 
-    SkRecorder* fRecorder;
-    SkRecord* fRecord;
+    // Draws issued to this canvas will be replayed by SkPlayback::draw().
+    // Any refs held on canvas() must be dropped before you may call releasePlayback().
+    SkCanvas* canvas();
+
+    // Release exclusive ownership of an SkPlayback to the caller.
+    // Any refs held on canvas() must be dropped before you may call releasePlayback().
+    SkPlayback* releasePlayback();
+
+private:
+    SkAutoTDelete<SkRecord> fRecord;
+    SkAutoTUnref<SkRecorder> fRecorder;
 };
 
 }  // namespace EXPERIMENTAL
 
 #endif//SkRecording_DEFINED