Chromium Code Reviews| Index: chrome/browser/extensions/app_shortcut_manager.cc |
| diff --git a/chrome/browser/extensions/app_shortcut_manager.cc b/chrome/browser/extensions/app_shortcut_manager.cc |
| index ea467f0923ea43ee2131433e8c908a3794f0c82e..687c71687cb80f289eacf34d62cd6d2f74f0ac51 100644 |
| --- a/chrome/browser/extensions/app_shortcut_manager.cc |
| +++ b/chrome/browser/extensions/app_shortcut_manager.cc |
| @@ -63,9 +63,9 @@ AppShortcutManager::~AppShortcutManager() {} |
| void AppShortcutManager::Observe(int type, |
| const content::NotificationSource& source, |
| const content::NotificationDetails& details) { |
| -#if !defined(OS_MACOSX) |
| switch (type) { |
| case chrome::NOTIFICATION_EXTENSION_INSTALLED: { |
| +#if !defined(OS_MACOSX) |
| const Extension* extension = content::Details<const Extension>( |
| details).ptr(); |
| if (extension->is_platform_app() && |
| @@ -85,6 +85,7 @@ void AppShortcutManager::Observe(int type, |
| UpdateApplicationShortcuts(extension); |
| #endif |
|
Avi (use Gerrit)
2013/02/05 00:19:25
Can you label some of these? It's hard to keep tra
jeremya
2013/02/05 02:38:14
Done.
|
| } |
| +#endif |
|
Avi (use Gerrit)
2013/02/05 00:19:25
...and here.
jeremya
2013/02/05 02:38:14
Done.
|
| break; |
| } |
| case chrome::NOTIFICATION_EXTENSION_UNINSTALLED: { |
| @@ -96,7 +97,6 @@ void AppShortcutManager::Observe(int type, |
| default: |
| NOTREACHED(); |
| } |
| -#endif |
| } |
| #if defined(OS_WIN) |