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

Unified Diff: chrome/browser/installable/installable_manager_unittest.cc

Issue 2641413002: Allow InstallableManager to fetch optional badge icon (Closed)
Patch Set: Fix rebase 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/installable/installable_manager_unittest.cc
diff --git a/chrome/browser/installable/installable_manager_unittest.cc b/chrome/browser/installable/installable_manager_unittest.cc
index b7ec00c5f8e265a1e2ecf2cd73c05b0970248df5..cf934a488688a81eb615a1250362a24590248304 100644
--- a/chrome/browser/installable/installable_manager_unittest.cc
+++ b/chrome/browser/installable/installable_manager_unittest.cc
@@ -26,12 +26,14 @@ class InstallableManagerUnitTest : public testing::Test {
manifest.start_url = GURL("http://example.com");
manifest.display = blink::WebDisplayModeStandalone;
- content::Manifest::Icon icon;
- icon.type = base::ASCIIToUTF16("image/png");
- icon.sizes.push_back(gfx::Size(144, 144));
- icon.purpose.push_back(IconPurpose::ANY);
- manifest.icons.push_back(icon);
-
+ content::Manifest::Icon primary_icon;
+ primary_icon.type = base::ASCIIToUTF16("image/png");
+ primary_icon.sizes.push_back(gfx::Size(144, 144));
+ primary_icon.purpose.push_back(IconPurpose::ANY);
+ manifest.icons.push_back(primary_icon);
+
+ // No need to include the optional badge icon as it does not affect the
+ // unit tests.
return manifest;
}
« no previous file with comments | « chrome/browser/installable/installable_manager_browsertest.cc ('k') | chrome/test/data/banners/manifest.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698