[Offline pages] Refactor URL conversions from TabAndroid

chrome/browser/android/offline_pages/offline_page_utils.cc

+// Copyright 2015 The Chromium Authors. All rights reserved.

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

Could these all just be static functions in OfflinePageModel instead of a new
utils file?

[jianli, 2015/12/15 23:53:34]

Since we have quite a few static methods, it would be better to aggregate them
in a utility class that contains only static methods, IMO.

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

Overall, yes we could have methods with similar functionality on the model.
What we are getting from this functions are 2 things:
* Extra check that the feature is enabled
* ability to query offline page model without having to write boilerplate
  code to get it. Browser context is enough in this case.

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

Done.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/android/offline_pages/offline_page_utils.h"
+
+#include "base/strings/string_piece.h"
+#include "base/strings/string_util.h"
+#include "chrome/browser/android/offline_pages/offline_page_mhtml_archiver.h"
+#include "chrome/browser/android/offline_pages/offline_page_model_factory.h"
+#include "components/offline_pages/offline_page_feature.h"
+#include "components/offline_pages/offline_page_item.h"
+#include "components/offline_pages/offline_page_model.h"
+#include "content/public/browser/browser_context.h"
+#include "url/gurl.h"
+
+namespace offline_pages {
+namespace android {
+
+namespace {
+
+// Returns an offline page originated from the |online_url|.
+const offline_pages::OfflinePageItem* GetOfflinePageByOnlineURL(
+    content::BrowserContext* browser_context,
+    const GURL& online_url) {
+  DCHECK(browser_context);
+
+  if (!offline_pages::IsOfflinePagesEnabled())
+    return nullptr;
+
+  offline_pages::OfflinePageModel* offline_page_model =
+      offline_pages::OfflinePageModelFactory::GetForBrowserContext(
+          browser_context);
+  if (!offline_page_model)
+    return nullptr;
+
+  return offline_page_model->GetPageByOnlineURL(online_url);
+}
+
+// Returns an offline page that is stored as the |offline_url|.
+const offline_pages::OfflinePageItem* GetOfflinePageByOfflineURL(
+    content::BrowserContext* browser_context,
+    const GURL& offline_url) {
+  DCHECK(browser_context);
+
+  if (!offline_pages::IsOfflinePagesEnabled())
+    return nullptr;
+
+  // Note that we first check if the url likely points to an offline page
+  // before calling GetPageByOfflineURL in order to avoid unnecessary lookup
+  // cost.
+  if (!MightBeOfflineURL(offline_url))
+    return nullptr;
+
+  offline_pages::OfflinePageModel* offline_page_model =
+      offline_pages::OfflinePageModelFactory::GetForBrowserContext(
+          browser_context);
+

[jianli, 2015/12/15 23:53:34]

nit: remove empty line

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

Done.

+  if (!offline_page_model)
+    return nullptr;
+
+  return offline_page_model->GetPageByOfflineURL(offline_url);
+}
+
+}  // namespace
+
+bool MightBeOfflineURL(const GURL& url) {
+  // It has to be a file URL ending with .mhtml extension.
+  return url.is_valid() && url.SchemeIsFile() &&
+         base::EndsWith(url.spec(),
+                        OfflinePageMHTMLArchiver::GetFileNameExtension(),
+                        base::CompareCase::INSENSITIVE_ASCII);
+}
+
+GURL GetOfflineURLByOnlineURL(content::BrowserContext* browser_context,
+                              const GURL& online_url) {
+  const offline_pages::OfflinePageItem* offline_page =
+      GetOfflinePageByOnlineURL(browser_context, online_url);
+  if (!offline_page)
+    return GURL();
+
+  return offline_page->GetOfflineURL();
+}
+
+GURL GetOnlineURLByOfflineURL(content::BrowserContext* browser_context,
+                              const GURL& offline_url) {
+  DCHECK(browser_context);

[jianli, 2015/12/15 23:53:34]

nit: DCHECK is not needed since it is already in GetOfflinePageByOfflineURL.

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

Done.

+
+  const offline_pages::OfflinePageItem* offline_page =
+      GetOfflinePageByOfflineURL(browser_context, offline_url);
+  if (!offline_page)
+    return GURL();
+
+  return offline_page->url;
+}
+
+int64 GetBookmarkIdByOfflineURL(content::BrowserContext* browser_context,
+                                const GURL& offline_url) {
+  DCHECK(browser_context);

[jianli, 2015/12/15 23:53:34]

ditto

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

Done.

+
+  const offline_pages::OfflinePageItem* offline_page =
+      GetOfflinePageByOfflineURL(browser_context, offline_url);
+  if (!offline_page)
+    return -1;
+
+  return offline_page->bookmark_id;
+}
+
+bool IsOfflinePage(content::BrowserContext* browser_context,
+                   const GURL& offline_url) {
+  DCHECK(browser_context);

[jianli, 2015/12/15 23:53:34]

ditto

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

Done.

+
+  return GetOfflinePageByOfflineURL(browser_context, offline_url) != nullptr;
+}
+
+bool HasOfflinePageForOnlineURL(content::BrowserContext* browser_context,
+                                const GURL& online_url) {
+  const offline_pages::OfflinePageItem* offline_page =
+      GetOfflinePageByOnlineURL(browser_context, online_url);
+  return offline_page && !offline_page->file_path.empty();
+}
+
+const bool HasOfflinePages(content::BrowserContext* browser_context) {
+  DCHECK(browser_context);
+
+  if (!offline_pages::IsOfflinePagesEnabled())
+    return false;
+
+  offline_pages::OfflinePageModel* offline_page_model =
+      offline_pages::OfflinePageModelFactory::GetForBrowserContext(
+          browser_context);
+  return !offline_page_model->GetAllPages().empty();
+}
+
+}  // namespace android
+}  // namespace offline_pages