diff options
| author | brettw <brettw@chromium.org> | 2015-10-09 14:30:46 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-10-09 21:31:32 +0000 |
| commit | e1f0af8b5a296f334fdeb0c6d7d2563a3fd95bc1 (patch) | |
| tree | 98af999b13adc270b3acfc0138ca075e825d5bb2 /chrome/browser/policy/policy_browsertest.cc | |
| parent | ca46ae47bc66b875e5dacc554acd68500f92ccf8 (diff) | |
| download | chromium_src-e1f0af8b5a296f334fdeb0c6d7d2563a3fd95bc1.zip chromium_src-e1f0af8b5a296f334fdeb0c6d7d2563a3fd95bc1.tar.gz chromium_src-e1f0af8b5a296f334fdeb0c6d7d2563a3fd95bc1.tar.bz2 | |
Remove forward includes of component pref names in Chrome's list.
Previously chrome/common/pref_names.h included pref name headers from some components. Presumably these were added when the corresponding code was componentized to avoid updating all callers.
However, this causes problems for GN's dependency checker, which correctly identifies that these includes lack a dependency to go with them.
This removes the forward includes and adds the proper finer-grained includes where necessary. I tried to remove the include for pref_names.h in a few places where it is no longer needed.
Move some out
BUG=
Review URL: https://codereview.chromium.org/1395863003
Cr-Commit-Position: refs/heads/master@{#353388}
Diffstat (limited to 'chrome/browser/policy/policy_browsertest.cc')
| -rw-r--r-- | chrome/browser/policy/policy_browsertest.cc | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/chrome/browser/policy/policy_browsertest.cc b/chrome/browser/policy/policy_browsertest.cc index d4b38f3..6fa57f2 100644 --- a/chrome/browser/policy/policy_browsertest.cc +++ b/chrome/browser/policy/policy_browsertest.cc @@ -88,8 +88,10 @@ #include "chrome/test/base/search_test_utils.h" #include "chrome/test/base/test_switches.h" #include "chrome/test/base/ui_test_utils.h" +#include "components/bookmarks/common/bookmark_pref_names.h" #include "components/content_settings/core/common/content_settings.h" #include "components/content_settings/core/common/content_settings_pattern.h" +#include "components/content_settings/core/common/pref_names.h" #include "components/infobars/core/infobar.h" #include "components/omnibox/browser/autocomplete_controller.h" #include "components/omnibox/browser/omnibox_edit_model.h" |