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

Unified Diff: sky/services/platform/url_request_types.cc

Issue 1157783002: Update to newer network service implementation and mojoms from monet (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 7 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
Index: sky/services/platform/url_request_types.cc
diff --git a/sky/services/platform/url_request_types.cc b/sky/services/platform/url_request_types.cc
index 18e4a96a4845385dbde06c295f054a7da178d358..367577b2c3ca6276330e04e1ceb451d3f6c8a8d0 100644
--- a/sky/services/platform/url_request_types.cc
+++ b/sky/services/platform/url_request_types.cc
@@ -31,21 +31,27 @@ class HeaderFlattener : public blink::WebHTTPHeaderVisitor {
if (LowerCaseEqualsASCII(name_latin1, "accept"))
has_accept_header_ = true;
- buffer_.push_back(name_latin1 + ": " + value_latin1);
+ auto header = HttpHeader::New();
+ header->name = name_latin1;
+ header->value = value_latin1;
+ buffer_.push_back(header.Pass());
}
- Array<String> GetBuffer() {
+ Array<HttpHeaderPtr> GetBuffer() {
// In some cases, WebKit doesn't add an Accept header, but not having the
// header confuses some web servers. See bug 808613.
if (!has_accept_header_) {
- buffer_.push_back("Accept: */*");
+ auto accept_header = HttpHeader::New();
+ accept_header->name = "Accept";
+ accept_header->value = "*/*";
+ buffer_.push_back(accept_header.Pass());
has_accept_header_ = true;
}
return buffer_.Pass();
}
private:
- Array<String> buffer_;
+ Array<HttpHeaderPtr> buffer_;
bool has_accept_header_;
};
« no previous file with comments | « sky/services/oknet/src/org/domokit/oknet/UrlLoaderImpl.java ('k') | sky/services/platform/weburlloader_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698