memory-infra: Remove attributes_for_testing
MemoryAllocatorDump no longer serializes directly to a TracedValue and
instead keeps track of calls to AddScalar/AddString in a vector. Due
to this we can write much better tests by looking at this vector
directly as opposed to tying all the tests to the format of the
TracedValue. This CL replaces all uses of attributes_for_testing() with
uses of entries() and removes attributes_for_testing()
which results in shorter more understandable tests.
Change-Id: I67fa2d3f6931baf067d5a27c5c82b42602d7acfd
Reviewed-on: https://chromium-review.googlesource.com/643508
Reviewed-by: Primiano Tucci <primiano@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Helen Li <xunjieli@chromium.org>
Reviewed-by: Siddhartha S <ssid@chromium.org>
Commit-Queue: Hector Dearman <hjd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#507356}
CrOS-Libchrome-Original-Commit: 2adfd4cfe9d436f6f0963057b571cc2bb1eb8a16
4 files changed