C++ readability review from original change https://chromiumcodereview.appspot.com/14263024/

chrome/test/chromedriver/chrome/log.h

Index: chrome/test/chromedriver/chrome/log.h
diff --git a/chrome/test/chromedriver/chrome/log.h b/chrome/test/chromedriver/chrome/log.h
index 49c3b9247a49473a849c01f247787e728b323d46..0a1cf7e6ce82b65937e19f80236f87e26195119d 100644
--- a/chrome/test/chromedriver/chrome/log.h
+++ b/chrome/test/chromedriver/chrome/log.h
@@ -10,9 +10,10 @@
 #include "base/compiler_specific.h"
 #include "base/time.h"
 
-// Accepts log entries that have a level, timestamp, and a string message.
+// Abstract class for logging entries with a level, timestamp, string message.
 class Log {
  public:
+  // Log entry severity level.
   enum Level {
     kDebug,
     kLog,
@@ -22,12 +23,12 @@ class Log {
 
   virtual ~Log() {}
 
-  // Log a message with an explicit timestamp.
-  virtual void AddEntry(const base::Time& time,
-                        Level level,
-                        const std::string& message) = 0;
+  // Adds an entry to the log.
+  virtual void AddEntryTimestamped(const base::Time& timestamp,
+                                   Level level,
+                                   const std::string& message) = 0;
 
-  // Implicit timestamp, default to current time.
+  // Adds an entry to the log, timestamped with the current time.
   void AddEntry(Level level, const std::string& message);
 };
 
@@ -39,9 +40,9 @@ class Logger : public Log {
   explicit Logger(Level min_log_level);
   virtual ~Logger();
 
-  virtual void AddEntry(const base::Time& time,
-                        Level level,
-                        const std::string& message) OVERRIDE;
+  virtual void AddEntryTimestamped(const base::Time& timestamp,
+                                   Level level,
+                                   const std::string& message) OVERRIDE;
 
  private:
   Level min_log_level_;