Prepare SVGImage for the default sizing algorithm

third_party/WebKit/Source/core/svg/graphics/SVGImage.h

Index: third_party/WebKit/Source/core/svg/graphics/SVGImage.h
diff --git a/third_party/WebKit/Source/core/svg/graphics/SVGImage.h b/third_party/WebKit/Source/core/svg/graphics/SVGImage.h
index c2009f217fc06e7d7a7b924d8382471624aad106..a2c4820940c880b2a06b0111059c8f061f87bbd9 100644
--- a/third_party/WebKit/Source/core/svg/graphics/SVGImage.h
+++ b/third_party/WebKit/Source/core/svg/graphics/SVGImage.h
@@ -55,7 +55,7 @@ public:
 
     bool isSVGImage() const override { return true; }
     bool isTextureBacked() override { return false; }
-    IntSize size() const override { return m_intrinsicSize; }
+    IntSize size() const override { return m_concreteObjectSize; }
 
     bool currentFrameHasSingleSecurityOrigin() const override;
 
@@ -90,6 +90,7 @@ private:
 
     String filenameExtension() const override;
 
+    FloatSize calculateConcreteObjectSize(const FloatSize&) const;

[pdr., 2016/02/23 04:42:28]

Is this FloatSize parameter needed? Can you just return an IntSize here?

[davve, 2016/02/23 08:34:37]

I agree is looks a bit silly now, but it's used in the follow-up CL:

https://codereview.chromium.org/1694263003/

I return a FloatSize in order to decide on how to do rounding in as few places
as possible.

     IntSize containerSize() const;
     bool usesContainerSize() const override { return true; }
     void computeIntrinsicDimensions(Length& intrinsicWidth, Length& intrinsicHeight, FloatSize& intrinsicRatio) override;
@@ -114,7 +115,7 @@ private:
 
     OwnPtrWillBePersistent<SVGImageChromeClient> m_chromeClient;
     OwnPtrWillBePersistent<Page> m_page;
-    IntSize m_intrinsicSize;
+    IntSize m_concreteObjectSize;

[pdr., 2016/02/23 04:42:28]

Can you add a comment here describing what the concrete object size is?

It may be useful to add a markdown file describing sizing in SVG in general (as
a followup).

[davve, 2016/02/23 08:34:37]

I copy a snippet from the spec and provide a link.
Noted.

 };
 
 DEFINE_IMAGE_TYPE_CASTS(SVGImage);