Revert to reading to smaps file by default

This change reverts to showmap's original behavior of using the provided
pid argument to read directly from /proc/<pid>/smaps instead of a
ProcessRecord. This is intended to address a memory regression possibly
caused by the number of ProcessRecords being created during showmaps of
all processes.

This doesn't affect ongoing work to merge libmeminfo tools for
bugreports because a bugreport-specific tool would call run_showmap
directly instead of calling the showmap binary.

Test: Check that showmap still works as intended.
Bug: 246665942
Change-Id: If88f21d96dc23677351b53d3f2d24ca61a7effc3
1 file changed