tree 4abd34e63480b78cfeea7d7c0aa5056cd80164d9
parent 514938dc6cc0ebcbd397764b2a59c9eec796cb97
author Eric Caruso <ejcaruso@chromium.org> 1526588114 -0700
committer chrome-bot <chrome-bot@chromium.org> 1526689200 -0700

libbrillo: avoid double-binding callbacks and operator()

The second bind is a no-op since it doesn't provide any extra
arguments, so it's just an unnecessary allocation. Also, it looks
like new versions of base::Bind don't like functor objects, so
instead of packaging the callback's environment as part of an
enclosing object we should just bind in the bits we need as
arguments.

BUG=b:37434548
TEST=unit tests

Change-Id: I4bed9857655958f300bc99fa9508f2e940fc4a9a
Reviewed-on: https://chromium-review.googlesource.com/1064880
Commit-Ready: Eric Caruso <ejcaruso@chromium.org>
Tested-by: Eric Caruso <ejcaruso@chromium.org>
Reviewed-by: Eric Caruso <ejcaruso@chromium.org>