shill: refactor WakeOnWiFi
A future CL will refactor WiFi to take WakeOnWiFi as a dependency
rather than instantiating it during the constructor, so that we
can avoid making SimpleAlarmTimers when there is no message loop
in tests. Since MockWakeOnWiFi inherits from the real WakeOnWiFi,
it would still instantiate SimpleAlarmTimers on construction.
This means we should factor out the public interface and have
both implementations inherit from that instead.
BUG=b:37434548
TEST=unit tests
Change-Id: I7c1f48b33866c51a661c8c96d9b20ed27305a311
Reviewed-on: https://chromium-review.googlesource.com/1072556
Commit-Ready: Kirtika Ruchandani <kirtika@chromium.org>
Tested-by: Kirtika Ruchandani <kirtika@chromium.org>
Reviewed-by: Kirtika Ruchandani <kirtika@chromium.org>
7 files changed