Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1496)

Unified Diff: content/browser/speech/google_streaming_remote_engine.cc

Issue 83433008: Cleanup: Remove legacy accessors from URLRequestContext. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/speech/google_streaming_remote_engine.cc
diff --git a/content/browser/speech/google_streaming_remote_engine.cc b/content/browser/speech/google_streaming_remote_engine.cc
index b6b68d17ced6e56691ba4f45547aaf959477021e..bee757c57a0193cdde56a0dee92989c7048d51c4 100644
--- a/content/browser/speech/google_streaming_remote_engine.cc
+++ b/content/browser/speech/google_streaming_remote_engine.cc
@@ -21,6 +21,7 @@
#include "google_apis/google_api_keys.h"
#include "net/base/escape.h"
#include "net/base/load_flags.h"
+#include "net/url_request/http_user_agent_settings.h"
#include "net/url_request/url_fetcher.h"
#include "net/url_request/url_request_context.h"
#include "net/url_request/url_request_context_getter.h"
@@ -563,7 +564,11 @@ std::string GoogleStreamingRemoteEngine::GetAcceptedLanguages() const {
// TODO(pauljensen): GoogleStreamingRemoteEngine should be constructed with
// a reference to the HttpUserAgentSettings rather than accessing the
// accept language through the URLRequestContext.
- std::string accepted_language_list = request_context->GetAcceptLanguage();
+ std::string accepted_language_list;
+ if (request_context->http_user_agent_settings()) {
+ accepted_language_list =
+ request_context->http_user_agent_settings()->GetAcceptLanguage();
+ }
size_t separator = accepted_language_list.find_first_of(",;");
if (separator != std::string::npos)
langs = accepted_language_list.substr(0, separator);
no longer working on chromium 2013/12/30 12:35:15 ditto.
Lei Zhang 2014/01/08 22:44:25 Done.

Powered by Google App Engine
This is Rietveld 408576698