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

Unified Diff: chrome/test/chromedriver/server/http_handler.cc

Issue 23566018: [chromedriver] Remove Logger and just use base LOG. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 3 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: chrome/test/chromedriver/server/http_handler.cc
diff --git a/chrome/test/chromedriver/server/http_handler.cc b/chrome/test/chromedriver/server/http_handler.cc
index 1211bbcc2682f8d89eaea9ba7159e47f34f730b5..66f09e5e0c4a91d3f771a85a9696e8eeecebcf07 100644
--- a/chrome/test/chromedriver/server/http_handler.cc
+++ b/chrome/test/chromedriver/server/http_handler.cc
@@ -20,7 +20,6 @@
#include "chrome/test/chromedriver/alert_commands.h"
#include "chrome/test/chromedriver/chrome/adb_impl.h"
#include "chrome/test/chromedriver/chrome/device_manager.h"
-#include "chrome/test/chromedriver/chrome/log.h"
#include "chrome/test/chromedriver/chrome/status.h"
#include "chrome/test/chromedriver/chrome/version.h"
#include "chrome/test/chromedriver/net/url_request_context_getter.h"
@@ -56,9 +55,8 @@ CommandMapping::CommandMapping(HttpMethod method,
CommandMapping::~CommandMapping() {}
-HttpHandler::HttpHandler(Log* log, const std::string& url_base)
- : log_(log),
- url_base_(url_base),
+HttpHandler::HttpHandler(const std::string& url_base)
+ : url_base_(url_base),
received_shutdown_(false),
command_map_(new CommandMap()),
weak_ptr_factory_(this) {}
@@ -66,11 +64,9 @@ HttpHandler::HttpHandler(Log* log, const std::string& url_base)
HttpHandler::HttpHandler(
const base::Closure& quit_func,
const scoped_refptr<base::SingleThreadTaskRunner> io_task_runner,
- Log* log,
const std::string& url_base,
int adb_port)
: quit_func_(quit_func),
- log_(log),
url_base_(url_base),
received_shutdown_(false),
weak_ptr_factory_(this) {
@@ -79,15 +75,14 @@ HttpHandler::HttpHandler(
#endif
context_getter_ = new URLRequestContextGetter(io_task_runner);
socket_factory_ = CreateSyncWebSocketFactory(context_getter_.get());
- adb_.reset(new AdbImpl(io_task_runner, log_, adb_port));
+ adb_.reset(new AdbImpl(io_task_runner, adb_port));
device_manager_.reset(new DeviceManager(adb_.get()));
CommandMapping commands[] = {
CommandMapping(kPost,
internal::kNewSessionPathPattern,
base::Bind(&ExecuteNewSession,
- NewSessionParams(log_,
- &session_thread_map_,
+ NewSessionParams(&session_thread_map_,
context_getter_,
socket_factory_,
device_manager_.get()))),
@@ -491,11 +486,8 @@ void HttpHandler::HandleCommand(
const net::HttpServerRequestInfo& request,
const std::string& trimmed_path,
const HttpResponseSenderFunc& send_response_func) {
- log_->AddEntry(Log::kLog,
- base::StringPrintf("handling command: %s %s %s",
- request.method.c_str(),
- trimmed_path.c_str(),
- request.data.c_str()));
+ VLOG(0) << "Handling command: " << request.method << " " << trimmed_path
+ << " " << request.data;
base::DictionaryValue params;
std::string session_id;
@@ -545,10 +537,8 @@ void HttpHandler::PrepareResponse(
CHECK(thread_checker_.CalledOnValidThread());
scoped_ptr<net::HttpServerResponseInfo> response =
PrepareResponseHelper(trimmed_path, status, value.Pass(), session_id);
- log_->AddEntry(Log::kLog,
- base::StringPrintf("sending response: %d %s",
- response->status_code(),
- response->body().c_str()));
+ VLOG(0) << "Sending response: " << response->status_code() << " "
+ << response->body();
send_response_func.Run(response.Pass());
if (trimmed_path == kShutdownPath)
quit_func_.Run();

Powered by Google App Engine
This is Rietveld 408576698