Refactor views app list services to allow more code sharing

chrome/browser/ui/app_list/app_list_factory.h

Index: chrome/browser/ui/app_list/app_list_factory.h
diff --git a/chrome/browser/ui/app_list/app_list_factory.h b/chrome/browser/ui/app_list/app_list_factory.h
deleted file mode 100644
index 5f86559a608352e4ae3275957325b7493af53f72..0000000000000000000000000000000000000000
--- a/chrome/browser/ui/app_list/app_list_factory.h
+++ /dev/null
@@ -1,27 +0,0 @@
-// Copyright 2013 The Chromium Authors. All rights reserved.

[Matt Giuca, 2014/04/29 03:52:51]

I'm confused about this, is it Reitveld's fault?

This file, along with app_list_shower and app_list_shower_unittest, have been
entirely erased, but Reitveld shows them as M instead of D. Have the files
actually been deleted, or just emptied?

[tapted, 2014/04/29 04:20:30]

I blame rietveld. The raw diff has

Index: chrome/browser/ui/app_list/app_list_factory.h
diff --git a/chrome/browser/ui/app_list/app_list_factory.h
b/chrome/browser/ui/app_list/app_list_factory.h
deleted file mode 100644


Uploading to a new CL might convince Rietveld to do the right thing.

-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#ifndef CHROME_BROWSER_UI_APP_LIST_APP_LIST_FACTORY_H_
-#define CHROME_BROWSER_UI_APP_LIST_APP_LIST_FACTORY_H_
-
-#include "base/callback_forward.h"
-#include "chrome/browser/ui/app_list/app_list.h"
-
-class AppListService;
-class Profile;
-
-namespace app_list {
-class PaginationModel;
-}
-
-// Factory for AppLists. Used to allow us to create fake app lists in tests.
-class AppListFactory {
- public:
-  virtual ~AppListFactory() {}
-  virtual AppList* CreateAppList(Profile* profile,
-                                 AppListService* service,
-                                 const base::Closure& on_should_dismiss) = 0;
-};
-
-#endif  // CHROME_BROWSER_UI_APP_LIST_APP_LIST_FACTORY_H_