Simplified internal class MemoryPressureObserver.
Removed extra indirection in MemoryPressureObserver.
MemoryPressureObserver::sync_observers_ used to be a (leaky) pointer. There was
no good reason for this extra level of indirection and extra dynamic memory
allocation. Made MemoryPressureObserver::sync_observers_ a non-pointer member.
The leak was irrelevant because only one instance of MemoryPressureObserver is
created and it is never deleted. Made this more explicit by declaring
MemoryPressureObserver's destructor as deleted.
Added some consts.
Added some comments.
Change-Id: I433191730e8d73804fae2ab7636f00b177ca08cd
Bug:
Reviewed-on: https://chromium-review.googlesource.com/786710
Commit-Queue: François Degros <fdegros@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#519039}
CrOS-Libchrome-Original-Commit: ad2ac19bbe6a4b4ec874aef844059819e95d2676
1 file changed