Add generateAppForLink function in chrome.management

chrome/browser/extensions/api/management/management_api.h

 // Copyright (c) 2012 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_EXTENSIONS_API_MANAGEMENT_MANAGEMENT_API_H_
 #define CHROME_BROWSER_EXTENSIONS_API_MANAGEMENT_MANAGEMENT_API_H_
 
 #include "base/compiler_specific.h"
+#include "base/task/cancelable_task_tracker.h"
+#include "chrome/browser/extensions/bookmark_app_helper.h"
 #include "chrome/browser/extensions/chrome_extension_function.h"
 #include "chrome/browser/extensions/extension_install_prompt.h"
 #include "chrome/browser/extensions/extension_uninstall_dialog.h"
+#include "components/favicon_base/favicon_types.h"
 #include "components/keyed_service/core/keyed_service.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "extensions/browser/browser_context_keyed_api_factory.h"
 #include "extensions/browser/event_router.h"
 
 class ExtensionService;
 class ExtensionUninstallDialog;
+class WebApplicationInfo;
 
 namespace extensions {
 
 class ManagementFunction : public ChromeSyncExtensionFunction {
  protected:
   virtual ~ManagementFunction() {}
 
   ExtensionService* service();
 };
 
@@ skipping 152 matching lines @@
   void OnCloseShortcutPrompt(bool created);
 
   static void SetAutoConfirmForTest(bool should_proceed);
 
  protected:
   virtual ~ManagementCreateAppShortcutFunction();
 
   virtual bool RunAsync() OVERRIDE;
 };
 
+class ManagementGenerateAppForLinkFunction : public AsyncManagementFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION("management.generateAppForLink",
+      MANAGEMENT_GENERATEAPPFORLINK);
+
+  ManagementGenerateAppForLinkFunction();
+
+ protected:
+  virtual ~ManagementGenerateAppForLinkFunction();
+
+  virtual bool RunAsync() OVERRIDE;
+
+ private:
+  struct AppInstallInfo {

[benwells, 2014/05/27 22:05:54]

Why do you need this struct? Seems like having a title_ and launch_url_ field is
equivalent to the way it is now.

You could also bind these into the OnFaviconForApp callback but that is probably
unnecessary.

[wjywbs, 2014/05/27 22:20:21]

Done.

+    AppInstallInfo() {}
+    ~AppInstallInfo() {}
+
+    std::string title;
+    GURL launch_url;
+  };
+
+  void OnFaviconForApp(const favicon_base::FaviconImageResult& image_result);
+
+  void FinishCreateBookmarkApp(const extensions::Extension* extension,
+                               const WebApplicationInfo& web_app_info);
+
+  scoped_ptr<AppInstallInfo> install_info_;
+
+  scoped_ptr<BookmarkAppHelper> bookmark_app_helper_;
+
+  // Used for favicon loading tasks.
+  base::CancelableTaskTracker cancelable_task_tracker_;
+};
+
 class ManagementEventRouter : public content::NotificationObserver {
  public:
   explicit ManagementEventRouter(Profile* profile);
   virtual ~ManagementEventRouter();
 
  private:
   // content::NotificationObserver implementation.
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
@@ skipping 34 matching lines @@
 
   // Created lazily upon OnListenerAdded.
   scoped_ptr<ManagementEventRouter> management_event_router_;
 
   DISALLOW_COPY_AND_ASSIGN(ManagementAPI);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_MANAGEMENT_MANAGEMENT_API_H_