[Offline pages] Refactor URL conversions from TabAndroid

chrome/browser/android/offline_pages/offline_page_utils.h

Index: chrome/browser/android/offline_pages/offline_page_utils.h
diff --git a/chrome/browser/android/offline_pages/offline_page_utils.h b/chrome/browser/android/offline_pages/offline_page_utils.h
new file mode 100644
index 0000000000000000000000000000000000000000..2a1b68a30b87c030826f0a906b1f55e95226e327
--- /dev/null
+++ b/chrome/browser/android/offline_pages/offline_page_utils.h
@@ -0,0 +1,52 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_ANDROID_OFFLINE_PAGES_OFFLINE_PAGE_UTILS_H_
+#define CHROME_BROWSER_ANDROID_OFFLINE_PAGES_OFFLINE_PAGE_UTILS_H_
+
+#include "base/basictypes.h"
+
+class GURL;
+
+namespace content {
+class BrowserContext;
+}
+
+namespace offline_pages {
+struct OfflinePageItem;
+
+namespace android {

[dewittj, 2015/12/15 18:30:31]

why namespace android?

[fgorski, 2015/12/16 21:43:16]

Yeah, I was thinking about it. It is because of where the code sits, but given
that we only really apply the namespace to offline_page_bridge, I think it is
reasonable to remove it from here.

+
+// Returns true if |url| might point to an offline page.
+bool MightBeOfflineURL(const GURL& url);
+
+// Gets an offline URL of an offline page with |online_url| if one is present in
+// the |browser_context|.
+GURL GetOfflineURLByOnlineURL(content::BrowserContext* browser_context,
+                              const GURL& online_url);
+
+// Gets an online URL of an offline page with |offline_url| if one is present in
+// the |browser_context|.
+GURL GetOnlineURLByOfflineURL(content::BrowserContext* browser_context,
+                              const GURL& offline_url);
+
+// Gets a bookmark ID related to the |offline_url|.
+int64 GetBookmarkIdByOfflineURL(content::BrowserContext* browser_context,
+                                const GURL& offline_url);
+
+// Checks whether a given |offline_url| points to an offline page.
+bool IsOfflinePage(content::BrowserContext* browser_context,
+                   const GURL& offline_url);
+
+// Checks whether a given |online_url| has a related offline page.
+bool HasOfflinePageForOnlineURL(content::BrowserContext* browser_context,
+                                const GURL& online_url);
+
+// Checks if there are any offline pages saved in the |browser_context|
+const bool HasOfflinePages(content::BrowserContext* browser_context);
+
+}  // namespace android
+}  // namespace offline_pages
+
+#endif  // CHROME_BROWSER_ANDROID_OFFLINE_PAGES_OFFLINE_PAGE_UTILS_H_