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

Unified Diff: components/rlz/rlz_tracker.cc

Issue 2621843002: rlz: get rid of the call to AppendHeaderIfMissing() (Closed)
Patch Set: compile Created 3 years, 11 months 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/rlz/rlz_tracker.cc
diff --git a/components/rlz/rlz_tracker.cc b/components/rlz/rlz_tracker.cc
index d18edb75570da4cf25139e843efbfb77e1924851..4a1abea602c943e19f2eae90865317410a6aa05f 100644
--- a/components/rlz/rlz_tracker.cc
+++ b/components/rlz/rlz_tracker.cc
@@ -14,11 +14,11 @@
#include "base/bind.h"
#include "base/message_loop/message_loop.h"
#include "base/strings/string_util.h"
+#include "base/strings/stringprintf.h"
#include "base/strings/utf_string_conversions.h"
#include "base/trace_event/trace_event.h"
#include "build/build_config.h"
#include "components/rlz/rlz_tracker_delegate.h"
-#include "net/http/http_util.h"
namespace rlz {
namespace {
@@ -443,9 +443,8 @@ std::string RLZTracker::GetAccessPointHttpHeader(rlz_lib::AccessPoint point) {
base::string16 rlz_string;
RLZTracker::GetAccessPointRlz(point, &rlz_string);
if (!rlz_string.empty()) {
- net::HttpUtil::AppendHeaderIfMissing("X-Rlz-String",
- base::UTF16ToUTF8(rlz_string),
- &extra_headers);
+ return base::StringPrintf("X-Rlz-String: %s\r\n",
+ base::UTF16ToUTF8(rlz_string).c_str());
}
return extra_headers;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698