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

Unified Diff: net/tools/get_server_time/get_server_time.cc

Issue 1131113004: Convert JsonWriter::Write to taking a const ref for the in-param (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix platform specific stuff Created 5 years, 7 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/tools/get_server_time/get_server_time.cc
diff --git a/net/tools/get_server_time/get_server_time.cc b/net/tools/get_server_time/get_server_time.cc
index 9915e8ee61e93d972ce63e60f0a559a0b0db2f67..fe1c1246a00afc70381db23fabfec0b829991e9b 100644
--- a/net/tools/get_server_time/get_server_time.cc
+++ b/net/tools/get_server_time/get_server_time.cc
@@ -122,7 +122,7 @@ class PrintingLogObserver : public net::NetLog::ThreadSafeObserver {
scoped_ptr<base::Value> params(entry.ParametersToValue());
std::string params_str;
if (params.get()) {
- base::JSONWriter::Write(params.get(), &params_str);
+ base::JSONWriter::Write(*params, &params_str);
params_str.insert(0, ": ");
}

Powered by Google App Engine
This is Rietveld 408576698