Implement CSSPaintValue and add a layout test.

third_party/WebKit/Source/modules/csspaint/CSSPaintImageGeneratorImpl.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 "modules/csspaint/CSSPaintImageGeneratorImpl.h"
 
 #include "core/dom/Document.h"
 #include "core/frame/LocalDOMWindow.h"
 #include "modules/csspaint/CSSPaintDefinition.h"
 #include "modules/csspaint/PaintWorklet.h"
@@ skipping 34 matching lines @@
   ASSERT(!m_definition);
   m_definition = definition;
 
   ASSERT(m_observer);
   m_observer->paintImageGeneratorReady();
 }
 
 PassRefPtr<Image> CSSPaintImageGeneratorImpl::paint(
     const LayoutObject& layoutObject,
     const IntSize& size,
-    float zoom) {
-  return m_definition ? m_definition->paint(layoutObject, size, zoom) : nullptr;
+    float zoom,
+    const CSSStyleValueVector* data) {
+  return m_definition ? m_definition->paint(layoutObject, size, zoom, data)
+                      : nullptr;
 }
 
 const Vector<CSSPropertyID>&
 CSSPaintImageGeneratorImpl::nativeInvalidationProperties() const {
   DEFINE_STATIC_LOCAL(Vector<CSSPropertyID>, emptyVector, ());
   return m_definition ? m_definition->nativeInvalidationProperties()
                       : emptyVector;
 }
 
 const Vector<AtomicString>&
 CSSPaintImageGeneratorImpl::customInvalidationProperties() const {
   DEFINE_STATIC_LOCAL(Vector<AtomicString>, emptyVector, ());
   return m_definition ? m_definition->customInvalidationProperties()
                       : emptyVector;
 }
 
 bool CSSPaintImageGeneratorImpl::hasAlpha() const {
   return m_definition && m_definition->hasAlpha();
 }
 
+const Vector<CSSSyntaxDescriptor>&
+CSSPaintImageGeneratorImpl::inputArgumentTypes() const {
+  DEFINE_STATIC_LOCAL(Vector<CSSSyntaxDescriptor>, emptyVector, ());

[haraken, 2017/02/21 23:51:55]

Nit: Is it really worth defining an empty vector? I guess we could just return
Vector<CSSSyntaxDescriptor>().

[renjieliu1, 2017/02/22 04:47:54]

the compiler will throw me an error says returning reference to local temporary
object

+  return m_definition ? m_definition->inputArgumentTypes() : emptyVector;
+}
+
 DEFINE_TRACE(CSSPaintImageGeneratorImpl) {
   visitor->trace(m_definition);
   visitor->trace(m_observer);
   CSSPaintImageGenerator::trace(visitor);
 }
 
 }  // namespace blink