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

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

Issue 2933743002: Move chrome/browser/manifest to content/browser. (Closed)
Patch Set: rebased 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
« no previous file with comments | « chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc ('k') | chrome/browser/manifest/OWNERS » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/installable/installable_manager.cc
diff --git a/chrome/browser/installable/installable_manager.cc b/chrome/browser/installable/installable_manager.cc
index a6d16ed16d76b633f176f256d50c4660ef6107cb..a529d291356af458ad5a6f23bdeed5eef7e4a234 100644
--- a/chrome/browser/installable/installable_manager.cc
+++ b/chrome/browser/installable/installable_manager.cc
@@ -7,13 +7,13 @@
#include "base/bind.h"
#include "base/stl_util.h"
#include "base/strings/string_util.h"
-#include "chrome/browser/manifest/manifest_icon_downloader.h"
-#include "chrome/browser/manifest/manifest_icon_selector.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/ssl/security_state_tab_helper.h"
#include "components/security_state/core/security_state.h"
#include "content/public/browser/browser_context.h"
#include "content/public/browser/browser_thread.h"
+#include "content/public/browser/manifest_icon_downloader.h"
+#include "content/public/browser/manifest_icon_selector.h"
#include "content/public/browser/navigation_handle.h"
#include "content/public/browser/service_worker_context.h"
#include "content/public/browser/storage_partition.h"
@@ -532,14 +532,14 @@ void InstallableManager::CheckAndFetchBestIcon(const IconParams& params) {
IconProperty& icon = icons_[params];
icon.fetched = true;
- GURL icon_url = ManifestIconSelector::FindBestMatchingIcon(
+ GURL icon_url = content::ManifestIconSelector::FindBestMatchingIcon(
manifest().icons, ideal_icon_size_in_px, minimum_icon_size_in_px,
icon_purpose);
if (icon_url.is_empty()) {
icon.error = NO_ACCEPTABLE_ICON;
} else {
- bool can_download_icon = ManifestIconDownloader::Download(
+ bool can_download_icon = content::ManifestIconDownloader::Download(
GetWebContents(), icon_url, ideal_icon_size_in_px,
minimum_icon_size_in_px,
base::Bind(&InstallableManager::OnIconFetched,
« no previous file with comments | « chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc ('k') | chrome/browser/manifest/OWNERS » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698