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

Unified Diff: content/public/browser/manifest_icon_selector.h

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/public/browser/manifest_icon_selector.h
diff --git a/chrome/browser/manifest/manifest_icon_selector.h b/content/public/browser/manifest_icon_selector.h
similarity index 95%
rename from chrome/browser/manifest/manifest_icon_selector.h
rename to content/public/browser/manifest_icon_selector.h
index d502aaff74fd7d721bd5a8fb4a0f320b73aaf31a..b3e240db63cd19760c9a7bfe1b9e5e139750fdfb 100644
--- a/chrome/browser/manifest/manifest_icon_selector.h
+++ b/content/public/browser/manifest_icon_selector.h
@@ -9,10 +9,11 @@
#include "content/public/common/manifest.h"
#include "url/gurl.h"
+namespace content {
please use gerrit instead 2017/06/12 23:57:22 Newline below.
zino 2017/06/13 16:07:45 Done.
// Selects the square icon with the supported image MIME types and the specified
// icon purpose that most closely matches the size constraints.
// This follows very basic heuristics -- improvements are welcome.
-class ManifestIconSelector {
+class CONTENT_EXPORT ManifestIconSelector {
public:
// Runs the algorithm to find the best matching icon in the icons listed in
// the Manifest. Size is defined in pixels.
@@ -32,5 +33,6 @@ class ManifestIconSelector {
private:
DISALLOW_IMPLICIT_CONSTRUCTORS(ManifestIconSelector);
};
+}
please use gerrit instead 2017/06/12 23:57:22 Newline above. // namespace content
zino 2017/06/13 16:07:45 Done.
#endif // CHROME_BROWSER_MANIFEST_MANIFEST_ICON_SELECTOR_H_

Powered by Google App Engine
This is Rietveld 408576698