SSL in EmbeddedTestServer

net/test/embedded_test_server/http_response.h

 // 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.
 
 #ifndef NET_TEST_EMBEDDED_TEST_SERVER_HTTP_RESPONSE_H_
 #define NET_TEST_EMBEDDED_TEST_SERVER_HTTP_RESPONSE_H_
 
 #include <map>
 #include <string>
 
 #include "base/basictypes.h"
+#include "base/callback.h"
 #include "base/compiler_specific.h"
 #include "base/strings/string_split.h"
 #include "net/http/http_status_code.h"
 
 namespace net {
 namespace test_server {
 
+// Callback called when the response is done being sent.
+typedef base::Callback<void(void)> SendDoneCallback;

[davidben, 2015/10/13 19:43:48]

Nit: using SendDoneCallback = base::Callback<void(void)>; for new code

[svaldez, 2015/10/13 20:54:44]

Done.

+
+// Callback called when the response is ready to be sent.
+typedef base::Callback<void(std::string response, SendDoneCallback done)>
+    SendCallback;
+
 // Interface for HTTP response implementations.
 class HttpResponse{
  public:
   virtual ~HttpResponse();
 
-  // Returns raw contents to be written to the network socket
+  // Calls |send| with the raw contents to be written to the network socket
   // in response. If you intend to make this a valid HTTP response,
   // it should start with "HTTP/x.x" line, followed by response headers.

[davidben, 2015/10/13 19:43:48]

Should add "May call |send| reentrantly." or something. Perhaps also add "If the
HttpResponse is destroyed before the operation completes, the callback will not
be called."

[svaldez, 2015/10/13 20:54:44]

Done.

-  virtual std::string ToResponseString() const = 0;
+  virtual void SendResponse(SendCallback send, SendDoneCallback done) = 0;

[davidben, 2015/10/13 19:43:48]

Why do you need both callbacks? It looks like SendDoneCallback is only ever
passed into SendCallback, right? So you could just bind it when calling
SendResponse.

[davidben, 2015/10/13 19:43:48]

Nit: I'd probably call this GetResponse or something about putting the response
to a string. As far as the HttpResponse is concerned, it's just asynchronously
providing a string. It's the EmbeddedTestServer and stuff that decides to send
it.

[svaldez, 2015/10/13 20:54:44]

I guess that's kind of true in the base case, however we also want to support
cases where SendResponse is the thing setting up and running the send task.

[svaldez, 2015/10/13 20:54:44]

Some subclasses (not implemented yet) will need to be able to do SendCallback
multiple times, without SendDoneCallback being called until the final one.
Notably chunking will need this.

 };
 
 // This class is used to handle basic HTTP responses with commonly used
 // response headers such as "Content-Type".
 class BasicHttpResponse : public HttpResponse {
  public:
   BasicHttpResponse();
   ~BasicHttpResponse() override;
 
   // The response code.
   HttpStatusCode code() const { return code_; }
   void set_code(HttpStatusCode code) { code_ = code; }
 
   // The content of the response.
   const std::string& content() const { return content_; }
   void set_content(const std::string& content) { content_ = content; }
 
   // The content type.
   const std::string& content_type() const { return content_type_; }
   void set_content_type(const std::string& content_type) {
     content_type_ = content_type;
   }
 
   // Adds a custom header.
   void AddCustomHeader(const std::string& key, const std::string& value) {
     custom_headers_.push_back(std::make_pair(key, value));
   }
 
   // Generates and returns a http response string.
-  std::string ToResponseString() const override;
+  std::string ToResponseString() const;
+
+  void SendResponse(SendCallback send, SendDoneCallback done) override;
 
  private:
   HttpStatusCode code_;
   std::string content_;
   std::string content_type_;
   base::StringPairs custom_headers_;
 
   DISALLOW_COPY_AND_ASSIGN(BasicHttpResponse);
 };
 
 }  // namespace test_server
 }  // namespace net
 
 #endif  // NET_TEST_EMBEDDED_TEST_SERVER_HTTP_RESPONSE_H_