summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDan Stoza <stoza@google.com>2018-07-31 16:23:38 -0700
committerDan Stoza <stoza@google.com>2018-08-01 18:10:43 +0000
commit56eb975447788a58b3a8f858655c964d35e0ba75 (patch)
treee9ea97aa86891d1c3bc4d8bab917108afcb7b9b8
parentbde9db506f38bb8caa6df016ca25fef5410b88b4 (diff)
downloadnative-56eb975447788a58b3a8f858655c964d35e0ba75.tar.gz
SF: Layer children should call parents' onFirstRef
Classes which inherit indirectly from Layer and which override onFirstRef need to also call their parents' onFirstRef to ensure that Layer is initialized correctly. Bug: 111854867 Test: atest UiAutomationTest#testWindowContentFrameStats Change-Id: I5e29b6619d1a2e48277ad465a64d1f13bb92905c Merged-In: I5ceb531a5d59587ab489342d0b19a42c1a286402
-rw-r--r--services/surfaceflinger/BufferLayer.cpp2
1 files changed, 2 insertions, 0 deletions
diff --git a/services/surfaceflinger/BufferLayer.cpp b/services/surfaceflinger/BufferLayer.cpp
index 12267adbc0..7ac1432198 100644
--- a/services/surfaceflinger/BufferLayer.cpp
+++ b/services/surfaceflinger/BufferLayer.cpp
@@ -701,6 +701,8 @@ bool BufferLayer::isOpaque(const Layer::State& s) const {
}
void BufferLayer::onFirstRef() {
+ Layer::onFirstRef();
+
// Creates a custom BufferQueue for SurfaceFlingerConsumer to use
sp<IGraphicBufferProducer> producer;
sp<IGraphicBufferConsumer> consumer;