[SVGDom] Add <svg> viewBox support

experimental/svg/model/SkSVGNode.h

Index: experimental/svg/model/SkSVGNode.h
diff --git a/experimental/svg/model/SkSVGNode.h b/experimental/svg/model/SkSVGNode.h
index 168b03c73bc2e152bdcbe8e9d6eacb46d258998f..7cc8d8a7c621df2919e66f9c0bfa1b726b72b7d3 100644
--- a/experimental/svg/model/SkSVGNode.h
+++ b/experimental/svg/model/SkSVGNode.h
@@ -31,18 +31,22 @@ public:
 
     virtual void appendChild(sk_sp<SkSVGNode>) = 0;
 
-    void render(SkCanvas*, const SkSVGRenderContext&) const;
+    void render(const SkSVGRenderContext&) const;
 
     void setAttribute(SkSVGAttribute, const SkSVGValue&);
 
 protected:
     SkSVGNode(SkSVGTag);
 
-    virtual void onRender(SkCanvas*, const SkSVGRenderContext&) const = 0;
+    // Called before onRender(), to apply local attributes to the context.  Unlike onRender(),
+    // onPrepareToRender() bubbles up the inheritance chain: overriders should always call
+    // INHERITED::onPrepareToRender(), unless they intend to short-circuit rendering by
+    // returning false.

[robertphillips, 2016/08/08 13:55:28]

// Returning false does what now?

[f(malita), 2016/08/08 17:13:29]

Added more comments.

+    virtual bool onPrepareToRender(SkSVGRenderContext*) const;
 
-    virtual void onSetAttribute(SkSVGAttribute, const SkSVGValue&);
+    virtual void onRender(const SkSVGRenderContext&) const = 0;
 
-    virtual const SkMatrix& onLocalMatrix() const;
+    virtual void onSetAttribute(SkSVGAttribute, const SkSVGValue&);
 
 private:
     SkSVGTag                    fTag;