Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1373)

Side by Side Diff: athena/extensions/shell/extensions_delegate_impl.cc

Issue 641683003: C++11 override style change for athena (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "athena/extensions/public/extensions_delegate.h" 5 #include "athena/extensions/public/extensions_delegate.h"
6 6
7 #include "athena/extensions/shell/athena_shell_app_window_client.h" 7 #include "athena/extensions/shell/athena_shell_app_window_client.h"
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "extensions/browser/install/extension_install_ui.h" 9 #include "extensions/browser/install/extension_install_ui.h"
10 #include "extensions/common/extension_set.h" 10 #include "extensions/common/extension_set.h"
11 #include "extensions/shell/browser/shell_extension_system.h" 11 #include "extensions/shell/browser/shell_extension_system.h"
12 12
13 namespace athena { 13 namespace athena {
14 namespace { 14 namespace {
15 15
16 class ShellExtensionsDelegate : public ExtensionsDelegate { 16 class ShellExtensionsDelegate : public ExtensionsDelegate {
17 public: 17 public:
18 explicit ShellExtensionsDelegate(content::BrowserContext* context) 18 explicit ShellExtensionsDelegate(content::BrowserContext* context)
19 : context_(context), 19 : context_(context),
20 extension_system_(static_cast<extensions::ShellExtensionSystem*>( 20 extension_system_(static_cast<extensions::ShellExtensionSystem*>(
21 extensions::ExtensionSystem::Get(context))) { 21 extensions::ExtensionSystem::Get(context))) {
22 extensions::AppWindowClient::Set(&app_window_client_); 22 extensions::AppWindowClient::Set(&app_window_client_);
23 } 23 }
24 24
25 virtual ~ShellExtensionsDelegate() { extensions::AppWindowClient::Set(NULL); } 25 ~ShellExtensionsDelegate() override { extensions::AppWindowClient::Set(NULL); }
26 26
27 private: 27 private:
28 // ExtensionsDelegate: 28 // ExtensionsDelegate:
29 virtual content::BrowserContext* GetBrowserContext() const override { 29 content::BrowserContext* GetBrowserContext() const override {
30 return context_; 30 return context_;
31 } 31 }
32 virtual const extensions::ExtensionSet& GetInstalledExtensions() override { 32 const extensions::ExtensionSet& GetInstalledExtensions() override {
33 shell_extensions_.Clear(); 33 shell_extensions_.Clear();
34 if (extension_system_->extension().get()) 34 if (extension_system_->extension().get())
35 shell_extensions_.Insert(extension_system_->extension()); 35 shell_extensions_.Insert(extension_system_->extension());
36 return shell_extensions_; 36 return shell_extensions_;
37 } 37 }
38 virtual bool LaunchApp(const std::string& app_id) override { 38 bool LaunchApp(const std::string& app_id) override {
39 extension_system_->LaunchApp(); 39 extension_system_->LaunchApp();
40 return true; 40 return true;
41 } 41 }
42 42
43 virtual bool UnloadApp(const std::string& app_id) override { return false; } 43 bool UnloadApp(const std::string& app_id) override { return false; }
44 44
45 virtual scoped_ptr<extensions::ExtensionInstallUI> CreateExtensionInstallUI() 45 scoped_ptr<extensions::ExtensionInstallUI> CreateExtensionInstallUI()
46 override { 46 override {
47 return scoped_ptr<extensions::ExtensionInstallUI>(); 47 return scoped_ptr<extensions::ExtensionInstallUI>();
48 } 48 }
49 49
50 content::BrowserContext* context_; 50 content::BrowserContext* context_;
51 extensions::ShellExtensionSystem* extension_system_; 51 extensions::ShellExtensionSystem* extension_system_;
52 extensions::ExtensionSet shell_extensions_; 52 extensions::ExtensionSet shell_extensions_;
53 53
54 AthenaShellAppWindowClient app_window_client_; 54 AthenaShellAppWindowClient app_window_client_;
55 55
56 DISALLOW_COPY_AND_ASSIGN(ShellExtensionsDelegate); 56 DISALLOW_COPY_AND_ASSIGN(ShellExtensionsDelegate);
57 }; 57 };
58 58
59 } // namespace 59 } // namespace
60 60
61 // static 61 // static
62 void ExtensionsDelegate::CreateExtensionsDelegate( 62 void ExtensionsDelegate::CreateExtensionsDelegate(
63 content::BrowserContext* context) { 63 content::BrowserContext* context) {
64 new ShellExtensionsDelegate(context); 64 new ShellExtensionsDelegate(context);
65 } 65 }
66 66
67 } // namespace athena 67 } // namespace athena
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698