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

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

Issue 2969163002: Remove AppBannerManager::event_request_id(). (Closed)
Patch Set: Refactor into method Created 3 years, 5 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"
(...skipping 168 matching lines...) Expand 10 before | Expand all | Expand 10 after
179 void AddToHomescreenManager::CreateInfoBarForWebApk( 179 void AddToHomescreenManager::CreateInfoBarForWebApk(
180 const ShortcutInfo& info, 180 const ShortcutInfo& info,
181 const SkBitmap& primary_icon, 181 const SkBitmap& primary_icon,
182 const SkBitmap& badge_icon) { 182 const SkBitmap& badge_icon) {
183 content::WebContents* web_contents = data_fetcher_->web_contents(); 183 content::WebContents* web_contents = data_fetcher_->web_contents();
184 banners::AppBannerManagerAndroid* app_banner_manager = 184 banners::AppBannerManagerAndroid* app_banner_manager =
185 banners::AppBannerManagerAndroid::FromWebContents(web_contents); 185 banners::AppBannerManagerAndroid::FromWebContents(web_contents);
186 banners::AppBannerInfoBarDelegateAndroid::Create( 186 banners::AppBannerInfoBarDelegateAndroid::Create(
187 web_contents, app_banner_manager->GetWeakPtr(), 187 web_contents, app_banner_manager->GetWeakPtr(),
188 base::MakeUnique<ShortcutInfo>(info), primary_icon, badge_icon, 188 base::MakeUnique<ShortcutInfo>(info), primary_icon, badge_icon,
189 -1 /* event_request_id */, true /* is_webapk */, 189 true /* is_webapk */, webapk::INSTALL_SOURCE_MENU);
190 webapk::INSTALL_SOURCE_MENU);
191 } 190 }
192 191
193 SkBitmap AddToHomescreenManager::FinalizeLauncherIconInBackground( 192 SkBitmap AddToHomescreenManager::FinalizeLauncherIconInBackground(
194 const SkBitmap& bitmap, 193 const SkBitmap& bitmap,
195 const GURL& url, 194 const GURL& url,
196 bool* is_generated) { 195 bool* is_generated) {
197 base::ThreadRestrictions::AssertIOAllowed(); 196 base::ThreadRestrictions::AssertIOAllowed();
198 197
199 return ShortcutHelper::FinalizeLauncherIconInBackground(bitmap, url, 198 return ShortcutHelper::FinalizeLauncherIconInBackground(bitmap, url,
200 is_generated); 199 is_generated);
201 } 200 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698