Update documentation and cleanup impls for localToSVGParentTransform()

third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceMarker.cpp

Index: third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceMarker.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceMarker.cpp b/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceMarker.cpp
index a1b5d8d38b837ef41695618d074a7a604b1c9361..39308cdea900542bc53cf37959e7bb44ba4485f6 100644
--- a/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceMarker.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceMarker.cpp
@@ -72,12 +72,9 @@ FloatRect LayoutSVGResourceMarker::markerBoundaries(
   return markerTransformation.mapRect(coordinates);
 }
 
-const AffineTransform& LayoutSVGResourceMarker::localToSVGParentTransform()
-    const {
-  m_localToParentTransform =
-      AffineTransform::translation(m_viewport.x(), m_viewport.y()) *
-      viewportTransform();
-  return m_localToParentTransform;
+AffineTransform LayoutSVGResourceMarker::localToSVGParentTransform() const {
+  return AffineTransform::translation(m_viewport.x(), m_viewport.y()) *
+         viewportTransform();
   // If this class were ever given a localTransform(), then the above would

[pdr., 2016/11/18 20:35:10]

Can you remove this comment?

[Xianzhu, 2016/11/18 22:19:12]

Done.

   // read:
   // return viewportTranslation * localTransform() * viewportTransform();