1007482 - Dumping layer tree once per request instead of output to log constantly

Status: RESOLVED WONTFIX

(Core :: Graphics: Layers, defect)

Description

[Solomon Chiu [:schiu]]

Currently the layer tree dumping is performed in every composing, which bring the performance down and hard to identify the specific information in the specific time. Hence we need a method/tool similar to "adb shell dumpsys", such that we can fetch the specific information once per request.

Comment 1

[u459114]

Please come out a prototype first. All the features on layout debugger can be put into this mechanism - dump on demand.

Comment 2

[Peter Chang[:pchang]]

Jerry, you tried to add the file node to dump some layer stuffs. Could you share to solomon?

solomon, please list the possible options and we can discuss which one is better.

With this tool, we can use "watch -n 0.3 "adb shell dumptree"" to see the layer tree changes dynamically.

Comment 3

[Jerry Shih[:jerry] (UTC+8) (inactive)]

(In reply to peter chang[:pchang][:peter] from comment #2)
> Jerry, you tried to add the file node to dump some layer stuffs. Could you
> share to solomon?
> 
> solomon, please list the possible options and we can discuss which one is
> better.
> 
> With this tool, we can use "watch -n 0.3 "adb shell dumptree"" to see the
> layer tree changes dynamically.

Please check:
http://dxr.mozilla.org/mozilla-central/source/xpcom/base/nsDumpUtils.h?from=nsDumpUtils.h&case=true#1
FdWatcher or FifoWatcher is what you want.

The "dumptree" can write command to a file or fifo at device, and then b2g can receive this command.

Comment 4

[Solomon Chiu [:schiu]]

I surveyed the implementation of Status Reporter in bug#964636, which output a report with JSON format,
which seems not feasible for tree style representation for dumping layer tree.
The way Jerry provided is the foundation of Status Reporter, I think it's a better way to pass the necessary control commands and data between tool and B2G.
I will also check if we use FakeSurfaceComposer to do "dumpsys", due to it inherit Android's class, and may have oppotunity to use dump().

Comment 5

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee didn't login in Bugzilla in the last 7 months, so the assignee is being reset.

Comment 6

[Boris Chiou [:boris]]

I think we don't need this any more.