Wrap TimeFormat unittests into unnamed namespace.

chrome/common/time_format.h

Index: chrome/common/time_format.h
diff --git a/chrome/common/time_format.h b/chrome/common/time_format.h
index d94ac14e30c0d436c5064349b9d39ec63cbab00c..cd2799e8b2db6e62aa2b004d722bcb5f7d44dd59 100644
--- a/chrome/common/time_format.h
+++ b/chrome/common/time_format.h
@@ -2,12 +2,11 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef CHROME_COMMON_TIME_FORMAT_H__
-#define CHROME_COMMON_TIME_FORMAT_H__
+#ifndef CHROME_COMMON_TIME_FORMAT_H_
+#define CHROME_COMMON_TIME_FORMAT_H_
 #pragma once
 
-// This file defines methods to format time values as strings.
-
+#include "base/basictypes.h"
 #include "base/string16.h"
 
 namespace base {
@@ -15,6 +14,7 @@ class Time;
 class TimeDelta;
 }
 
+// This class defines methods to format time values as strings.

[Paweł Hajdan Jr., 2011/10/18 09:59:35]

nit: Omit "This class defines". Or maybe the whole thing should be changed from
a static class to a namespace?

[tfarina, 2011/10/18 16:00:53]

Yeah, that is what I had in mind for a follow up patch. But I was not sure if it
would be a good idea.

 class TimeFormat {
  public:
   // TimeElapsed, TimeRemaining and TimeRemainingShort functions:
@@ -46,6 +46,9 @@ class TimeFormat {
   // time once at the beginning and pass it for each computation.
   static string16 RelativeDate(const base::Time& time,
                                const base::Time* optional_midnight_today);
+
+ private:
+  DISALLOW_IMPLICIT_CONSTRUCTORS(TimeFormat);
 };
 
-#endif  // CHROME_COMMON_TIME_FORMAT_H__
+#endif  // CHROME_COMMON_TIME_FORMAT_H_