Chromium Code Reviews| Index: cc/tiles/software_image_decode_cache.h |
| diff --git a/cc/tiles/software_image_decode_cache.h b/cc/tiles/software_image_decode_cache.h |
| index 1d3e2ac75c40d603c8a3009edfce9d0885aac908..2da980681079258e14889d56eb3f74c4b96fdb42 100644 |
| --- a/cc/tiles/software_image_decode_cache.h |
| +++ b/cc/tiles/software_image_decode_cache.h |
| @@ -140,6 +140,7 @@ class CC_EXPORT SoftwareImageDecodeCache |
| void SetShouldAggressivelyFreeResources( |
| bool aggressively_free_resources) override {} |
| void ClearCache() override; |
| + size_t GetMaximumMemoryLimit() const override; |
| // Decode the given image and store it in the cache. This is only called by an |
| // image decode task from a worker thread. |
| @@ -220,7 +221,7 @@ class CC_EXPORT SoftwareImageDecodeCache |
| size_t GetCurrentUsageSafe() const; |
| private: |
| - size_t limit_bytes_; |
| + const size_t limit_bytes_; |
| base::CheckedNumeric<size_t> current_usage_bytes_; |
| }; |
| @@ -306,6 +307,8 @@ class CC_EXPORT SoftwareImageDecodeCache |
| // The members below this comment can only be accessed if the lock is held to |
| // ensure that they are safe to access on multiple threads. |
| + // The exception is accessing |locked_images_budget_.total_limit_bytes()|, |
| + // which is const and thread safe. |
|
Khushal
2017/04/28 00:12:53
nit: Can you make |limit_bytes_| in MemoryBudget c
Khushal
2017/04/28 00:18:46
Note to self: Look for colors other than green.
vmpstr
2017/04/28 17:18:30
Acknowledged.
|
| base::Lock lock_; |
| // Decoded images and ref counts (predecode path). |