Make CSSStyleImageValue a member of CanvasImageSource.

third_party/WebKit/Source/core/css/cssom/CSSStyleImageValue.cpp

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/css/cssom/CSSStyleImageValue.h"
 
 namespace blink {
 
-double CSSStyleImageValue::intrinsicWidth(bool& isNull)
+double CSSStyleImageValue::intrinsicWidth(bool& isNull) const
 {
     isNull = isCachePending();
     if (isNull)
         return 0;
     return imageLayoutSize().width().toDouble();
 }
 
-double CSSStyleImageValue::intrinsicHeight(bool& isNull)
+double CSSStyleImageValue::intrinsicHeight(bool& isNull) const
 {
     isNull = isCachePending();
     if (isNull)
         return 0;
     return imageLayoutSize().height().toDouble();
 }
 
 double CSSStyleImageValue::intrinsicRatio(bool& isNull)
 {
     isNull = isCachePending();
     if (isNull) {
         return 0;
     }
     if (intrinsicHeight(isNull) == 0) {
         isNull = true;
         return 0;
     }
     return intrinsicWidth(isNull) / intrinsicHeight(isNull);
 }
 
+FloatSize CSSStyleImageValue::elementSize(const FloatSize& defaultObjectSize) const
+{
+    bool notUsed;
+    return FloatSize(intrinsicWidth(notUsed), intrinsicHeight(notUsed));
+}
+
+int CSSStyleImageValue::sourceHeight()
+{
+    bool notUsed;
+    return intrinsicHeight(notUsed);
+}
+
+int CSSStyleImageValue::sourceWidth()
+{
+    bool notUsed;
+    return intrinsicWidth(notUsed);
+}
+
+PassRefPtr<Image> CSSStyleImageValue::getSourceImageForCanvas(SourceImageStatus* status, AccelerationHint, SnapshotReason,

[ikilpatrick, 2016/09/27 18:06:00]

would it be better to change this method to accept a layoutobject, to that we
can just call:
StyleImage::image
? 

[Gleb Lanbin, 2016/09/27 19:37:18]

probably, but this method is not defined here. It's defined as a pure virtual
method in CanvasImageSource.

+    const FloatSize&) const
+{
+    if (isCachePending())
+        return nullptr;
+    return m_imageValue->cachedImage()->cachedImage()->getImage()->imageForDefaultFrame();

[Justin Novosad, 2016/09/28 15:41:58]

Confident you don't need any nullptr checks anywhere in this chain?

[Gleb Lanbin, 2016/09/28 16:58:22]

Done.

+}
+
 } // namespace blink