Add a DEPS entry for profile.(cc|h) to deter new deps on //c/b

We should try to move profile to its own GN target (see linked bug).
To do so we need to remove a few more deps from profile onto headers in
//c/b. This CL adds a DEPS entry to prevent/deter new deps on //c/b
from being added in the mean time.

Bug: 1149277
Change-Id: I190c49fb3432ba275db3fb605f5f57e709779113
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2568994Reviewed-by: Mihai Sardarescu <msarda@chromium.org>
Reviewed-by: Christian Dullweber <dullweber@chromium.org>
Reviewed-by: Gabriel Charette <gab@chromium.org>
Commit-Queue: Glen Robertson <glenrob@chromium.org>
Cr-Commit-Position: refs/heads/master@{#834483}

chrome/browser/profiles/DEPS

@@ -4,4 +4,18 @@ specific_include_rules = {
     # is created.
     "+chrome/browser/performance_manager/persistence/site_data/site_data_cache_facade_factory.h"
   ],
+  # TODO(crbug.com/1149277): Remove include rules for profile once it is in its
+  # own GN target. GN will prevent dependency cycles more reliably.
+  "profile\.(cc|h)": [
+    # Profile is a dependency for most of //chrome/browser for keyed services.
+    # It should not depend on anything in the //chrome/browser target.
+    "-chrome/browser",
+    "+chrome/browser/profiles/profile.h",
+    # Files without dependencies on //chrome/browser:
+    "+chrome/browser/browsing_data/chrome_browsing_data_remover_constants.h",
+    "+chrome/browser/profiles/android/jni_headers",
+    "+chrome/browser/profiles/profile_observer.h",
+    # TODO(crbug.com/1149277): Remove this:
+    "+chrome/browser/first_run/first_run.h",
+  ]
 }