shill: remove unnecessary callback member variables
Various shill classes construct a callback object and keep that in a
member variable for future use. As these callback objects don't need to
capture any state at the time of construction, they can be constructed
when they are first needed. Also, as they are not accessed by outside
their associated class (e.g. in unit tests), they don't need to be kept
as a member variable of their associated class. This CL simply removes
these callback member variables by constructing the callback objects in
place where they are first needed.
BUG=None
TEST=Run unit tests.
Change-Id: I831b0122c7bc5fc97a1d418c403940429b6a30ba
Reviewed-on: https://chromium-review.googlesource.com/510082
Commit-Ready: Ben Chan <benchan@chromium.org>
Tested-by: Ben Chan <benchan@chromium.org>
Reviewed-by: Eric Caruso <ejcaruso@chromium.org>
12 files changed