Wrap TimeFormat unittests into unnamed namespace.

chrome/common/time_format.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // 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 {
 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:
   // These functions return a localized string of approximate time duration. The
   // conditions are simpler than PastTime since these functions are used for
   // in-progress operations and users have different expectations of units.
 
   // Returns times in elapsed-format: "3 mins ago", "2 days ago".
   static string16 TimeElapsed(const base::TimeDelta& delta);
 
@@ skipping 11 matching lines @@
   // TODO(tc): This should be able to do things like "Last week".  This
   //    requires handling singluar/plural for all languages.
   //
   // The second parameter is optional, it is midnight of "Now" for relative day
   // computations: Time::Now().LocalMidnight()
   // If NULL, the current day's midnight will be retrieved, which can be
   // slow. If many items are being processed, it is best to get the current
   // 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_