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

Unified Diff: chrome/common/json_value_serializer.h

Issue 384103: This patch is the rest of the lint cleanup minus the other open issue. (Closed)
Patch Set: Created 11 years, 1 month 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
« no previous file with comments | « chrome/common/important_file_writer.h ('k') | chrome/common/libxml_utils.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/common/json_value_serializer.h
diff --git a/chrome/common/json_value_serializer.h b/chrome/common/json_value_serializer.h
index 3dd8f140a3daa8c4f668c66b8f0cc4c359e7e1cd..1f1e556e6f3114328da0f49ef8847a475d9e2502 100644
--- a/chrome/common/json_value_serializer.h
+++ b/chrome/common/json_value_serializer.h
@@ -16,7 +16,7 @@ class JSONStringValueSerializer : public ValueSerializer {
// json_string is the string that will be source of the deserialization
// or the destination of the serialization. The caller of the constructor
// retains ownership of the string.
- JSONStringValueSerializer(std::string* json_string)
+ explicit JSONStringValueSerializer(std::string* json_string)
: json_string_(json_string),
initialized_with_const_string_(false),
pretty_print_(false),
@@ -25,7 +25,7 @@ class JSONStringValueSerializer : public ValueSerializer {
// This version allows initialization with a const string reference for
// deserialization only.
- JSONStringValueSerializer(const std::string& json_string)
+ explicit JSONStringValueSerializer(const std::string& json_string)
: json_string_(&const_cast<std::string&>(json_string)),
initialized_with_const_string_(true),
pretty_print_(false),
« no previous file with comments | « chrome/common/important_file_writer.h ('k') | chrome/common/libxml_utils.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698