commit b3aac2520d0ba88ba1aa69c30213bbc21944401e
author Alex Vakulenko <avakulenko@chromium.org> Wed May 07 17:35:24 2014 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Sat May 10 01:32:40 2014 +0000
tree 33cde25604f6ce02fd1ceb1a0b2f08c3c987cf20
parent 48e37283190da8d8f953ef6b199d2782a3a0153b

buffet: Added advanced error reporting

Created chromeos::Error class that encapsulates rich error
information from various system domains.

Swept GCD device registration, HTTP transport and utilities
to always return additional error information when the caller
requests it. This includes internal errors, general HTTP errors
as well as parsing and returning specific GCD and OAuth2 server
error responses.

Also fixed a number of existing linter warnings.

BUG=chromium:366709
TEST=All existing and new unit tests pass.

Change-Id: Ic01622a8efa3dc365ec106e595b09536818b9b23
Reviewed-on: https://chromium-review.googlesource.com/198772
Tested-by: Alex Vakulenko <avakulenko@chromium.org>
Reviewed-by: Christopher Wiley <wiley@chromium.org>
Commit-Queue: Alex Vakulenko <avakulenko@chromium.org>

buffet/http_transport_fake.cc

diff --git a/buffet/http_transport_fake.cc b/buffet/http_transport_fake.cc
index 4a3a781..782404a 100644
--- a/buffet/http_transport_fake.cc
+++ b/buffet/http_transport_fake.cc
@@ -4,6 +4,8 @@
 
 #include "buffet/http_transport_fake.h"
 
+#include <utility>
+
 #include <base/json/json_reader.h>
 #include <base/json/json_writer.h>
 #include <base/logging.h>
@@ -14,8 +16,12 @@
 #include "buffet/mime_utils.h"
 #include "buffet/url_utils.h"
 
-using namespace chromeos;
-using namespace chromeos::http::fake;
+namespace chromeos {
+
+using http::fake::Transport;
+using http::fake::ServerRequestResponseBase;
+using http::fake::ServerRequest;
+using http::fake::ServerResponse;
 
 Transport::Transport() {
   VLOG(1) << "fake::Transport created";
@@ -32,7 +38,7 @@
     const HeaderList& headers,
     const std::string& user_agent,
     const std::string& referer,
-    std::string* error_msg) {
+    ErrorPtr* error) {
   HeaderList headers_copy = headers;
   if (!user_agent.empty()) {
     headers_copy.push_back(std::make_pair(http::request_header::kUserAgent,
@@ -45,11 +51,8 @@
   std::unique_ptr<http::Connection> connection(
       new http::fake::Connection(url, method, transport));
   CHECK(connection) << "Unable to create Connection object";
-  if (!connection->SendHeaders(headers_copy)) {
+  if (!connection->SendHeaders(headers_copy, error))
     connection.reset();
-    if (error_msg)
-      *error_msg = "Failed to send request headers";
-  }
   request_count_++;
   return connection;
 }
@@ -253,3 +256,5 @@
   }
   return std::string();
 }
+
+}  // namespace chromeos