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

Side by Side Diff: chrome/browser/android/webapps/add_to_homescreen_manager.cc

Issue 2670963002: Android: Fire the appinstalled event on WebAPK installation. (Closed)
Patch Set: Respond to review. Created 3 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
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 "chrome/browser/android/webapps/add_to_homescreen_manager.h" 5 #include "chrome/browser/android/webapps/add_to_homescreen_manager.h"
6 6
7 #include "base/android/jni_android.h" 7 #include "base/android/jni_android.h"
8 #include "base/android/jni_string.h" 8 #include "base/android/jni_string.h"
9 #include "base/location.h" 9 #include "base/location.h"
10 #include "base/memory/ptr_util.h" 10 #include "base/memory/ptr_util.h"
11 #include "base/strings/string16.h" 11 #include "base/strings/string16.h"
12 #include "base/strings/utf_string_conversions.h" 12 #include "base/strings/utf_string_conversions.h"
13 #include "base/threading/sequenced_worker_pool.h" 13 #include "base/threading/sequenced_worker_pool.h"
14 #include "chrome/browser/android/banners/app_banner_infobar_delegate_android.h" 14 #include "chrome/browser/android/banners/app_banner_infobar_delegate_android.h"
15 #include "chrome/browser/android/banners/app_banner_manager_android.h"
15 #include "chrome/browser/android/shortcut_helper.h" 16 #include "chrome/browser/android/shortcut_helper.h"
16 #include "chrome/browser/android/webapk/chrome_webapk_host.h" 17 #include "chrome/browser/android/webapk/chrome_webapk_host.h"
17 #include "chrome/browser/android/webapk/webapk_install_service.h" 18 #include "chrome/browser/android/webapk/webapk_install_service.h"
18 #include "chrome/browser/android/webapk/webapk_metrics.h" 19 #include "chrome/browser/android/webapk/webapk_metrics.h"
19 #include "chrome/browser/banners/app_banner_settings_helper.h" 20 #include "chrome/browser/banners/app_banner_settings_helper.h"
20 #include "chrome/browser/installable/installable_manager.h" 21 #include "chrome/browser/installable/installable_manager.h"
21 #include "content/public/browser/browser_thread.h" 22 #include "content/public/browser/browser_thread.h"
22 #include "content/public/browser/render_frame_host.h" 23 #include "content/public/browser/render_frame_host.h"
23 #include "content/public/browser/web_contents.h" 24 #include "content/public/browser/web_contents.h"
24 #include "jni/AddToHomescreenManager_jni.h" 25 #include "jni/AddToHomescreenManager_jni.h"
(...skipping 87 matching lines...) Expand 10 before | Expand all | Expand 10 after
112 add_shortcut_pending_ = false; 113 add_shortcut_pending_ = false;
113 114
114 content::WebContents* web_contents = data_fetcher_->web_contents(); 115 content::WebContents* web_contents = data_fetcher_->web_contents();
115 if (!web_contents) 116 if (!web_contents)
116 return; 117 return;
117 118
118 RecordAddToHomescreen(); 119 RecordAddToHomescreen();
119 ShortcutHelper::AddToLauncherWithSkBitmap(web_contents, info, icon); 120 ShortcutHelper::AddToLauncherWithSkBitmap(web_contents, info, icon);
120 121
121 // Fire the appinstalled event. 122 // Fire the appinstalled event.
122 blink::mojom::InstallationServicePtr installation_service; 123 banners::AppBannerManagerAndroid* app_banner_manager =
123 web_contents->GetMainFrame()->GetRemoteInterfaces()->GetInterface( 124 banners::AppBannerManagerAndroid::FromWebContents(web_contents);
124 mojo::MakeRequest(&installation_service)); 125 app_banner_manager->OnInstall();
125 DCHECK(installation_service);
126 installation_service->OnInstall();
127 } 126 }
128 127
129 void AddToHomescreenManager::RecordAddToHomescreen() { 128 void AddToHomescreenManager::RecordAddToHomescreen() {
130 // Record that the shortcut has been added, so no banners will be shown 129 // Record that the shortcut has been added, so no banners will be shown
131 // for this app. 130 // for this app.
132 content::WebContents* web_contents = data_fetcher_->web_contents(); 131 content::WebContents* web_contents = data_fetcher_->web_contents();
133 if (!web_contents) 132 if (!web_contents)
134 return; 133 return;
135 134
136 AppBannerSettingsHelper::RecordBannerEvent( 135 AppBannerSettingsHelper::RecordBannerEvent(
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
184 Java_AddToHomescreenManager_onReadyToAdd(env, java_ref_, java_bitmap); 183 Java_AddToHomescreenManager_onReadyToAdd(env, java_ref_, java_bitmap);
185 184
186 if (add_shortcut_pending_) 185 if (add_shortcut_pending_)
187 AddShortcut(info, primary_icon); 186 AddShortcut(info, primary_icon);
188 } 187 }
189 188
190 void AddToHomescreenManager::CreateInfoBarForWebApk( 189 void AddToHomescreenManager::CreateInfoBarForWebApk(
191 const ShortcutInfo& info, 190 const ShortcutInfo& info,
192 const SkBitmap& primary_icon, 191 const SkBitmap& primary_icon,
193 const SkBitmap& badge_icon) { 192 const SkBitmap& badge_icon) {
193 content::WebContents* web_contents = data_fetcher_->web_contents();
194 banners::AppBannerManagerAndroid* app_banner_manager =
195 banners::AppBannerManagerAndroid::FromWebContents(web_contents);
194 banners::AppBannerInfoBarDelegateAndroid::Create( 196 banners::AppBannerInfoBarDelegateAndroid::Create(
195 data_fetcher_->web_contents(), nullptr, info.user_title, 197 web_contents, app_banner_manager->GetWeakPtr(), info.user_title,
196 base::MakeUnique<ShortcutInfo>(info), primary_icon, badge_icon, 198 base::MakeUnique<ShortcutInfo>(info), primary_icon, badge_icon,
197 -1 /* event_request_id */, webapk::INSTALL_SOURCE_MENU); 199 -1 /* event_request_id */, webapk::INSTALL_SOURCE_MENU);
198 } 200 }
199 201
200 SkBitmap AddToHomescreenManager::FinalizeLauncherIconInBackground( 202 SkBitmap AddToHomescreenManager::FinalizeLauncherIconInBackground(
201 const SkBitmap& bitmap, 203 const SkBitmap& bitmap,
202 const GURL& url, 204 const GURL& url,
203 bool* is_generated) { 205 bool* is_generated) {
204 DCHECK(content::BrowserThread::GetBlockingPool()->RunsTasksOnCurrentThread()); 206 DCHECK(content::BrowserThread::GetBlockingPool()->RunsTasksOnCurrentThread());
205 207
206 return ShortcutHelper::FinalizeLauncherIconInBackground(bitmap, url, 208 return ShortcutHelper::FinalizeLauncherIconInBackground(bitmap, url,
207 is_generated); 209 is_generated);
208 } 210 }
OLDNEW
« no previous file with comments | « chrome/browser/android/banners/app_banner_infobar_delegate_android.cc ('k') | chrome/browser/banners/app_banner_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698