AppListController refactoring part 1: AppListShower implementation.

ui/app_list/app_list_shower_delegate_factory.h

+// Copyright 2016 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.
+
+#ifndef UI_APP_LIST_APP_LIST_SHOWER_DELEGATE_FACTORY_H_
+#define UI_APP_LIST_APP_LIST_SHOWER_DELEGATE_FACTORY_H_
+
+#include "base/memory/scoped_ptr.h"
+#include "ui/app_list/app_list_shower.h"
+#include "ui/app_list/app_list_shower_delegate.h"

[xiyuan, 2016/03/24 20:31:11]

nit: Since we forward declared AppListShower and AppListShowerDelegate, we don't
need the two headers. Or if we need to headers, then remove the forward decl.

[mfomitchev, 2016/03/29 17:33:31]

Done.

+
+namespace app_list {
+class AppListShower;

[xiyuan, 2016/03/24 20:31:11]

nit: insert an empty line

[mfomitchev, 2016/03/29 17:33:31]

Done.

+class AppListShowerDelegate;
+
+class AppListShowerDelegateFactory {
+ public:
+  virtual ~AppListShowerDelegateFactory() {}
+
+  virtual scoped_ptr<app_list::AppListShowerDelegate> GetDelegate(
+      app_list::AppListShower* shower) = 0;

[xiyuan, 2016/03/24 20:31:11]

nit: "app_list::" is redundant

[mfomitchev, 2016/03/29 17:33:31]

Done.

+};
+
+}  // namespace app_list
+
+#endif  // UI_APP_LIST_APP_LIST_SHOWER_DELEGATE_FACTORY_H_