commit 0acc384bb994f7f8bbe9f4060d0b1b212744879f
author Dominik Laskowski <domlaskowski@google.com> Thu Apr 07 11:23:42 2022 -0700
committer Dominik Laskowski <domlaskowski@google.com> Wed Apr 20 08:43:52 2022 -0700
tree 3d9a1ea9f0cae4810ae55c4f39e157c772eb1417
parent 3c363247085f9ca75450a383b476312d96a490cc

SF: Add dumpsys flags to dump displays

Dump display configuration (--displays) and composition state (--comp-
displays) separately.

Use consistent formatting. Remove noise, e.g. repetitive display modes,
physical display state that is expectedly missing for virtual displays.

Bug: 182939859
Test: dumpsys SurfaceFlinger --displays
Test: dumpsys SurfaceFlinger --comp-displays
Change-Id: I787eedb82d76c74363e653a648ae88699096cc47

libs/ui/DeviceProductInfo.cpp

diff --git a/libs/ui/DeviceProductInfo.cpp b/libs/ui/DeviceProductInfo.cpp
index 4d6ce43..04d9d3c 100644
--- a/libs/ui/DeviceProductInfo.cpp
+++ b/libs/ui/DeviceProductInfo.cpp
@@ -57,7 +57,7 @@
 }
 
 void DeviceProductInfo::dump(std::string& result) const {
-    StringAppendF(&result, "{name=%s, ", name.c_str());
+    StringAppendF(&result, "{name=\"%s\", ", name.c_str());
     StringAppendF(&result, "manufacturerPnpId=%s, ", manufacturerPnpId.data());
     StringAppendF(&result, "productId=%s, ", productId.c_str());