Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(777)

Side by Side Diff: blimp/client/feature/compositor/blimp_client_picture_cache.cc

Issue 1982893002: [blimp] Add SkPicture caching support. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed comments from vmpstr, including adding //cc/blimp Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "blimp/client/feature/compositor/blimp_client_picture_cache.h"
6
7 #include <utility>
8 #include <vector>
9
10 #include "third_party/skia/include/core/SkStream.h"
11
12 namespace blimp {
13 namespace client {
14 namespace {
15
16 // Helper function to deserialize the content of |picture_data| into an
17 // SkPicture.
18 sk_sp<const SkPicture> DeserializePicture(
19 SkPicture::InstallPixelRefProc pixel_deserializer,
20 const cc::PictureData& picture_data) {
21 SkMemoryStream stream(picture_data.data);
22 return SkPicture::MakeFromStream(&stream, pixel_deserializer);
23 }
24
25 } // namespace
26
27 BlimpClientPictureCache::BlimpClientPictureCache(
28 SkPicture::InstallPixelRefProc pixel_deserializer)
29 : pixel_deserializer_(pixel_deserializer) {}
30
31 BlimpClientPictureCache::~BlimpClientPictureCache() = default;
32
33 sk_sp<const SkPicture> BlimpClientPictureCache::GetPicture(
34 uint32_t engine_picture_id) {
35 return GetPictureFromCache(engine_picture_id);
36 }
37
38 void BlimpClientPictureCache::ApplyCacheUpdate(
39 const std::vector<cc::PictureData>& cache_update) {
40 // Add new pictures from the |cache_update| to |pictures_|.
41 for (const cc::PictureData& picture_data : cache_update) {
42 DCHECK(pictures_.find(picture_data.unique_id) == pictures_.end());
43 sk_sp<const SkPicture> deserialized_picture =
44 DeserializePicture(pixel_deserializer_, picture_data);
45
46 pictures_[picture_data.unique_id] = std::move(deserialized_picture);
47
48 #if DCHECK_IS_ON()
49 last_added_.insert(picture_data.unique_id);
50 #endif // DCHECK_IS_ON()
51 }
52 }
53
54 void BlimpClientPictureCache::Flush() {
55 // Calculate which pictures can now be removed. |added| is only used for
56 // verifying that what we calculated matches the new items that have been
57 // inserted into the cache.
58 std::vector<uint32_t> added;
59 std::vector<uint32_t> removed;
60 reference_tracker_.CommitRefCounts(&added, &removed);
61
62 VerifyCacheUpdateMatchesReferenceTrackerData(added);
63
64 RemoveUnusedPicturesFromCache(removed);
65 reference_tracker_.ClearRefCounts();
66 }
67
68 void BlimpClientPictureCache::MarkUsed(uint32_t engine_picture_id) {
69 reference_tracker_.IncrementRefCount(engine_picture_id);
70 }
71
72 void BlimpClientPictureCache::RemoveUnusedPicturesFromCache(
73 const std::vector<uint32_t>& removed) {
74 for (const auto& engine_picture_id : removed) {
75 auto entry = pictures_.find(engine_picture_id);
76 DCHECK(entry != pictures_.end());
77 pictures_.erase(entry);
78 }
79 }
80
81 sk_sp<const SkPicture> BlimpClientPictureCache::GetPictureFromCache(
82 uint32_t engine_picture_id) {
83 DCHECK(pictures_.find(engine_picture_id) != pictures_.end());
84 return pictures_[engine_picture_id];
85 }
86
87 void BlimpClientPictureCache::VerifyCacheUpdateMatchesReferenceTrackerData(
88 const std::vector<uint32_t>& new_tracked_items) {
89 #if DCHECK_IS_ON()
90 DCHECK_EQ(new_tracked_items.size(), last_added_.size());
91 DCHECK(std::unordered_set<uint32_t>(new_tracked_items.begin(),
92 new_tracked_items.end()) == last_added_);
93 last_added_.clear();
94 #endif // DCHECK_IS_ON()
95 }
96
97 } // namespace client
98 } // namespace blimp
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698