Add WARN_UNUSED_RESULT to scoped_refptr::release.

Also fixed some possible leaks related by scoped_refptr::release.

See https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/b5thcBgubsM


BUG=


Review URL: https://chromiumcodereview.appspot.com/10874054

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153719 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 2 insertions, 3 deletions

diff --git a/chrome/service/cloud_print/cloud_print_connector.cc b/chrome/service/cloud_print/cloud_print_connector.cc
index 4187ab0..9d8fc351 100644
--- a/chrome/service/cloud_print/cloud_print_connector.cc
+++ b/chrome/service/cloud_print/cloud_print_connector.cc
@@ -54,7 +54,7 @@ bool CloudPrintConnector::InitPrintSystem() {
   }
   cloud_print::PrintSystem::PrintSystemResult result = print_system_->Init();
   if (!result.succeeded()) {
-    print_system_.release();
+    print_system_ = NULL;
     // We could not initialize the print system. We need to notify the server.
     ReportUserMessage(kPrintSystemFailedMessageId, result.message());
     return false;
@@ -86,7 +86,7 @@ void CloudPrintConnector::Stop() {
   DCHECK(IsRunning());
   // Do uninitialization here.
   pending_tasks_.clear();
-  print_server_watcher_.release();
+  print_server_watcher_ = NULL;
   request_ = NULL;
 }
 
@@ -588,4 +588,3 @@ bool CloudPrintConnector::IsSamePrinter(const std::string& name1,
                                         const std::string& name2) const {
   return (0 == base::strcasecmp(name1.c_str(), name2.c_str()));
 }
-