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

Unified Diff: net/test/embedded_test_server/http_response.cc

Issue 1376593007: SSL in EmbeddedTestServer (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Adding TODO to fix handlers post-Start. Created 5 years, 2 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: net/test/embedded_test_server/http_response.cc
diff --git a/net/test/embedded_test_server/http_response.cc b/net/test/embedded_test_server/http_response.cc
index 04155b5ffa529781480646dfa57f8a39caea081b..e3b43591d1f2d370304005850fcca9103ee85e88 100644
--- a/net/test/embedded_test_server/http_response.cc
+++ b/net/test/embedded_test_server/http_response.cc
@@ -33,12 +33,13 @@ std::string BasicHttpResponse::ToResponseString() const {
code_,
http_reason_phrase.c_str());
base::StringAppendF(&response_builder, "Connection: close\r\n");
- base::StringAppendF(&response_builder,
- "Content-Length: %" PRIuS "\r\n",
- content_.size());
- base::StringAppendF(&response_builder,
- "Content-Type: %s\r\n",
- content_type_.c_str());
+
+ if (content_type_.size() != 0 || content_.size() != 0) {
+ base::StringAppendF(&response_builder, "Content-Length: %" PRIuS "\r\n",
+ content_.size());
+ base::StringAppendF(&response_builder, "Content-Type: %s\r\n",
+ content_type_.c_str());
+ }
for (size_t i = 0; i < custom_headers_.size(); ++i) {
const std::string& header_name = custom_headers_[i].first;
const std::string& header_value = custom_headers_[i].second;
@@ -54,5 +55,10 @@ std::string BasicHttpResponse::ToResponseString() const {
return response_builder + content_;
}
+void BasicHttpResponse::SendResponse(const SendBytesCallback& send,
+ const SendCompleteCallback& done) {
+ send.Run(ToResponseString(), done);
+}
+
} // namespace test_server
} // namespace net

Powered by Google App Engine
This is Rietveld 408576698