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/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;
 
 namespace extensions {
@@ skipping 160 matching lines @@
   void OnCloseShortcutPrompt(bool created);
 
   static void SetAutoConfirmForTest(bool should_proceed);
 
  protected:
   virtual ~ManagementCreateAppShortcutFunction();
 
   virtual bool RunAsync() OVERRIDE;
 };
 
+class ManagementGenerateAppForLinkFunction : public AsyncManagementFunction,
+                                         public content::NotificationObserver {

[benwells, 2014/05/22 02:49:25]

This indentation looks wrong. Did you run git cl format?

[wjywbs, 2014/05/22 03:21:35]

Done.

+ public:
+  DECLARE_EXTENSION_FUNCTION("management.generateAppForLink",
+      MANAGEMENT_GENERATEAPPFORLINK);
+
+  ManagementGenerateAppForLinkFunction();
+
+ protected:
+  virtual ~ManagementGenerateAppForLinkFunction();
+
+  virtual bool RunAsync() OVERRIDE;
+
+ private:
+  struct AppInstallInfo {
+    AppInstallInfo() {}
+    ~AppInstallInfo() {}
+
+    std::string title;
+    GURL launch_url;
+  };
+
+  void OnFaviconForApp(const favicon_base::FaviconImageResult& image_result);
+
+  scoped_ptr<AppInstallInfo> install_info_;
+
+  // Used for favicon loading tasks.
+  base::CancelableTaskTracker cancelable_task_tracker_;
+
+  // Overridden from content::NotificationObserver:

[benwells, 2014/05/22 02:49:25]

Methods should go before fields. Put this up after the struct.

[wjywbs, 2014/05/22 03:21:35]

Done.

+  virtual void Observe(int type,
+                       const content::NotificationSource& source,
+                       const content::NotificationDetails& details) OVERRIDE;
+
+  content::NotificationRegistrar registrar_;
+};
+
 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_