[blimp] Add SkPicture caching support.

cc/playback/recording_source.h

 // 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.
 
 #ifndef CC_PLAYBACK_RECORDING_SOURCE_H_
 #define CC_PLAYBACK_RECORDING_SOURCE_H_
 
 #include <stddef.h>
 
 #include <memory>
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "cc/base/cc_export.h"
 #include "cc/base/invalidation_region.h"
 #include "third_party/skia/include/core/SkColor.h"
 #include "ui/gfx/geometry/rect.h"
 #include "ui/gfx/geometry/size.h"
 
 namespace cc {
 
 namespace proto {
 class RecordingSource;
 }  // namespace proto
 
+class ClientPictureCache;
 class ContentLayerClient;
 class DisplayItemList;
+class EnginePictureCache;
 class RasterSource;
-class ImageSerializationProcessor;
 class Region;
 
 class CC_EXPORT RecordingSource {
  public:
   // TODO(schenney) Remove RECORD_WITH_SK_NULL_CANVAS when we no longer
   // support a non-Slimming Paint path.
   enum RecordingMode {
     RECORD_NORMALLY,
     RECORD_WITH_SK_NULL_CANVAS,
     RECORD_WITH_PAINTING_DISABLED,
     RECORD_WITH_CACHING_DISABLED,
     RECORD_WITH_CONSTRUCTION_DISABLED,
     RECORD_WITH_SUBSEQUENCE_CACHING_DISABLED,
     RECORDING_MODE_COUNT,  // Must be the last entry.
   };
 
   RecordingSource();
   virtual ~RecordingSource();
 
-  void ToProtobuf(
-      proto::RecordingSource* proto,
-      ImageSerializationProcessor* image_serialization_processor) const;
-  void FromProtobuf(const proto::RecordingSource& proto,
-                    ImageSerializationProcessor* image_serialization_processor);
+  void ToProtobuf(proto::RecordingSource* proto) const;
+  void FromProtobuf(const proto::RecordingSource& proto);
 
   bool UpdateAndExpandInvalidation(ContentLayerClient* painter,
                                    Region* invalidation,
                                    const gfx::Size& layer_size,
                                    int frame_number,
                                    RecordingMode recording_mode);
   gfx::Size GetSize() const;
   void SetEmptyBounds();
   void SetSlowdownRasterScaleFactor(int factor);
   void SetGenerateDiscardableImagesMetadata(bool generate_metadata);
   void SetBackgroundColor(SkColor background_color);
   void SetRequiresClear(bool requires_clear);
 
   void SetNeedsDisplayRect(const gfx::Rect& layer_rect);
 
+  void SetEnginePictureCache(EnginePictureCache* engine_picture_cache);
+
+  void SetClientPictureCache(ClientPictureCache* client_picture_cache);
+
   // These functions are virtual for testing.
   virtual scoped_refptr<RasterSource> CreateRasterSource(
       bool can_use_lcd_text) const;
   virtual bool IsSuitableForGpuRasterization() const;
 
   gfx::Rect recorded_viewport() const { return recorded_viewport_; }
 
  protected:
   void Clear();
 
@@ skipping 13 matching lines @@
  private:
   void UpdateInvalidationForNewViewport(const gfx::Rect& old_recorded_viewport,
                                         const gfx::Rect& new_recorded_viewport,
                                         Region* invalidation);
   void FinishDisplayItemListUpdate();
 
   friend class RasterSource;
 
   void DetermineIfSolidColor();
 
+  // Updates the EnginePictureCache about the SkPictures currently in use.
+  void MarkForRegistrationEngine();

[vmpstr, 2016/06/16 22:09:59]

Similarly here... is it possible to expose the iteration of the display items
instead?

I know I said that I'd prefer you move these onto the raster source, but
again... I'm really not a fan of having engine/client pointers on an object that
is otherwise independent of blimp and just represents a way to draw something
into a canvas

[nyquist, 2016/06/24 11:11:14]

Done.

+  void MarkForUnregistrationEngine();
+
+  // Updates the ClientPictureCache about the SkPictures currently in use.
+  void MarkForRegistrationClient();
+  void MarkForUnregistrationClient();
+
+  // EnginePictureCache is used when the RecordingSource is instantiated on an
+  // engine, and the ClientPictureCache is only used on the client. They are
+  // never used together, but since the APIs are not the same, there needs
+  // to be two different members. Both caches are used to track which
+  // SkPictures are in use within the RecordingSource.
+  EnginePictureCache* engine_picture_cache_;
+  ClientPictureCache* client_picture_cache_;
+
   InvalidationRegion invalidation_;
 
   DISALLOW_COPY_AND_ASSIGN(RecordingSource);
 };
 
 }  // namespace cc
 
 #endif  // CC_PLAYBACK_RECORDING_SOURCE_H_