commit fc4b6ed24402460e52bb4dc2c672c65c2984f998
author Doğancan Emek <dogancanemek@google.com> Wed Jul 26 15:24:04 2023 +0000
committer Doğancan Emek <dogancanemek@google.com> Thu Aug 03 09:47:57 2023 +0000
tree 45c6afa49a0df7c040c454d41d59495c91715abe
parent cd760e3feff7a272e07c03203fa9d8c77a8fd4b4

Adding DPI to Traces

To enable Flickr testing on large screen devices, a DPI value needed to be added on the traces

Test: add_dpi_to_traces

Bug: 284145257

Change-Id: I1063c971dbddabf670004147944ee398cd832c9a

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 4f7df3f..86937f5 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -6127,6 +6127,13 @@
         displayProto->set_id(display->getId().value);
         displayProto->set_name(display->getDisplayName());
         displayProto->set_layer_stack(display->getLayerStack().id);
+
+        if (!display->isVirtual()) {
+            const auto dpi = display->refreshRateSelector().getActiveMode().modePtr->getDpi();
+            displayProto->set_dpi_x(dpi.x);
+            displayProto->set_dpi_y(dpi.y);
+        }
+
         LayerProtoHelper::writeSizeToProto(display->getWidth(), display->getHeight(),
                                            [&]() { return displayProto->mutable_size(); });
         LayerProtoHelper::writeToProto(display->getLayerStackSpaceRect(), [&]() {

services/surfaceflinger/layerproto/display.proto

diff --git a/services/surfaceflinger/layerproto/display.proto b/services/surfaceflinger/layerproto/display.proto
index c8cd926..64de775 100644
--- a/services/surfaceflinger/layerproto/display.proto
+++ b/services/surfaceflinger/layerproto/display.proto
@@ -35,4 +35,8 @@
     TransformProto transform = 6;
 
     bool is_virtual = 7;
+
+    double dpi_x = 8;
+
+    double dpi_y = 9;
 }