[blimp] Add SkPicture caching support.

blimp/client/feature/compositor/client_image_serialization_processor.cc

 // 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 "blimp/client/feature/compositor/client_image_serialization_processor.h"
 
+#include <memory>
+
+#include "base/memory/ptr_util.h"
+#include "blimp/client/feature/compositor/blimp_client_picture_cache.h"
 #include "blimp/client/feature/compositor/blimp_image_decoder.h"
-#include "third_party/skia/include/core/SkPicture.h"
 
 class SkPixelSerializer;
 
 namespace blimp {
 namespace client {
 
-ClientImageSerializationProcessor::ClientImageSerializationProcessor() {
-  pixel_deserializer_ = &BlimpImageDecoder;
-}
+ClientImageSerializationProcessor::ClientImageSerializationProcessor() {}
 
 ClientImageSerializationProcessor::~ClientImageSerializationProcessor() {}
 
-SkPixelSerializer* ClientImageSerializationProcessor::GetPixelSerializer() {
+std::unique_ptr<cc::EnginePictureCache>
+ClientImageSerializationProcessor::CreateEnginePictureCache() {
   return nullptr;
 }
 
-SkPicture::InstallPixelRefProc
-ClientImageSerializationProcessor::GetPixelDeserializer() {
-  return pixel_deserializer_;
+std::unique_ptr<cc::ClientPictureCache>
+ClientImageSerializationProcessor::CreateClientPictureCache() {
+  return base::WrapUnique(new BlimpClientPictureCache(&BlimpImageDecoder));

[vmpstr, 2016/06/01 00:10:56]

BlimpImageDecoder is a function? If so, it should be named DecodeBlimpImage or
something? It's a bit confusing if a function is named with a noun. 

[nyquist, 2016/06/04 00:24:57]

Done.

 }
 
 }  // namespace client
 }  // namespace blimp