ExtensionDelegate to abstract extension implementation

athena/extensions/public/extensions_delegate.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 ATHENA_EXTENSIONS_PUBLIC_EXTENSIONS_DELEGATE_H_
+#define ATHENA_EXTENSIONS_PUBLIC_EXTENSIONS_DELEGATE_H_
+
+#include <string>
+
+#include "athena/athena_export.h"
+
+namespace content {
+class BrowserContext;
+}
+
+namespace extensions {
+class ExtensionSet;
+}
+
+namespace athena {
+
+// A delegate interface to extension implentation.
+class ATHENA_EXPORT ExtensionsDelegate {
+ public:
+  static ExtensionsDelegate* Get(content::BrowserContext* context);
+
+  // Initializes the extension delegate for app shell environment.
+  static void InitExtensionsDelegateForShell(content::BrowserContext* context);
+
+  // Deletes the singleton instance. This must be called in the reverse
+  // order of the initialization.
+  static void Shutdown();
+
+  ExtensionsDelegate();
+  virtual ~ExtensionsDelegate();
+
+  virtual content::BrowserContext* GetBrowserContext() const = 0;
+
+  // Returns the set of extensions that are currently installed.

[Yoyo Zhou, 2014/08/21 22:40:29]

I find the ownership here a little unintuitive, so it's probably worth a comment
at least. Alternatively, you could create a new ExtensionSet and say that
ownership goes to the caller; I think that's somewhat better. That should be
commented too.

[oshima, 2014/08/21 23:22:57]

Changed to const ExtensionSet& I think that's better.

+  virtual extensions::ExtensionSet* GetInstalledExtensions() = 0;
+
+  // Launch an application specified by |app_id|.
+  virtual void LaunchApp(const std::string& app_id) = 0;
+};
+
+}  // namespace athena
+
+#endif  // ATHENA_EXTENSIONS_PUBLIC_EXTENSIONS_DELEGATE_H_