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

Unified Diff: src/core/SkImageFilter.cpp

Issue 1763223002: Switch SkMergeImageFilter over to new onFilterImage interface (Closed) Base URL: https://skia.googlesource.com/skia.git@if-follow-on
Patch Set: Remove extraneous files Created 4 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: src/core/SkImageFilter.cpp
diff --git a/src/core/SkImageFilter.cpp b/src/core/SkImageFilter.cpp
index a233871bff69465171f64764702dc4d36a92c7dc..374a561030254561941ecf21e67d4a65cdfffe80 100644
--- a/src/core/SkImageFilter.cpp
+++ b/src/core/SkImageFilter.cpp
@@ -231,6 +231,8 @@ SkSpecialImage* SkImageFilter::filterImage(SkSpecialImage* src, const Context& c
SkSpecialImage* result = this->onFilterImage(src, context, offset);
if (result && context.cache()) {
context.cache()->set(key, result, *offset);
+ SkAutoMutexAcquire mutex(fMutex);
Stephen White 2016/03/07 18:58:25 Does this belong in this change?
robertphillips 2016/03/08 15:44:22 Split out into https://codereview.chromium.org/177
+ fCacheKeys.push_back(key);
}
return result;

Powered by Google App Engine
This is Rietveld 408576698