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

Unified Diff: apps/shortcut_manager.cc

Issue 12378037: Move apps_shortcut_manager into src/apps (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 7 years, 9 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « apps/shortcut_manager.h ('k') | apps/shortcut_manager_factory.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: apps/shortcut_manager.cc
diff --git a/chrome/browser/extensions/app_shortcut_manager.cc b/apps/shortcut_manager.cc
similarity index 71%
rename from chrome/browser/extensions/app_shortcut_manager.cc
rename to apps/shortcut_manager.cc
index d1c5a4ff29edcfa993dc8222a1f2d93abbf057fb..e3fb8e6995ecefa0b8d3b3bb1e8a8c1093e938bc 100644
--- a/chrome/browser/extensions/app_shortcut_manager.cc
+++ b/apps/shortcut_manager.cc
@@ -2,30 +2,22 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "chrome/browser/extensions/app_shortcut_manager.h"
+#include "apps/shortcut_manager.h"
#include "base/bind.h"
-#include "base/command_line.h"
#include "base/compiler_specific.h"
-#include "base/logging.h"
-#include "base/utf_string_conversions.h"
-#include "chrome/browser/extensions/image_loader.h"
-#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/shell_integration.h"
#include "chrome/browser/ui/web_applications/web_app_ui.h"
#include "chrome/browser/web_applications/web_app.h"
#include "chrome/common/chrome_notification_types.h"
-#include "chrome/common/chrome_switches.h"
-#include "chrome/common/extensions/extension_resource.h"
#include "content/public/browser/notification_details.h"
#include "content/public/browser/notification_source.h"
-#include "grit/theme_resources.h"
-#include "skia/ext/image_operations.h"
-#include "ui/base/resource/resource_bundle.h"
-namespace extensions {
+using extensions::Extension;
-AppShortcutManager::AppShortcutManager(Profile* profile)
+namespace apps {
+
+ShortcutManager::ShortcutManager(Profile* profile)
: profile_(profile),
ALLOW_THIS_IN_INITIALIZER_LIST(weak_factory_(this)) {
registrar_.Add(this, chrome::NOTIFICATION_EXTENSION_INSTALLED,
@@ -34,9 +26,9 @@ AppShortcutManager::AppShortcutManager(Profile* profile)
content::Source<Profile>(profile_));
}
-AppShortcutManager::~AppShortcutManager() {}
+ShortcutManager::~ShortcutManager() {}
-void AppShortcutManager::Observe(int type,
+void ShortcutManager::Observe(int type,
const content::NotificationSource& source,
const content::NotificationDetails& details) {
switch (type) {
@@ -45,7 +37,7 @@ void AppShortcutManager::Observe(int type,
const Extension* extension = content::Details<const Extension>(
details).ptr();
if (extension->is_platform_app() &&
- extension->location() != Manifest::COMPONENT) {
+ extension->location() != extensions::Manifest::COMPONENT) {
web_app::UpdateShortcutInfoAndIconForApp(*extension, profile_,
base::Bind(&web_app::UpdateAllShortcuts));
}
@@ -63,11 +55,11 @@ void AppShortcutManager::Observe(int type,
}
}
-void AppShortcutManager::DeleteApplicationShortcuts(
+void ShortcutManager::DeleteApplicationShortcuts(
const Extension* extension) {
ShellIntegration::ShortcutInfo delete_info =
web_app::ShortcutInfoForExtensionAndProfile(extension, profile_);
web_app::DeleteAllShortcuts(delete_info);
}
-} // namespace extensions
+} // namespace apps
« no previous file with comments | « apps/shortcut_manager.h ('k') | apps/shortcut_manager_factory.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698