paint: Introduce PaintImage that wraps SkImage in paint calls.

third_party/WebKit/Source/platform/graphics/StaticBitmapImage.cpp

 // Copyright 2014 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 "platform/graphics/StaticBitmapImage.h"
 
 #include "platform/graphics/AcceleratedStaticBitmapImage.h"
 #include "platform/graphics/GraphicsContext.h"
 #include "platform/graphics/ImageObserver.h"
 #include "platform/graphics/UnacceleratedStaticBitmapImage.h"
+#include "platform/graphics/paint/PaintImage.h"
 #include "third_party/skia/include/core/SkCanvas.h"
 #include "third_party/skia/include/core/SkImage.h"
 #include "third_party/skia/include/core/SkPaint.h"
 
 namespace blink {
 
 PassRefPtr<StaticBitmapImage> StaticBitmapImage::Create(sk_sp<SkImage> image) {
   if (!image)
     return nullptr;
   if (image->isTextureBacked())
     return AcceleratedStaticBitmapImage::CreateFromSharedContextImage(
         std::move(image));
   return UnacceleratedStaticBitmapImage::Create(std::move(image));
 }
 
 void StaticBitmapImage::DrawHelper(PaintCanvas* canvas,
                                    const PaintFlags& flags,
                                    const FloatRect& dst_rect,
                                    const FloatRect& src_rect,
                                    ImageClampingMode clamp_mode,
                                    sk_sp<SkImage> image) {
   FloatRect adjusted_src_rect = src_rect;
   adjusted_src_rect.Intersect(SkRect::Make(image->bounds()));
 
   if (dst_rect.IsEmpty() || adjusted_src_rect.IsEmpty())
     return;  // Nothing to draw.
 
-  canvas->drawImageRect(std::move(image), adjusted_src_rect, dst_rect, &flags,
-                        WebCoreClampingModeToSkiaRectConstraint(clamp_mode));
+  auto animation_type = MaybeAnimated() ? PaintImage::AnimationType::ANIMATED

[pdr., 2017/04/12 17:37:05]

WDYT of just changing the return type of MaybeAnimated and
CurrentFrameIsComplete to be your new enums?

[vmpstr, 2017/04/12 18:58:44]

Hmm, there are callers outside of graphics that use this (like LayoutBox) and I
think the PaintImage guts should remain limited to graphics.. So I would prefer
to leave that as is. WDYT?

+                                        : PaintImage::AnimationType::STATIC;
+  auto completion_state = CurrentFrameIsComplete()
+                              ? PaintImage::CompletionState::DONE
+                              : PaintImage::CompletionState::PARTIALLY_DONE;
+  canvas->drawImageRect(
+      PaintImage(std::move(image), animation_type, completion_state),
+      adjusted_src_rect, dst_rect, &flags,
+      WebCoreClampingModeToSkiaRectConstraint(clamp_mode));
 }
 
 }  // namespace blink