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

Unified Diff: chrome/browser/android/webapps/add_to_homescreen_data_fetcher_unittest.cc

Issue 2669133003: Update AddToHomescreenDataFetcher to accept badge icon for WebAPK. (Closed)
Patch Set: Addressing comments Created 3 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/android/webapps/add_to_homescreen_data_fetcher_unittest.cc
diff --git a/chrome/browser/android/webapps/add_to_homescreen_data_fetcher_unittest.cc b/chrome/browser/android/webapps/add_to_homescreen_data_fetcher_unittest.cc
index b471f19673450a50c779a81553602e46aadbb7b0..c8bfaf1bf4f3ad4f56063c92402ec05061852373 100644
--- a/chrome/browser/android/webapps/add_to_homescreen_data_fetcher_unittest.cc
+++ b/chrome/browser/android/webapps/add_to_homescreen_data_fetcher_unittest.cc
@@ -131,7 +131,8 @@ class ObserverWaiter : public AddToHomescreenDataFetcher::Observer {
}
void OnDataAvailable(const ShortcutInfo& info,
- const SkBitmap& icon) override {
+ const SkBitmap& primary_icon,
+ const SkBitmap& badge_icon) override {
data_available_ = true;
if (!quit_closure_.is_null())
quit_closure_.Run();
@@ -203,7 +204,7 @@ class AddToHomescreenDataFetcherTest : public ChromeRenderViewHostTestHarness {
scoped_refptr<AddToHomescreenDataFetcher> BuildFetcher(
bool check_webapk_compatible,
AddToHomescreenDataFetcher::Observer* observer) {
- return new AddToHomescreenDataFetcher(web_contents(), 1, 1, 1, 1,
+ return new AddToHomescreenDataFetcher(web_contents(), 1, 1, 1, 1, 1,
check_webapk_compatible, observer);
}

Powered by Google App Engine
This is Rietveld 408576698