diff --git a/libs/hwui/PathCache.cpp b/libs/hwui/PathCache.cpp index 27030728df7f1..bdb44a6941181 100644 --- a/libs/hwui/PathCache.cpp +++ b/libs/hwui/PathCache.cpp @@ -446,9 +446,7 @@ void PathCache::precache(const SkPath* path, const SkPaint* paint) { if (mProcessor == nullptr) { mProcessor = new PathProcessor(Caches::getInstance()); } - if (!mProcessor->add(task)) { - mProcessor->process(task); - } + mProcessor->add(task); } } diff --git a/libs/hwui/TessellationCache.cpp b/libs/hwui/TessellationCache.cpp index d9d06bf8560ee..7edb9fb4a36a8 100644 --- a/libs/hwui/TessellationCache.cpp +++ b/libs/hwui/TessellationCache.cpp @@ -386,10 +386,7 @@ void TessellationCache::precacheShadows(const Matrix4* drawTransform, const Rect if (mShadowProcessor == nullptr) { mShadowProcessor = new ShadowProcessor(Caches::getInstance()); } - if (!mShadowProcessor->add(task)) { - mShadowProcessor->process(task); - } - + mShadowProcessor->add(task); task->incStrong(nullptr); // not using sp<>s, so manually ref while in the cache mShadowCache.put(key, task.get()); } @@ -424,9 +421,7 @@ TessellationCache::Buffer* TessellationCache::getOrCreateBuffer( if (mProcessor == nullptr) { mProcessor = new TessellationProcessor(Caches::getInstance()); } - if (!mProcessor->add(task)) { - mProcessor->process(task); - } + mProcessor->add(task); mCache.put(entry, buffer); } return buffer; diff --git a/libs/hwui/thread/TaskProcessor.h b/libs/hwui/thread/TaskProcessor.h index ec6519cf85ad9..82538e9c7342d 100644 --- a/libs/hwui/thread/TaskProcessor.h +++ b/libs/hwui/thread/TaskProcessor.h @@ -39,7 +39,17 @@ public: TaskProcessor(TaskManager* manager): mManager(manager) { } virtual ~TaskProcessor() { } - bool add(const sp >& task); + void add(const sp >& task) { + if (!addImpl(task)) { + // fall back to immediate execution + process(task); + } + } + + virtual void onProcess(const sp >& task) = 0; + +private: + bool addImpl(const sp >& task); virtual void process(const sp& task) override { sp > realTask = static_cast* >(task.get()); @@ -48,13 +58,11 @@ public: onProcess(realTask); } - virtual void onProcess(const sp >& task) = 0; - TaskManager* mManager; }; template -bool TaskProcessor::add(const sp >& task) { +bool TaskProcessor::addImpl(const sp >& task) { if (mManager) { sp > self(this); return mManager->addTask(task, self);