[blimp] Add SkPicture caching support.

blimp/engine/renderer/blimp_engine_picture_cache.h

+// 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.
+
+#ifndef BLIMP_ENGINE_RENDERER_BLIMP_ENGINE_PICTURE_CACHE_H_
+#define BLIMP_ENGINE_RENDERER_BLIMP_ENGINE_PICTURE_CACHE_H_
+
+#include <memory>
+#include <unordered_map>
+#include <vector>
+
+#include "base/macros.h"
+#include "blimp/common/compositor/blimp_picture_cache_registry.h"
+#include "cc/proto/picture_cache.h"
+#include "third_party/skia/include/core/SkPicture.h"
+
+class SkPixelSerializer;
+
+namespace blimp {
+namespace engine {
+
+// BlimpEnginePictureCache provides functionality for caching SkPictures before
+// they are sent from the engine to the client. The cache is cleared after
+// every time it is flushed, but the expected state of what the client already
+// has cached is tracked. It is required to update this cache when an SkPicture
+// starts being used and when it is not longer in use by calling
+// MarkPictureForRegistration and MarkPictureForUnregistration respectively.

[Kevin M, 2016/06/06 23:33:59]

What's the lifetime of an engine picture cache?

[nyquist, 2016/06/10 22:02:24]

Done.

+class BlimpEnginePictureCache : public cc::EnginePictureCache {
+ public:
+  explicit BlimpEnginePictureCache(SkPixelSerializer* pixel_serializer);
+  ~BlimpEnginePictureCache() override;
+
+  // cc::EnginePictureCache implementation.
+  void MarkPictureForUnregistration(const SkPicture* picture) override;
+  void MarkPictureForRegistration(const SkPicture* picture) override;
+  cc::PictureCacheUpdate CalculateCacheUpdateAndFlush() override;
+
+ private:
+  // A serializer that be used to pass in to SkPicture::serialize(...) for
+  // serializing the SkPicture to a stream.
+  SkPixelSerializer* pixel_serializer_;
+
+  // The current cache of pictures. Cleared after every flush.

[Kevin M, 2016/06/06 23:33:59]

What is a "flush" here?

[nyquist, 2016/06/10 22:02:24]

Clarified comment.

+  std::unordered_map<uint32_t, cc::PictureData> pictures_;
+
+  // A ref-counted registry of SkPictures that are currently in use.
+  BlimpPictureCacheRegistry registry_;
+
+  DISALLOW_COPY_AND_ASSIGN(BlimpEnginePictureCache);
+};
+
+}  // namespace engine
+}  // namespace blimp
+
+#endif  // BLIMP_ENGINE_RENDERER_BLIMP_ENGINE_PICTURE_CACHE_H_