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

Unified Diff: chrome/common/json_value_serializer.h

Issue 62117: linux: pass a bunch more ui tests. (Closed)
Patch Set: more tests Created 11 years, 8 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
« no previous file with comments | « chrome/chrome.gyp ('k') | chrome/common/pref_service_uitest.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 92bd06924ca75a57cacb1d7ed7ea4c924e7b1f04..2c71e47b6b6883264ce9875505cee1096113bdca 100644
--- a/chrome/common/json_value_serializer.h
+++ b/chrome/common/json_value_serializer.h
@@ -2,8 +2,8 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef CHROME_COMMON_JSON_VALUE_SERIALIZER_H__
-#define CHROME_COMMON_JSON_VALUE_SERIALIZER_H__
+#ifndef CHROME_COMMON_JSON_VALUE_SERIALIZER_H_
+#define CHROME_COMMON_JSON_VALUE_SERIALIZER_H_
#include <string>
@@ -67,7 +67,7 @@ class JSONFileValueSerializer : public ValueSerializer {
// deserialization or the destination of the serialization.
// When deserializing, the file should exist, but when serializing, the
// serializer will attempt to create the file at the specified location.
- JSONFileValueSerializer(const FilePath& json_file_path)
+ explicit JSONFileValueSerializer(const FilePath& json_file_path)
: json_file_path_(json_file_path) {}
~JSONFileValueSerializer() {}
@@ -92,7 +92,7 @@ class JSONFileValueSerializer : public ValueSerializer {
private:
FilePath json_file_path_;
- DISALLOW_EVIL_CONSTRUCTORS(JSONFileValueSerializer);
+ DISALLOW_IMPLICIT_CONSTRUCTORS(JSONFileValueSerializer);
};
-#endif // CHROME_COMMON_JSON_VALUE_SERIALIZER_H__
+#endif // CHROME_COMMON_JSON_VALUE_SERIALIZER_H_
« no previous file with comments | « chrome/chrome.gyp ('k') | chrome/common/pref_service_uitest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698