Small refactoring for app_search_provider.

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

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/app_list/search/app_search_provider.h"
 
 #include <string>
 
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/chrome_notification_types.h"
@@ skipping 87 matching lines @@
   apps_.clear();
   ExtensionRegistry* registry = ExtensionRegistry::Get(profile_);
   AddApps(registry->enabled_extensions());
   AddApps(registry->disabled_extensions());
   AddApps(registry->terminated_extensions());
 }
 
 void AppSearchProvider::Observe(int type,
                                 const content::NotificationSource& source,
                                 const content::NotificationDetails& detaila) {
-  RefreshApps();
+  switch (type) {
+    case chrome::NOTIFICATION_EXTENSION_LOADED_DEPRECATED:
+    case chrome::NOTIFICATION_EXTENSION_UNINSTALLED:
+      RefreshApps();
+    default:

[tfarina, 2014/05/09 23:33:47]

I think you need a break; after RefreshApps(), don't you?

+      NOTREACHED();
+  }
 }
 
 }  // namespace app_list