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

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

Issue 2284933002: Remove OfflineURL from offline page (Closed)
Patch Set: Fix trybot Created 4 years, 4 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.cc
diff --git a/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc b/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc
index b5a951545af9be41aa2c5886e690c3c99674cf40..4b30580e8833562aaf592db832ae2122158e052d 100644
--- a/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc
+++ b/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc
@@ -37,21 +37,11 @@
namespace {
// Looks up the original, online URL of the site requested. The URL from the
-// WebContents may be an offline page or a distilled article which is not
-// appropriate for a home screen shortcut.
+// WebContents may be a distilled article which is not appropriate for a home
+// screen shortcut.
GURL GetShortcutUrl(content::BrowserContext* browser_context,
const GURL& actual_url) {
- GURL original_url =
- dom_distiller::url_utils::GetOriginalUrlFromDistillerUrl(actual_url);
-
- // If URL points to an offline content, get original URL.
- GURL online_url =
- offline_pages::OfflinePageUtils::MaybeGetOnlineURLForOfflineURL(
- browser_context, original_url);
- if (online_url.is_valid())
- return online_url;
-
- return original_url;
+ return dom_distiller::url_utils::GetOriginalUrlFromDistillerUrl(actual_url);
}
InstallableParams ParamsToPerformInstallableCheck(int ideal_icon_size_in_dp,

Powered by Google App Engine
This is Rietveld 408576698