Skip to content

Commit 6d4d0db

Browse files
committed
Disable skipping frames on empty damage
Bug: 22592975 Optimization added in MNC is not quite correct, occasionally it will fail to repaint when it was supposed to leading to buffer corruption. Disable the optimization for now. Change-Id: I34dfdfb357eda298198043ded7335d4588a003fd
1 parent 0faca96 commit 6d4d0db

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

libs/hwui/renderthread/CanvasContext.cpp

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -229,10 +229,11 @@ void CanvasContext::draw() {
229229
SkRect dirty;
230230
mDamageAccumulator.finish(&dirty);
231231

232-
if (dirty.isEmpty() && Properties::skipEmptyFrames) {
233-
mCurrentFrameInfo->addFlag(FrameInfoFlags::SkippedFrame);
234-
return;
235-
}
232+
// TODO: Re-enable after figuring out cause of b/22592975
233+
// if (dirty.isEmpty() && Properties::skipEmptyFrames) {
234+
// mCurrentFrameInfo->addFlag(FrameInfoFlags::SkippedFrame);
235+
// return;
236+
// }
236237

237238
mCurrentFrameInfo->markIssueDrawCommandsStart();
238239

0 commit comments

Comments
 (0)