Sync removal of Default apps.

chrome/browser/ui/app_list/app_list_syncable_service_factory.cc

Index: chrome/browser/ui/app_list/app_list_syncable_service_factory.cc
diff --git a/chrome/browser/ui/app_list/app_list_syncable_service_factory.cc b/chrome/browser/ui/app_list/app_list_syncable_service_factory.cc
index 616bd0cfc1801f3bd045b675e4b7ab70c68840ad..98f49b1a764c978d53a3fdbec62e2b447def167e 100644
--- a/chrome/browser/ui/app_list/app_list_syncable_service_factory.cc
+++ b/chrome/browser/ui/app_list/app_list_syncable_service_factory.cc
@@ -40,9 +40,7 @@ BrowserContextKeyedService*
 AppListSyncableServiceFactory::BuildServiceInstanceFor(
     content::BrowserContext* browser_context) const {
   Profile* profile = static_cast<Profile*>(browser_context);
-  ExtensionService* extension_service =
-      extensions::ExtensionSystem::Get(profile)->extension_service();
-  return new AppListSyncableService(profile, extension_service);
+  return new AppListSyncableService(profile);

[tapted, 2013/12/19 01:20:57]

Is there a secondary reason for this change, apart from style? I had this
originally at
https://codereview.chromium.org/77773002/diff/170001/chrome/browser/ui/app_li...,
and benwells@ suggested it was clearer to feed in the ExtensionService
dependency here explicitly.

[stevenjb, 2013/12/19 01:48:44]

Hmm, I guess that's a fair point. I did this so that we weren't doing the lookup
in multiple places. Instead I'll store extension_service_ as a member of
AppListSyncableService, independent of profile_.

[tapted, 2013/12/19 02:18:02]

Hm. Caching the extension service data member/cache scares me a bit, but it does
seem silly to fetch it in both places. TBH, I think I like the earlier patchset
better (sorry!). I think having to now use the ExtensionService in multiple
places in AppListSyncableService is perhaps reason enough not to pass it in
here.

We could also still pass it in here, to be explicit. But either way, I think I
prefer the anonymous GetExtensionService(Profile*) function over the
ExtensionService* data member.

 }
 
 void AppListSyncableServiceFactory::RegisterProfilePrefs(