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

Unified Diff: ios/chrome/browser/reading_list/offline_url_utils.h

Issue 2606873002: Move the offline URL trimming to GetFormattedURL. (Closed)
Patch Set: clean Created 4 years 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: ios/chrome/browser/reading_list/offline_url_utils.h
diff --git a/ios/chrome/browser/reading_list/offline_url_utils.h b/ios/chrome/browser/reading_list/offline_url_utils.h
index 479b75ca4cf627fd2320d5bf3ccddbf953a9da48..ab2bea2de67f5656c3e9ce1a446805eb5271c0e2 100644
--- a/ios/chrome/browser/reading_list/offline_url_utils.h
+++ b/ios/chrome/browser/reading_list/offline_url_utils.h
@@ -8,6 +8,7 @@
#include <string>
#include "base/files/file_path.h"
+#include "base/strings/string16.h"
#include "url/gurl.h"
namespace reading_list {
@@ -30,6 +31,10 @@ GURL FileURLForDistilledURL(const GURL& distilled_url,
// Returns whether the URL points to a chrome offline URL.
bool IsOfflineURL(const GURL& url);
+// Strips scheme from the original URL of the offline page. This is meant to be
+// used by UI.
+base::string16 StripSchemeFromOnlineUrl(base::string16 online_url);
Eugene But (OOO till 7-30) 2016/12/28 17:02:59 Should this be |const base::string16&online_url|?
Eugene But (OOO till 7-30) 2016/12/28 17:02:59 Optional nit: Camel case Url is correct per C++ St
Olivier 2016/12/29 12:37:20 Done.
+
} // namespace reading_list
#endif // IOS_CHROME_BROWSER_READING_LIST_OFFLINE_URL_UTILS_H_

Powered by Google App Engine
This is Rietveld 408576698