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}
diff --git a/chrome/browser/prefs/pref_functional_browsertest.cc b/chrome/browser/prefs/pref_functional_browsertest.cc
index 0127db2..55b93a9 100644
--- a/chrome/browser/prefs/pref_functional_browsertest.cc
+++ b/chrome/browser/prefs/pref_functional_browsertest.cc
@@ -16,9 +16,11 @@
#include "chrome/test/base/testing_browser_process.h"
#include "chrome/test/base/testing_profile.h"
#include "chrome/test/base/ui_test_utils.h"
+#include "components/bookmarks/common/bookmark_pref_names.h"
#include "components/content_settings/core/browser/website_settings_info.h"
#include "components/content_settings/core/browser/website_settings_registry.h"
#include "components/content_settings/core/common/content_settings_types.h"
+#include "components/content_settings/core/common/pref_names.h"
#include "components/syncable_prefs/pref_service_syncable.h"
#include "content/public/browser/web_contents.h"
#include "content/public/test/browser_test_utils.h"
