Remove AppsClient::GetLoadedBrowserContexts

apps/DEPS

 include_rules = [
   "+base",
   "+content/public/app",
   "+content/public/browser",
   "+content/public/common",
   "+content/public/test",
   "+components/keyed_service",
   "+components/pref_registry/pref_registry_syncable.h",
   "+components/sessions",
   "+components/user_manager",
   "+components/web_modal",
   "+extensions",
   "+net/base",
   "+skia/ext",
   "+third_party/skia/include",
   "+ui",
   "+win8",
 
   # Temporary allowed includes.
   # TODO(benwells): remove these (http://crbug.com/159366)
+  "+chrome/browser/browser_process.h",

[tapted, 2014/09/19 07:59:03]

We shouldn't be adding things here..

   "+chrome/browser/chrome_notification_types.h",
   "+chrome/browser/chromeos/login/users/user_manager.h",
   "+chrome/browser/lifetime/application_lifetime.h",
   "+chrome/browser/profiles",
   "+chrome/common/chrome_switches.h",
 
   # For default app icon.
   "!grit/theme_resources.h",
 
   # Pieces of the extensions system that need to move to src/extensions.
@@ skipping 18 matching lines @@
     "+chrome/browser/ui/browser.h",
     "+chrome/test/base/in_process_browser_test.h",
     "+chrome/test/base/interactive_test_utils.h",
     "+chrome/test/base/testing_profile.h",
     # Temporary allowed testing include.
     # TODO(benwells): Move app_browsertest_util into apps component once there
     # is an apps shell to test with.
     "+chrome/browser/apps/app_browsertest_util.h",
   ]
 }