From 9a7231033b19c936e4e2f76667289a7295eb008b Mon Sep 17 00:00:00 2001 From: Vishnu Nair Date: Wed, 28 Nov 2018 15:04:08 -0800 Subject: [PATCH] WinScope: Support boundless surfaces in WinScope SurfaceFlinger trace relies on Display Root#0 layer size to set the display bounds. This layer will no longer have a size once b/114413815 is fixed. Instead, sf transform will set the bounds to undefined and the bounds be calculated to be the union of the layer sizes. see: http://x20web/teams/android-window-manager/winscope/winscope-staging-vishnun.html Bug: 114413815 Test: build winscope and open trace Change-Id: Ic04d524f0e095374321ef818f0d0ff420a2d3827 --- tools/winscope/src/transform_sf.js | 5 ----- 1 file changed, 5 deletions(-) diff --git a/tools/winscope/src/transform_sf.js b/tools/winscope/src/transform_sf.js index 3bd070cbc..8949b41f0 100644 --- a/tools/winscope/src/transform_sf.js +++ b/tools/winscope/src/transform_sf.js @@ -52,10 +52,6 @@ function transform_layer(layer, {parentHidden}) { } else { rect = undefined; } - var bounds = undefined; - if (layer.name.startsWith("Display Root#0")) { - bounds = {width: layer.size.w, height: layer.size.h}; - } if (layer.zOrderRelativeOf !== -1) { chips.push(RELATIVE_Z_CHIP); } @@ -77,7 +73,6 @@ function transform_layer(layer, {parentHidden}) { [layer.resolvedChildren, transform_layer_with_parent_hidden], ], rect, - bounds, highlight: rect, chips, visible,