summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Leon Scroggins <scroggo@google.com> 2023-04-10 13:54:36 +0000
committer Android (Google) Code Review <android-gerrit@google.com> 2023-04-10 13:54:36 +0000
commit61608cc762e4a87d16594ae400dac6f6fa6a2605 (patch)
tree78244fd0a76752c5ee8f78955bdc0293e02f757c
parent69b990e3d11e7a87548b2a17ed6c0e78ad0f8784 (diff)
parentf5272953652b8cee06b1376a7324dd9741160644 (diff)
Merge "Temporary workaround for getDisplayStats" into udc-dev
-rw-r--r--services/surfaceflinger/SurfaceFlinger.cpp5
1 files changed, 4 insertions, 1 deletions
diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index c4b83828f3..8394ffbca8 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -1149,7 +1149,10 @@ status_t SurfaceFlinger::getDisplayStats(const sp<IBinder>& displayToken,
displayIdOpt = getPhysicalDisplayIdLocked(displayToken);
}
- if (!displayIdOpt) {
+ // TODO (b/277364366): Clients should be updated to pass in the display they
+ // want, rather than us picking an arbitrary one (the pacesetter, in this
+ // case).
+ if (displayToken && !displayIdOpt) {
ALOGE("%s: Invalid physical display token %p", __func__, displayToken.get());
return NAME_NOT_FOUND;
}