app_list: Drive app integration.

chrome/browser/apps/drive/drive_app_provider.h

+// Copyright 2014 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 CHROME_BROWSER_APPS_DRIVE_DRIVE_APP_PROVIDER_H_
+#define CHROME_BROWSER_APPS_DRIVE_DRIVE_APP_PROVIDER_H_
+
+#include <set>
+#include <string>
+#include <vector>
+
+#include "base/macros.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/memory/scoped_vector.h"
+#include "base/memory/weak_ptr.h"
+#include "chrome/browser/drive/drive_app_registry_observer.h"
+#include "chrome/browser/extensions/install_observer.h"
+
+namespace app_list {
+class AppListModel;
+}
+
+namespace drive {
+struct DriveAppInfo;
+}
+
+class BrowserContextKeyedServiceFactory;
+class ExtensionService;
+class Profile;
+
+namespace extensions {
+
+class DriveAppConverter;
+class DriveAppMapping;
+class DriveServiceBridge;
+
+// DriveAppProvider is the integration point for Drive apps. It ensures each
+// Drive app has a corresponding Chrome app in the extension system. If there
+// is no matching Chrome app, a local URL app would be created. The class
+// processes app changes from both DriveAppRegistry and extension system to
+// keep the two in sync.
+class DriveAppProvider : public drive::DriveAppRegistryObserver,
+                         public InstallObserver {
+ public:
+  DriveAppProvider(Profile* profile, app_list::AppListModel* model);
+  virtual ~DriveAppProvider();
+
+  // Appends PKS factories this class depends on.
+  static void AppendDependsOnFactories(
+      std::set<BrowserContextKeyedServiceFactory*>* factories);
+
+  void SetDriveServiceBridgeForTest(scoped_ptr<DriveServiceBridge> test_bridge);
+
+ private:
+  friend class DriveAppProviderTest;
+
+  typedef std::set<std::string> IdSet;
+  typedef std::vector<drive::DriveAppInfo> DriveAppInfos;
+
+  // Migrate launcher model data of |old_chrome_app_id| to |new_chrome_app_id|.

[benwells, 2014/06/05 01:24:39]

Can you explain under which circumstances this can happen?

[xiyuan, 2014/06/05 17:48:32]

Updated the comment to explain when this is called.

+  void MigrateModelSettings(const std::string& drive_app_id,
+                            const std::string& old_chrome_app_id,
+                            const std::string& new_chrome_app_id);
+
+  // Updates drive app id to chrome app mapping with the preference
+  // to keep chrome app over URL app and new app over existing app.

[benwells, 2014/06/05 01:24:39]

I don't understand this comment, specifically the bits about keeping one over
another. Is new_app the chrome / url app?

Maybe also adding when this happens would make it easier to understand.

[xiyuan, 2014/06/05 17:48:32]

Update the comments to be more specific.

+  void UpdateMappingAndExtensionSystem(const std::string& drive_app_id,
+                                       const Extension* new_app);
+
+  // Deferred processing of relevant extension installed message.
+  void ProcessDeferredOnExtensionInstalled(const std::string drive_app_id,
+                                           const std::string chrome_app_id);
+
+  void SchedulePendingConverters();
+  void OnLocalAppConverted(const DriveAppConverter* converter, bool success);
+
+  bool IsDriveAppUpToDate(const drive::DriveAppInfo& drive_app) const;
+
+  void AddOrUpdateDriveApp(const drive::DriveAppInfo& drive_app);
+  void RemoveDriveApp(const std::string& drive_app_id);
+
+  // drive::DriveAppRegistryObserver overrides:
+  virtual void OnDriveAppRegistryUpdated() OVERRIDE;
+
+  // InstallObserver overrides:
+  virtual void OnExtensionInstalled(const Extension* extension) OVERRIDE;
+  virtual void OnExtensionUninstalled(const Extension* extension) OVERRIDE;
+
+  Profile* profile_;
+  app_list::AppListModel* model_;  // Not owned. Must out live this class.
+
+  scoped_ptr<DriveServiceBridge> service_bridge_;
+  scoped_ptr<DriveAppMapping> mapping_;
+  DriveAppInfos drive_apps_;
+
+  // Tracks the pending web app convertions.
+  ScopedVector<DriveAppConverter> pending_converters_;
+
+  base::WeakPtrFactory<DriveAppProvider> weak_ptr_factory_;
+
+  DISALLOW_COPY_AND_ASSIGN(DriveAppProvider);
+};
+
+}  // namespace extensions
+
+#endif  // CHROME_BROWSER_APPS_DRIVE_DRIVE_APP_PROVIDER_H_