Add missing, and remove unnecessary, 'explicit' from constructors. (base/)

The style guide says that constructors which can be called with one
argument should be explicit.

For constructors which cannot be called with exactly one argument,
there is no reason to mark them explicit.

BUG=163357


Review URL: https://chromiumcodereview.appspot.com/11779022

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@176881 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 4 insertions, 4 deletions

diff --git a/base/test/trace_event_analyzer.h b/base/test/trace_event_analyzer.h
index a472639..e2bb7de 100644
--- a/base/test/trace_event_analyzer.h
+++ b/base/test/trace_event_analyzer.h
@@ -352,7 +352,7 @@ class Query {
 
   // Find BEGIN events of given |name| which also have associated END events.
   static Query MatchBeginName(const std::string& name) {
-    return (Query(EVENT_NAME) == name) && MatchBeginWithEnd();
+    return (Query(EVENT_NAME) == Query(name)) && MatchBeginWithEnd();
   }
 
   // Match given Process ID and Thread ID.
@@ -452,16 +452,16 @@ class Query {
   };
 
   // Compare with the given member.
-  Query(TraceEventMember member);
+  explicit Query(TraceEventMember member);
 
   // Compare with the given member argument value.
   Query(TraceEventMember member, const std::string& arg_name);
 
   // Compare with the given string.
-  Query(const std::string& str);
+  explicit Query(const std::string& str);
 
   // Compare with the given number.
-  Query(double num);
+  explicit Query(double num);
 
   // Construct a boolean Query that returns (left <binary_op> right).
   Query(const Query& left, const Query& right, Operator binary_op);