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

Side by Side Diff: apps/shell/browser/shell_extension_system.cc

Issue 298883006: Rename NOTIFICATION_EXTENSION_INSTALLED to (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add comment Created 6 years, 7 months 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
« no previous file with comments | « apps/app_window.cc ('k') | chrome/browser/apps/app_browsertest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "apps/shell/browser/shell_extension_system.h" 5 #include "apps/shell/browser/shell_extension_system.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "apps/shell/browser/api/shell/shell_api.h" 9 #include "apps/shell/browser/api/shell/shell_api.h"
10 #include "base/file_util.h" 10 #include "base/file_util.h"
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
51 << " failed with: " << load_error; 51 << " failed with: " << load_error;
52 return false; 52 return false;
53 } 53 }
54 app_id_ = extension->id(); 54 app_id_ = extension->id();
55 55
56 // TODO(jamescook): We may want to do some of these things here: 56 // TODO(jamescook): We may want to do some of these things here:
57 // * Create a PermissionsUpdater. 57 // * Create a PermissionsUpdater.
58 // * Call PermissionsUpdater::GrantActivePermissions(). 58 // * Call PermissionsUpdater::GrantActivePermissions().
59 // * Call ExtensionService::SatisfyImports(). 59 // * Call ExtensionService::SatisfyImports().
60 // * Call ExtensionPrefs::OnExtensionInstalled(). 60 // * Call ExtensionPrefs::OnExtensionInstalled().
61 // * Send NOTIFICATION_EXTENSION_INSTALLED. 61 // * Send NOTIFICATION_EXTENSION_INSTALLED_DEPRECATED.
62 62
63 ExtensionRegistry::Get(browser_context_)->AddEnabled(extension); 63 ExtensionRegistry::Get(browser_context_)->AddEnabled(extension);
64 64
65 RegisterExtensionWithRequestContexts(extension); 65 RegisterExtensionWithRequestContexts(extension);
66 66
67 content::NotificationService::current()->Notify( 67 content::NotificationService::current()->Notify(
68 chrome::NOTIFICATION_EXTENSION_LOADED_DEPRECATED, 68 chrome::NOTIFICATION_EXTENSION_LOADED_DEPRECATED,
69 content::Source<BrowserContext>(browser_context_), 69 content::Source<BrowserContext>(browser_context_),
70 content::Details<const Extension>(extension)); 70 content::Details<const Extension>(extension));
71 71
(...skipping 102 matching lines...) Expand 10 before | Expand all | Expand 10 after
174 174
175 const OneShotEvent& ShellExtensionSystem::ready() const { 175 const OneShotEvent& ShellExtensionSystem::ready() const {
176 return ready_; 176 return ready_;
177 } 177 }
178 178
179 ContentVerifier* ShellExtensionSystem::content_verifier() { 179 ContentVerifier* ShellExtensionSystem::content_verifier() {
180 return NULL; 180 return NULL;
181 } 181 }
182 182
183 } // namespace extensions 183 } // namespace extensions
OLDNEW
« no previous file with comments | « apps/app_window.cc ('k') | chrome/browser/apps/app_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698