diff options
author | hashimoto@chromium.org <hashimoto@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-04-11 02:25:35 +0000 |
---|---|---|
committer | hashimoto@chromium.org <hashimoto@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-04-11 02:25:35 +0000 |
commit | 4a404e5356b27578de71b9b41de686768d2ecd42 (patch) | |
tree | b14264adfaaaf505677a2cf1ec7f5dafe636bae5 /chromeos/dbus/update_engine_client.cc | |
parent | 0834e1b153c28dec0ae9e90080ad7a106f1c8c84 (diff) | |
download | chromium_src-4a404e5356b27578de71b9b41de686768d2ecd42.zip chromium_src-4a404e5356b27578de71b9b41de686768d2ecd42.tar.gz chromium_src-4a404e5356b27578de71b9b41de686768d2ecd42.tar.bz2 |
chromeos: Convert D-Bus client classes' callback arguments to const reference
BUG=chromium-os:28578
TEST=build success
Review URL: https://chromiumcodereview.appspot.com/10024054
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@131697 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chromeos/dbus/update_engine_client.cc')
-rw-r--r-- | chromeos/dbus/update_engine_client.cc | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/chromeos/dbus/update_engine_client.cc b/chromeos/dbus/update_engine_client.cc index 55f5c68..5e3e21a 100644 --- a/chromeos/dbus/update_engine_client.cc +++ b/chromeos/dbus/update_engine_client.cc @@ -87,7 +87,8 @@ class UpdateEngineClientImpl : public UpdateEngineClient { } // UpdateEngineClient override. - virtual void RequestUpdateCheck(UpdateCheckCallback callback) OVERRIDE { + virtual void RequestUpdateCheck( + const UpdateCheckCallback& callback) OVERRIDE { dbus::MethodCall method_call( update_engine::kUpdateEngineInterface, update_engine::kAttemptUpdate); @@ -135,7 +136,8 @@ class UpdateEngineClientImpl : public UpdateEngineClient { } // UpdateEngineClient override. - virtual void GetReleaseTrack(GetReleaseTrackCallback callback) OVERRIDE { + virtual void GetReleaseTrack( + const GetReleaseTrackCallback& callback) OVERRIDE { dbus::MethodCall method_call( update_engine::kUpdateEngineInterface, update_engine::kGetTrack); @@ -156,7 +158,7 @@ class UpdateEngineClientImpl : public UpdateEngineClient { private: // Called when a response for RequestUpdateCheck() is received. - void OnRequestUpdateCheck(UpdateCheckCallback callback, + void OnRequestUpdateCheck(const UpdateCheckCallback& callback, dbus::Response* response) { if (!response) { LOG(ERROR) << "Failed to request update check"; @@ -183,7 +185,7 @@ class UpdateEngineClientImpl : public UpdateEngineClient { } // Called when a response for GetReleaseTrack() is received. - void OnGetReleaseTrack(GetReleaseTrackCallback callback, + void OnGetReleaseTrack(const GetReleaseTrackCallback& callback, dbus::Response* response) { if (!response) { LOG(ERROR) << "Failed to request getting release track"; @@ -254,12 +256,14 @@ class UpdateEngineClientStubImpl : public UpdateEngineClient { virtual void RemoveObserver(Observer* observer) OVERRIDE {} virtual bool HasObserver(Observer* observer) OVERRIDE { return false; } - virtual void RequestUpdateCheck(UpdateCheckCallback callback) OVERRIDE { + virtual void RequestUpdateCheck( + const UpdateCheckCallback& callback) OVERRIDE { callback.Run(UPDATE_RESULT_NOTIMPLEMENTED); } virtual void RebootAfterUpdate() OVERRIDE {} virtual void SetReleaseTrack(const std::string& track) OVERRIDE {} - virtual void GetReleaseTrack(GetReleaseTrackCallback callback) OVERRIDE { + virtual void GetReleaseTrack( + const GetReleaseTrackCallback& callback) OVERRIDE { callback.Run("beta-channel"); } virtual Status GetLastStatus() OVERRIDE { return Status(); } |