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

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

Issue 2379923002: Implement "appinstalled" event on Android. (Closed)
Patch Set: Minor refactor. Created 4 years, 2 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/guid.h" 9 #include "base/guid.h"
10 #include "base/location.h" 10 #include "base/location.h"
11 #include "base/memory/ptr_util.h" 11 #include "base/memory/ptr_util.h"
12 #include "base/strings/string16.h" 12 #include "base/strings/string16.h"
13 #include "base/strings/utf_string_conversions.h" 13 #include "base/strings/utf_string_conversions.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/shortcut_helper.h" 15 #include "chrome/browser/android/shortcut_helper.h"
16 #include "chrome/browser/android/webapk/chrome_webapk_host.h" 16 #include "chrome/browser/android/webapk/chrome_webapk_host.h"
17 #include "chrome/browser/android/webapk/webapk_metrics.h" 17 #include "chrome/browser/android/webapk/webapk_metrics.h"
18 #include "chrome/browser/banners/app_banner_settings_helper.h" 18 #include "chrome/browser/banners/app_banner_settings_helper.h"
19 #include "content/public/browser/browser_thread.h" 19 #include "content/public/browser/browser_thread.h"
20 #include "content/public/browser/render_frame_host.h"
20 #include "content/public/browser/web_contents.h" 21 #include "content/public/browser/web_contents.h"
21 #include "content/public/common/origin_util.h" 22 #include "content/public/common/origin_util.h"
22 #include "jni/AddToHomescreenManager_jni.h" 23 #include "jni/AddToHomescreenManager_jni.h"
24 #include "mojo/public/cpp/bindings/interface_request.h"
25 #include "services/shell/public/cpp/interface_provider.h"
23 #include "ui/gfx/android/java_bitmap.h" 26 #include "ui/gfx/android/java_bitmap.h"
24 27
25 using base::android::JavaParamRef; 28 using base::android::JavaParamRef;
26 using base::android::ScopedJavaLocalRef; 29 using base::android::ScopedJavaLocalRef;
27 30
28 jlong InitializeAndStart(JNIEnv* env, 31 jlong InitializeAndStart(JNIEnv* env,
29 const JavaParamRef<jobject>& obj, 32 const JavaParamRef<jobject>& obj,
30 const JavaParamRef<jobject>& java_web_contents) { 33 const JavaParamRef<jobject>& java_web_contents) {
31 content::WebContents* web_contents = 34 content::WebContents* web_contents =
32 content::WebContents::FromJavaWebContents(java_web_contents); 35 content::WebContents::FromJavaWebContents(java_web_contents);
(...skipping 75 matching lines...) Expand 10 before | Expand all | Expand 10 after
108 content::WebContents* web_contents = data_fetcher_->web_contents(); 111 content::WebContents* web_contents = data_fetcher_->web_contents();
109 if (!web_contents) 112 if (!web_contents)
110 return; 113 return;
111 114
112 RecordAddToHomescreen(); 115 RecordAddToHomescreen();
113 116
114 const std::string& uid = base::GenerateGUID(); 117 const std::string& uid = base::GenerateGUID();
115 ShortcutHelper::AddToLauncherWithSkBitmap( 118 ShortcutHelper::AddToLauncherWithSkBitmap(
116 web_contents->GetBrowserContext(), info, uid, icon, 119 web_contents->GetBrowserContext(), info, uid, icon,
117 data_fetcher_->FetchSplashScreenImageCallback(uid)); 120 data_fetcher_->FetchSplashScreenImageCallback(uid));
121
dominickn 2016/10/12 22:35:09 This doesn't account for WebAPKs, which call Short
Matt Giuca 2016/12/09 04:36:48 As with Desktop, let's leave WebAPKs for another C
122 // Fire the appinstalled event.
123 if (!installation_service_) {
124 web_contents->GetMainFrame()->GetRemoteInterfaces()->GetInterface(
125 mojo::GetProxy(&installation_service_));
126 DCHECK(installation_service_);
127 }
128 installation_service_->OnInstall();
118 } 129 }
119 130
120 void AddToHomescreenManager::RecordAddToHomescreen() { 131 void AddToHomescreenManager::RecordAddToHomescreen() {
121 // Record that the shortcut has been added, so no banners will be shown 132 // Record that the shortcut has been added, so no banners will be shown
122 // for this app. 133 // for this app.
123 content::WebContents* web_contents = data_fetcher_->web_contents(); 134 content::WebContents* web_contents = data_fetcher_->web_contents();
124 if (!web_contents) 135 if (!web_contents)
125 return; 136 return;
126 137
127 AppBannerSettingsHelper::RecordBannerEvent( 138 AppBannerSettingsHelper::RecordBannerEvent(
(...skipping 51 matching lines...) Expand 10 before | Expand all | Expand 10 after
179 190
180 SkBitmap AddToHomescreenManager::FinalizeLauncherIconInBackground( 191 SkBitmap AddToHomescreenManager::FinalizeLauncherIconInBackground(
181 const SkBitmap& bitmap, 192 const SkBitmap& bitmap,
182 const GURL& url, 193 const GURL& url,
183 bool* is_generated) { 194 bool* is_generated) {
184 DCHECK(content::BrowserThread::GetBlockingPool()->RunsTasksOnCurrentThread()); 195 DCHECK(content::BrowserThread::GetBlockingPool()->RunsTasksOnCurrentThread());
185 196
186 return ShortcutHelper::FinalizeLauncherIconInBackground(bitmap, url, 197 return ShortcutHelper::FinalizeLauncherIconInBackground(bitmap, url,
187 is_generated); 198 is_generated);
188 } 199 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698