diff options
| author | kmadhusu@chromium.org <kmadhusu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-05-10 01:22:11 +0000 |
|---|---|---|
| committer | kmadhusu@chromium.org <kmadhusu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-05-10 01:22:11 +0000 |
| commit | 5fa510ea11c1c1ce478a28b68a72c8448754df18 (patch) | |
| tree | 60045f00c460ae6cd346bdda6723f8cd1ea0d065 /chrome | |
| parent | 26c6d5022ff348a8ce620203bac8f9181d355131 (diff) | |
| download | chromium_src-5fa510ea11c1c1ce478a28b68a72c8448754df18.zip chromium_src-5fa510ea11c1c1ce478a28b68a72c8448754df18.tar.gz chromium_src-5fa510ea11c1c1ce478a28b68a72c8448754df18.tar.bz2 | |
Coverity: Fix pass by value errors.
CID=103965, 103964
BUG=none
TEST=none
Review URL: https://chromiumcodereview.appspot.com/10386054
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136214 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome')
4 files changed, 5 insertions, 5 deletions
diff --git a/chrome/service/cloud_print/cloud_print_proxy.cc b/chrome/service/cloud_print/cloud_print_proxy.cc index 516e780..6b4e2ce 100644 --- a/chrome/service/cloud_print/cloud_print_proxy.cc +++ b/chrome/service/cloud_print/cloud_print_proxy.cc @@ -266,7 +266,7 @@ void CloudPrintProxy::OnPrintSystemUnavailable() { void CloudPrintProxy::OnUnregisterPrinters( const std::string& auth_token, - const std::list<std::string> printer_ids) { + const std::list<std::string>& printer_ids) { ShutdownBackend(); wipeout_.reset(new CloudPrintWipeout(this, cloud_print_server_url_)); wipeout_->UnregisterPrinters(auth_token, printer_ids); diff --git a/chrome/service/cloud_print/cloud_print_proxy.h b/chrome/service/cloud_print/cloud_print_proxy.h index 6816540..2caad1b 100644 --- a/chrome/service/cloud_print/cloud_print_proxy.h +++ b/chrome/service/cloud_print/cloud_print_proxy.h @@ -66,7 +66,7 @@ class CloudPrintProxy : public CloudPrintProxyFrontend, virtual void OnPrintSystemUnavailable() OVERRIDE; virtual void OnUnregisterPrinters( const std::string& auth_token, - const std::list<std::string> printer_ids) OVERRIDE; + const std::list<std::string>& printer_ids) OVERRIDE; // CloudPrintWipeout::Client implementation. virtual void OnUnregisterPrintersComplete() OVERRIDE; diff --git a/chrome/service/cloud_print/cloud_print_proxy_backend.cc b/chrome/service/cloud_print/cloud_print_proxy_backend.cc index b11bffb..fd11cae 100644 --- a/chrome/service/cloud_print/cloud_print_proxy_backend.cc +++ b/chrome/service/cloud_print/cloud_print_proxy_backend.cc @@ -112,7 +112,7 @@ class CloudPrintProxyBackend::Core void NotifyAuthenticationFailed(); void NotifyPrintSystemUnavailable(); void NotifyUnregisterPrinters(const std::string& auth_token, - const std::list<std::string> printer_ids); + const std::list<std::string>& printer_ids); // Init XMPP channel void InitNotifications(const std::string& robot_email, @@ -498,7 +498,7 @@ void CloudPrintProxyBackend::Core::NotifyPrintSystemUnavailable() { void CloudPrintProxyBackend::Core::NotifyUnregisterPrinters( const std::string& auth_token, - const std::list<std::string> printer_ids) { + const std::list<std::string>& printer_ids) { DCHECK(MessageLoop::current() == backend_->frontend_loop_); backend_->frontend_->OnUnregisterPrinters(auth_token, printer_ids); } diff --git a/chrome/service/cloud_print/cloud_print_proxy_backend.h b/chrome/service/cloud_print/cloud_print_proxy_backend.h index 1218438..d60e2fc 100644 --- a/chrome/service/cloud_print/cloud_print_proxy_backend.h +++ b/chrome/service/cloud_print/cloud_print_proxy_backend.h @@ -43,7 +43,7 @@ class CloudPrintProxyFrontend { // Receive auth token and list of printers. virtual void OnUnregisterPrinters( const std::string& auth_token, - const std::list<std::string> printer_ids) = 0; + const std::list<std::string>& printer_ids) = 0; protected: // Don't delete through SyncFrontend interface. |