test IWYU fixups for base

base/time/default_clock.h

 // Copyright (c) 2012 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 BASE_DEFAULT_CLOCK_H_
 #define BASE_DEFAULT_CLOCK_H_
 
 #include "base/base_export.h"
 #include "base/compiler_specific.h"
 #include "base/time/clock.h"
+#include "base/time/time.h"

[Nico, 2013/08/29 17:35:59]

Does this one make sense? Time is only used in the Now() function, which is an
OVERRIDE, so the include for the superclass necessarily pulls time.h in already,
no?

[Mostyn Bramley-Moore, 2013/08/29 18:43:51]

Done.

(I'll put together a list of the cases where we think IWYU reports something we
disagree with.)

 
 namespace base {
 
 // DefaultClock is a Clock implementation that uses Time::Now().
 class BASE_EXPORT DefaultClock : public Clock {
  public:
   virtual ~DefaultClock();
 
   // Simply returns Time::Now().
   virtual Time Now() OVERRIDE;
 };
 
 }  // namespace base
 
 #endif  // BASE_DEFAULT_CLOCK_H_