diff options
| author | Alex Deymo <deymo@chromium.org> | 2014-11-10 19:55:35 -0800 |
|---|---|---|
| committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2014-11-12 19:56:58 +0000 |
| commit | aab50e31f0b80ed53a9b8d5dbabcf943974bd32c (patch) | |
| tree | df2ac7ac92e6eb5dfc24c87af119eb6cb3ee65fb /update_manager/real_time_provider_unittest.cc | |
| parent | e89487039d0eca8130d822cabf75a729609509e0 (diff) | |
Include the implemented header first in all projects.
The Google C++ style guide dictates that foo.cc and foo_unittest.cc
should include foo.h in the first place, so missing headers in foo.h
are detected with a compile error of the module implementing them and
not when another module uses them.
This CL sweeps across all the .cc file in platform2 enforcing this.
BUG=None
TEST=cbuildbot amd64-generic
Change-Id: I41835835caba13f54c3c844ecf552eb0e47efa9d
Reviewed-on: https://chromium-review.googlesource.com/228894
Tested-by: Alex Deymo <deymo@chromium.org>
Reviewed-by: Alex Deymo <deymo@chromium.org>
Commit-Queue: Alex Vakulenko <avakulenko@chromium.org>
Diffstat (limited to 'update_manager/real_time_provider_unittest.cc')
| -rw-r--r-- | update_manager/real_time_provider_unittest.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/update_manager/real_time_provider_unittest.cc b/update_manager/real_time_provider_unittest.cc index 3532ff36..7e27a4e4 100644 --- a/update_manager/real_time_provider_unittest.cc +++ b/update_manager/real_time_provider_unittest.cc @@ -2,6 +2,8 @@ // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. +#include "update_engine/update_manager/real_time_provider.h" + #include <memory> #include <base/logging.h> @@ -9,7 +11,6 @@ #include <gtest/gtest.h> #include "update_engine/fake_clock.h" -#include "update_engine/update_manager/real_time_provider.h" #include "update_engine/update_manager/umtest_utils.h" using base::Time; |