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

Unified Diff: content/browser/manifest/manifest_icon_selector_unittest.cc

Issue 2933743002: Move chrome/browser/manifest to content/browser. (Closed)
Patch Set: Move chrome/browser/manifest to content/browser. Created 3 years, 6 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: content/browser/manifest/manifest_icon_selector_unittest.cc
diff --git a/chrome/browser/manifest/manifest_icon_selector_unittest.cc b/content/browser/manifest/manifest_icon_selector_unittest.cc
similarity index 98%
rename from chrome/browser/manifest/manifest_icon_selector_unittest.cc
rename to content/browser/manifest/manifest_icon_selector_unittest.cc
index f819b766afd77d49a8901db2c6e1fe1317170e8b..06406f5373790961c1d4b0eff3072e5421b035a9 100644
--- a/chrome/browser/manifest/manifest_icon_selector_unittest.cc
+++ b/content/browser/manifest/manifest_icon_selector_unittest.cc
@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "chrome/browser/manifest/manifest_icon_selector.h"
+#include "content/public/browser/manifest_icon_selector.h"
#include <string>
#include <vector>
@@ -33,6 +33,8 @@ static content::Manifest::Icon CreateIcon(const std::string& url,
} // anonymous namespace
+namespace content {
+
TEST(ManifestIconSelector, NoIcons) {
// No icons should return the empty URL.
std::vector<content::Manifest::Icon> icons;
@@ -106,8 +108,8 @@ TEST(ManifestIconSelector, PurposeFiltering) {
sizes_144.push_back(gfx::Size(144, 144));
std::vector<content::Manifest::Icon> icons;
- icons.push_back(
- CreateIcon("http://foo.com/icon_48.png", "", sizes_48, IconPurpose::BADGE));
+ icons.push_back(CreateIcon("http://foo.com/icon_48.png", "", sizes_48,
+ IconPurpose::BADGE));
icons.push_back(
CreateIcon("http://foo.com/icon_96.png", "", sizes_96, IconPurpose::ANY));
icons.push_back(CreateIcon("http://foo.com/icon_144.png", "", sizes_144,
@@ -117,8 +119,8 @@ TEST(ManifestIconSelector, PurposeFiltering) {
icons, 48, kMinimumIconSize, IconPurpose::BADGE);
EXPECT_EQ("http://foo.com/icon_48.png", url.spec());
- url = ManifestIconSelector::FindBestMatchingIcon(
- icons, 48, kMinimumIconSize, IconPurpose::ANY);
+ url = ManifestIconSelector::FindBestMatchingIcon(icons, 48, kMinimumIconSize,
+ IconPurpose::ANY);
EXPECT_EQ("http://foo.com/icon_96.png", url.spec());
url = ManifestIconSelector::FindBestMatchingIcon(icons, 96, kMinimumIconSize,
@@ -493,3 +495,4 @@ TEST(ManifestIconSelector, UseAnyIfNoIdealSize) {
EXPECT_EQ("http://foo.com/icon.png", url.spec());
}
}
+}
please use gerrit instead 2017/06/12 23:57:22 newline above. // namespace content
zino 2017/06/13 16:07:45 Done.

Powered by Google App Engine
This is Rietveld 408576698