Provide mutable members of UrlRequestContext via pure-virtual interface

content/browser/speech/google_one_shot_remote_engine.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 "content/browser/speech/google_one_shot_remote_engine.h"
 
 #include <vector>
 
 #include "base/json/json_reader.h"
 #include "base/string_number_conversions.h"
@@ skipping 159 matching lines @@
   DCHECK(!url_fetcher_.get());
   std::string lang_param = config_.language;
 
   if (lang_param.empty() && url_context_) {
     // If no language is provided then we use the first from the accepted
     // language list. If this list is empty then it defaults to "en-US".
     // Example of the contents of this list: "es,en-GB;q=0.8", ""
     net::URLRequestContext* request_context =
         url_context_->GetURLRequestContext();
     DCHECK(request_context);
-    std::string accepted_language_list = request_context->accept_language();
+    std::string accepted_language_list = request_context->GetAcceptLanguage();

[willchan no longer on Chromium, 2012/10/06 19:32:49]

Long-term, I think we should construct this guy with a reference to the
HttpUserAgentSettings directly, rather than going through the URLRequestContext.
Leave a TODO?

Since you're new, I'll explain that a TODO doesn't mean that you are going to do
it, but leave your username so that people know who to ask about it. Or explain
it in a bug and link that.

     size_t separator = accepted_language_list.find_first_of(",;");
     lang_param = accepted_language_list.substr(0, separator);
   }
 
   if (lang_param.empty())
     lang_param = "en-US";
 
   std::vector<std::string> parts;
   parts.push_back("lang=" + net::EscapeQueryParamValue(lang_param, true));
 
@@ skipping 96 matching lines @@
 
 bool GoogleOneShotRemoteEngine::IsRecognitionPending() const {
   return url_fetcher_ != NULL;
 }
 
 int GoogleOneShotRemoteEngine::GetDesiredAudioChunkDurationMs() const {
   return kAudioPacketIntervalMs;
 }
 
 }  // namespace speech