Remove obsoleted function and add test

chrome/browser/android/offline_pages/offline_page_utils.cc

 // 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.
 
 #include "chrome/browser/android/offline_pages/offline_page_utils.h"
 
 #include "base/bind.h"
 #include "base/location.h"
 #include "base/optional.h"
 #include "base/strings/string_number_conversions.h"
@@ skipping 74 matching lines @@
 // static
 bool OfflinePageUtils::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);
 }
 
 // static
-GURL OfflinePageUtils::MaybeGetOfflineURLForOnlineURL(
-    content::BrowserContext* browser_context,
-    const GURL& online_url) {
-  const OfflinePageItem* offline_page =
-      MaybeGetBestOfflinePageForOnlineURL(browser_context, online_url);
-  if (!offline_page)
-    return GURL();
-
-  return offline_page->GetOfflineURL();
-}
-
-// static
 void OfflinePageUtils::GetOfflineURLForOnlineURL(
     content::BrowserContext* browser_context,
     const GURL& online_url,
     const base::Callback<void(const GURL&)>& callback) {
   OfflinePageModel* offline_page_model =
       OfflinePageModelFactory::GetForBrowserContext(browser_context);
   if (!offline_page_model) {
     base::ThreadTaskRunnerHandle::Get()->PostTask(
         FROM_HERE,
         base::Bind(&OnGetPageByOfflineURLDone, callback, base::nullopt));
@@ skipping 59 matching lines @@
   if (!offline_page)
     return;
 
   OfflinePageModel* offline_page_model =
       OfflinePageModelFactory::GetForBrowserContext(browser_context);
   DCHECK(offline_page_model);
   offline_page_model->MarkPageAccessed(offline_page->offline_id);
 }
 
 }  // namespace offline_pages