OfflinePageModel cleanups - improved CanSavePage() and moved static kInvalidOfflineId to impl

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/strings/string_number_conversions.h"
 #include "base/strings/string_piece.h"
 #include "base/strings/string_util.h"
 #include "chrome/browser/android/offline_pages/offline_page_mhtml_archiver.h"
@@ skipping 43 matching lines @@
       OfflinePageModelFactory::GetForBrowserContext(browser_context);
   if (!offline_page_model)
     return nullptr;
 
   return offline_page_model->MaybeGetPageByOfflineURL(offline_url);
 }
 
 }  // namespace
 
 // static
+bool OfflinePageUtils::CanSaveURL(const GURL& url) {
+  return url.is_valid() && url.SchemeIsHTTPOrHTTPS();
+}
+
+// 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::GetOfflineURLForOnlineURL(
@@ skipping 44 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