Implement SkPicture::bytesUsed() for SkRecord backend

src/core/SkRTree.h

Index: src/core/SkRTree.h
diff --git a/src/core/SkRTree.h b/src/core/SkRTree.h
index 440411d9b0b96f9d035f5840f065f4f3aa038d07..8fe3d552f2cb162252f1b37921e6856a88764864 100644
--- a/src/core/SkRTree.h
+++ b/src/core/SkRTree.h
@@ -54,6 +54,8 @@ public:
     // These values were empirically determined to produce reasonable performance in most cases.
     static const int kMinChildren = 6,
                      kMaxChildren = 11;
+    virtual size_t bytesUsed() const SK_OVERRIDE;

[mtklein, 2014/11/18 19:26:14]

Might want to move this above the "only public for tests" line, but I don't
think it'll be a big source of confusion either way.

[tomhudson, 2014/11/18 19:35:12]

Since we don't really want this used for production decision-making, 'only for
tests' seems like a reasonable way to tag it.

[mtklein, 2014/11/18 19:41:58]

Uh, okay, but that's a more confusing answer than "oh, didn't see that".

This use case is not testing.  We'd not expect people to look down here before
deciding whether or not to use
ApproximateBytesUsed().  If we're going to have a note that says "Don't use this
for anything but diagnostics,"
it belongs in that header.

This method isn't public for testing.  It's public to match the SkBBH interface.

+
 private:
     struct Node;