First pass at Comment API

src/core/SkCanvas.cpp

 
 /*
  * Copyright 2008 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 "SkCanvas.h"
@@ skipping 1953 matching lines @@
                                    looper.paint());
     }
 
     LOOPER_END
 }
 
 void SkCanvas::drawData(const void* data, size_t length) {
     // do nothing. Subclasses may do something with the data
 }
 
+void SkCanvas::beginCommentGroup(const char* description) {

[djsollen, 2013/05/28 14:19:11]

why not just put the empty impl in the header?

[robertphillips, 2013/05/28 15:45:31]

Done - I was mimicking what was done for drawData.

+    // do nothing. Subclasses may do something
+}
+
+void SkCanvas::addComment(const char* kywd, const char* value) {
+    // do nothing. Subclasses may do something
+}
+
+void SkCanvas::endCommentGroup() {
+    // do nothing. Subclasses may do something
+}
+
 //////////////////////////////////////////////////////////////////////////////
 // These methods are NOT virtual, and therefore must call back into virtual
 // methods, rather than actually drawing themselves.
 //////////////////////////////////////////////////////////////////////////////
 
 void SkCanvas::drawARGB(U8CPU a, U8CPU r, U8CPU g, U8CPU b,
                         SkXfermode::Mode mode) {
     SkPaint paint;
 
     paint.setARGB(a, r, g, b);
@@ skipping 144 matching lines @@
     return *paint;
 }
 
 const SkRegion& SkCanvas::LayerIter::clip() const { return fImpl->getClip(); }
 int SkCanvas::LayerIter::x() const { return fImpl->getX(); }
 int SkCanvas::LayerIter::y() const { return fImpl->getY(); }
 
 ///////////////////////////////////////////////////////////////////////////////
 
 SkCanvas::ClipVisitor::~ClipVisitor() { }