app_list: Re-enable people search.

chrome/browser/ui/app_list/search/search_controller_factory.cc

Index: chrome/browser/ui/app_list/search/search_controller_factory.cc
diff --git a/chrome/browser/ui/app_list/search/search_controller_factory.cc b/chrome/browser/ui/app_list/search/search_controller_factory.cc
index a193f41af46f19dd345c344b2d510c5807b47ee9..d1a5a2c4caab4d04b2a1d8ea4f4fc6163c0eba60 100644
--- a/chrome/browser/ui/app_list/search/search_controller_factory.cc
+++ b/chrome/browser/ui/app_list/search/search_controller_factory.cc
@@ -55,6 +55,9 @@ scoped_ptr<SearchController> CreateSearchController(
   controller->AddProvider(Mixer::WEBSTORE_GROUP,
                           scoped_ptr<SearchProvider>(
                               new WebstoreProvider(profile, list_controller)));
+  controller->AddProvider(
+      Mixer::PEOPLE_GROUP,

[Matt Giuca, 2015/01/15 01:37:11]

Note the wrapping here is different to the above... I've no idea why but
ClangFormat chooses to format this line differently. (Even if I re-run
ClangFormat over this whole block of code, it still formats this line
differently to the others.)

[tapted, 2015/01/15 01:45:34]

I think more points are assigned to to not splitting the arguments to
PeopleProvider. I think s/WebstoreProvider/WebstoProvider/ would format that
like this too. (or `Omnibo`)

+      scoped_ptr<SearchProvider>(new PeopleProvider(profile, list_controller)));
   if (IsSuggestionsSearchProviderEnabled()) {
     controller->AddProvider(
         Mixer::SUGGESTIONS_GROUP,