Fix a bunch of minor Coverity issues.

- pass more things by const reference
- check return codes (minor severity)

Review URL: http://codereview.chromium.org/149643

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

1 files changed, 3 insertions, 3 deletions

diff --git a/chrome/browser/net/chrome_url_request_context.cc b/chrome/browser/net/chrome_url_request_context.cc
index 0ddd745..3668328 100644
--- a/chrome/browser/net/chrome_url_request_context.cc
+++ b/chrome/browser/net/chrome_url_request_context.cc
@@ -334,12 +334,12 @@ void ChromeURLRequestContext::Observe(NotificationType type,
                             &ChromeURLRequestContext::OnAcceptLanguageChange,
                             accept_language));
     } else if (*pref_name_in == prefs::kCookieBehavior) {
-      net::CookiePolicy::Type type = net::CookiePolicy::FromInt(
+      net::CookiePolicy::Type policy_type = net::CookiePolicy::FromInt(
           prefs_->GetInteger(prefs::kCookieBehavior));
       g_browser_process->io_thread()->message_loop()->PostTask(FROM_HERE,
           NewRunnableMethod(this,
                             &ChromeURLRequestContext::OnCookiePolicyChange,
-                            type));
+                            policy_type));
     }
   } else if (NotificationType::EXTENSIONS_LOADED == type) {
     ExtensionPaths* new_paths = new ExtensionPaths;
@@ -418,7 +418,7 @@ bool ChromeURLRequestContext::allowSendingCookies(const URLRequest* request)
 }
 
 void ChromeURLRequestContext::OnAcceptLanguageChange(
-    std::string accept_language) {
+    const std::string& accept_language) {
   DCHECK(MessageLoop::current() ==
          ChromeThread::GetMessageLoop(ChromeThread::IO));
   accept_language_ =