HTTPTransport: callers should be able to obtain the HTTP response body

util/net/http_transport.h

 // Copyright 2014 The Crashpad Authors. All rights reserved.
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at
 //
 //     http://www.apache.org/licenses/LICENSE-2.0
 //
 // Unless required by applicable law or agreed to in writing, software
 // distributed under the License is distributed on an "AS IS" BASIS,
@@ skipping 56 matching lines @@
   void SetBodyStream(scoped_ptr<HTTPBodyStream> stream);
 
   //! \brief Sets the timeout for the HTTP request. The default is 15 seconds.
   //!
   //! \param[in] timeout The request timeout, in seconds.
   void SetTimeout(double timeout);
 
   //! \brief Performs the HTTP request with the configured parameters and waits
   //!     for the execution to complete.
   //!
+  //! \param[out] response On success, this will be set to the HTTP response
+  //!     body. This parameter is optional and may be set to `nullptr` if the
+  //!     response body is not required.
+  //!
   //! \return Whether or not the request was successful, defined as returning
   //!     a HTTP status 200 (OK) code.
-  virtual bool ExecuteSynchronously() = 0;
+  virtual bool ExecuteSynchronously(std::string* response_body) = 0;
 
  protected:
   HTTPTransport();
 
   const std::string& url() const { return url_; }
   const std::string& method() const { return method_; }
   const HTTPHeaders& headers() const { return headers_; }
   HTTPBodyStream* body_stream() const { return body_stream_.get(); }
   double timeout() const { return timeout_; }
 
  private:
   std::string url_;
   std::string method_;
   HTTPHeaders headers_;
   scoped_ptr<HTTPBodyStream> body_stream_;
   double timeout_;
 
   DISALLOW_COPY_AND_ASSIGN(HTTPTransport);
 };
 
 }  // namespace crashpad
 
 #endif  // CRASHPAD_UTIL_NET_HTTP_TRANSPORT_H_