tree c950af949cb80e3eb0afa5a917c61c536f8b1a41
parent 3776d862e2c3484b9b797264a1fef6782cb748d4
author Lorenzo Colitti <lorenzo@google.com> 1652862757 +0900
committer Junyu Lai <junyulai@google.com> 1652889063 +0800

Don't clobber existing history entries.

Currently, adding a history to a NetworkStatsCollection.Builder
will overwrite any history that was previously passed in with the
same key. This breaks the importer (which is the primary/only
caller of this code), because the importer re-uses the same
NetworkStatsCollection object to import multiple files.

Instead, simply add any passed-in entries after the ones that
were already there. Require the caller to pass in entries in
order, because NetworkStatsHistory internally assumes that
entris are always sorted.

Ignore-AOSP-First: in a topic with internal-only changes
Bug: 230289468
Test: manually verified this unbreaks the importer
Change-Id: Ic8647ff28fca78d579d5f759f96a864877f8158b