diff options
author | Mathias Agopian <mathias@google.com> | 2009-10-07 16:44:10 -0700 |
---|---|---|
committer | Mathias Agopian <mathias@google.com> | 2009-10-07 16:44:10 -0700 |
commit | e700501d0e888ead9ac6456c0a6fd74d634aa5fb (patch) | |
tree | 25bb0963dd0b96b7c17285f6ddddd536f6465f30 /libs/surfaceflinger/Layer.cpp | |
parent | 8e53c5595dd6846ddbfd39549849ea7a289d9897 (diff) | |
download | frameworks_native-e700501d0e888ead9ac6456c0a6fd74d634aa5fb.zip frameworks_native-e700501d0e888ead9ac6456c0a6fd74d634aa5fb.tar.gz frameworks_native-e700501d0e888ead9ac6456c0a6fd74d634aa5fb.tar.bz2 |
fix [2170319] gmail bulk operation checkbox latency on passion
This also fixes [2152536] ANR in browser
When SF is enqueuing buffers faster than SF dequeues them.
The update flag in SF is not counted and under some situations SF will only
dequeue the first buffer. The state at this point is not technically
corrupted, it's valid, but just delayed by one buffer.
In the case of the Browser ANR, because the last enqueued buffer was delayed
the resizing of the current buffer couldn't happen.
The system would always fall back onto its feet if anything -else- in
tried to draw, because the "late" buffer would be picked up then.
Diffstat (limited to 'libs/surfaceflinger/Layer.cpp')
-rw-r--r-- | libs/surfaceflinger/Layer.cpp | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/libs/surfaceflinger/Layer.cpp b/libs/surfaceflinger/Layer.cpp index 0258cee..2a3e667 100644 --- a/libs/surfaceflinger/Layer.cpp +++ b/libs/surfaceflinger/Layer.cpp @@ -460,8 +460,10 @@ void Layer::lockPageFlip(bool& recomputeVisibleRegions) } } - // FIXME: signal an event if we have more buffers waiting - // mFlinger->signalEvent(); + if (lcblk->getQueuedCount()) { + // signal an event if we have more buffers waiting + mFlinger->signalEvent(); + } if (!mPostedDirtyRegion.isEmpty()) { reloadTexture( mPostedDirtyRegion ); |