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

Unified Diff: chrome/browser/ui/webui/ntp/app_launcher_handler.cc

Issue 142993002: Remove unused members from WebApplicationInfo. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 11 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/webui/ntp/app_launcher_handler.cc
diff --git a/chrome/browser/ui/webui/ntp/app_launcher_handler.cc b/chrome/browser/ui/webui/ntp/app_launcher_handler.cc
index cada3231b3c5faf1be3471f0ead6a114f6b130b4..8f7eb84bc153a63ad71278786353ec8ec96b751d 100644
--- a/chrome/browser/ui/webui/ntp/app_launcher_handler.cc
+++ b/chrome/browser/ui/webui/ntp/app_launcher_handler.cc
@@ -701,7 +701,6 @@ void AppLauncherHandler::HandleGenerateAppForLink(const base::ListValue* args) {
}
scoped_ptr<AppInstallInfo> install_info(new AppInstallInfo());
- install_info->is_bookmark_app = true;
install_info->title = title;
install_info->app_url = launch_url;
install_info->page_ordinal = page_ordinal;
@@ -733,10 +732,8 @@ void AppLauncherHandler::OnFaviconForApp(
scoped_ptr<AppInstallInfo> install_info,
const chrome::FaviconImageResult& image_result) {
scoped_ptr<WebApplicationInfo> web_app(new WebApplicationInfo());
- web_app->is_bookmark_app = install_info->is_bookmark_app;
web_app->title = install_info->title;
web_app->app_url = install_info->app_url;
- web_app->urls.push_back(install_info->app_url);
if (!image_result.image.IsEmpty()) {
WebApplicationInfo::IconInfo icon;

Powered by Google App Engine
This is Rietveld 408576698