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

Unified Diff: chrome/browser/ui/web_applications/web_app_ui.cc

Issue 14241002: Move the parsing of app.launch related keys out of Extension class. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 8 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
Index: chrome/browser/ui/web_applications/web_app_ui.cc
diff --git a/chrome/browser/ui/web_applications/web_app_ui.cc b/chrome/browser/ui/web_applications/web_app_ui.cc
index 418e26f57b2175d07e49c7bdc861fcc50a889fc6..2b8317069d0b52cf1c037e3d12c56c90ba60271c 100644
--- a/chrome/browser/ui/web_applications/web_app_ui.cc
+++ b/chrome/browser/ui/web_applications/web_app_ui.cc
@@ -17,6 +17,7 @@
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/web_applications/web_app.h"
#include "chrome/common/chrome_notification_types.h"
+#include "chrome/common/extensions/app_launcher_info.h"
#include "chrome/common/extensions/extension.h"
#include "chrome/common/extensions/manifest_handlers/icons_handler.h"
#include "content/public/browser/browser_thread.h"
@@ -408,7 +409,7 @@ void UpdateShortcutInfoForApp(const extensions::Extension& app,
ShellIntegration::ShortcutInfo* shortcut_info) {
shortcut_info->extension_id = app.id();
shortcut_info->is_platform_app = app.is_platform_app();
- shortcut_info->url = GURL(app.launch_web_url());
+ shortcut_info->url = GURL(extensions::AppLauncherInfo::GetLaunchWebURL(&app));
shortcut_info->title = UTF8ToUTF16(app.name());
shortcut_info->description = UTF8ToUTF16(app.description());
shortcut_info->extension_path = app.path();

Powered by Google App Engine
This is Rietveld 408576698