Use BrowserContext as key in API. Switch Autofill to use BC in place of Profile.

chrome/browser/autofill/DEPS

 include_rules = [
   # Autofill is being made into a Browser Component, so we have these
   # three basic rules followed by temporary exceptions.  Please don't
   # add to the list of exceptions!
   "-chrome/browser",
   "+chrome/browser/api",
   "+chrome/browser/autofill",
 
   # Permanently-allowed DEPS beyond the standard Browser
   # Components-like DEPS above go here.
   "+third_party/libphonenumber",  # For phone number i18n.
 
   # TODO(joi): Bring this list to zero.
   #
   # Do not add to the list of temporarily-allowed dependencies below,
   # and please do not introduce more #includes of these files.
+
+  # Can go once ProfileSyncService is converted.
   "!chrome/browser/profiles/profile.h",
+
+  # Moving to api
+  "!chrome/browser/profiles/profile_keyed_service.h",
+
+  # Will go when PersonalDataManagerFactory moves to c/b/configuration.
   "!chrome/browser/profiles/profile_dependency_manager.h",
-  "!chrome/browser/profiles/profile_keyed_service.h",
   "!chrome/browser/profiles/profile_keyed_service_factory.h",
+
+  # Needs abstract base.

[erikwright (departed), 2012/09/04 19:42:14]

As a side comment, I am always confused when I see "abstract base" but it turns
out to be pure-virtual (possibly with exception of a static ForContext etc.,
which is not really an exception). I think of Abstract as implying that there is
a partial implementation.

Unless I'm wrong about terminology, I wonder if we could use "pure-virtual
interface" instead. Or "component interface". The former might be a detail that
one could argue is technically not required (though will probably always be
true) and the latter is lacking a formal meaning in object-oriented design and
thus has no baggage.

I realize, in this case, that this particular comment is already removed in a
pipelined CL.

[Jói, 2012/09/04 20:10:25]

We could just say "API" since that's what we're calling the directory housing
the extracted APIs.

   "!chrome/browser/sync/profile_sync_service.h",
   "!chrome/browser/sync/profile_sync_service_factory.h",
   "!chrome/browser/sync/profile_sync_service_observer.h",
+
   "!chrome/browser/ui/tab_contents/tab_contents.h",
+
   # Do not add to the list of temporarily-allowed dependencies above,
   # and please do not introduce more #includes of these files.
 
   # TODO(akalin): Remove this dependency.
   "!sync/util/data_encryption_win.h",
 ]
 
 specific_include_rules = {
   # TODO(joi): Bring this list to zero.
   #
@@ skipping 12 matching lines @@
     "!chrome/browser/ui/tab_contents/tab_autofill_manager_delegate.h",
     "!chrome/browser/ui/tab_contents/test_tab_contents.h",
   ],
 
   # TODO(joi): Get rid of the need for this, e.g. by moving
   # PersonalDataManagerFactory to chrome/browser/configuration.
   'personal_data_manager_factory\.cc': [
     "!chrome/browser/webdata/web_data_service_factory.h",
   ],
 }