Provide mutable members of UrlRequestContext via pure-virtual interface

chrome/browser/automation/url_request_automation_job.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/automation/url_request_automation_job.h"
 
 #include "base/bind.h"
 #include "base/compiler_specific.h"
 #include "base/message_loop.h"
 #include "base/time.h"
@@ skipping 404 matching lines @@
 
   // Strip unwanted headers.
   net::HttpRequestHeaders new_request_headers;
   new_request_headers.MergeFrom(request_->extra_request_headers());
   for (size_t i = 0; i < arraysize(kFilteredHeaderStrings); ++i)
     new_request_headers.RemoveHeader(kFilteredHeaderStrings[i]);
 
   // Only add default Accept-Language and Accept-Charset if the request
   // didn't have them specified.
   if (!new_request_headers.HasHeader(
-      net::HttpRequestHeaders::kAcceptLanguage) &&
-      !request_->context()->accept_language().empty()) {
-    new_request_headers.SetHeader(net::HttpRequestHeaders::kAcceptLanguage,
-                                  request_->context()->accept_language());
+      net::HttpRequestHeaders::kAcceptLanguage)) {
+    std::string accept_language = request_->context()->GetAcceptLanguage();
+    if (!accept_language.empty()) {
+      new_request_headers.SetHeader(net::HttpRequestHeaders::kAcceptLanguage,
+                                    accept_language);
+    }
   }
   if (!new_request_headers.HasHeader(
-      net::HttpRequestHeaders::kAcceptCharset) &&
-      !request_->context()->accept_charset().empty()) {
-    new_request_headers.SetHeader(net::HttpRequestHeaders::kAcceptCharset,
-                                  request_->context()->accept_charset());
+      net::HttpRequestHeaders::kAcceptCharset)) {
+    std::string accept_charset = request_->context()->GetAcceptCharset();
+    if (!accept_charset.empty()) {
+      new_request_headers.SetHeader(net::HttpRequestHeaders::kAcceptCharset,
+                                    accept_charset);
+    }
   }
 
   // Ensure that we do not send username and password fields in the referrer.
   GURL referrer(request_->GetSanitizedReferrer());
 
   // The referrer header must be suppressed if the preceding URL was
   // a secure one and the new one is not.
   if (referrer.SchemeIsSecure() && !request_->url().SchemeIsSecure()) {
     DVLOG(1) << "Suppressing referrer header since going from secure to "
                 "non-secure";
@@ skipping 43 matching lines @@
   if (!is_done()) {
     NotifyDone(request_status_);
   }
   // Reset any pending reads.
   if (pending_buf_) {
     pending_buf_ = NULL;
     pending_buf_size_ = 0;
     NotifyReadComplete(0);
   }
 }