Move the offline URL trimming to GetFormattedURL.

ios/chrome/browser/reading_list/offline_url_utils_unittest.cc

 // Copyright 2016 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 "ios/chrome/browser/reading_list/offline_url_utils.h"
 
 #include <string>
 
 #include "base/files/file_path.h"
+#include "base/strings/utf_string_conversions.h"
+#include "base/test/gtest_util.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "url/gurl.h"
 
 // Checks the distilled URL for the page is chrome://offline/MD5/page.html;
 TEST(OfflineURLUtilsTest, DistilledURLForPathTest) {
   base::FilePath page_path("MD5/page.html");
   GURL distilled_url = reading_list::DistilledURLForPath(page_path, GURL());
   EXPECT_EQ("chrome://offline/MD5/page.html", distilled_url.spec());
 }
 
@@ skipping 41 matching lines @@
   EXPECT_EQ("/profile_path/Offline/MD5/page.html", file_url.path());
 
   GURL resource_url;
   file_url = reading_list::FileURLForDistilledURL(distilled_url, offline_path,
                                                   &resource_url);
   EXPECT_TRUE(resource_url.is_valid());
   EXPECT_TRUE(resource_url.SchemeIsFile());
   EXPECT_EQ("/profile_path/Offline/MD5/", resource_url.path());
 }
 
+// Checks that the offline URLs are correctly detected by |IsOfflineURL|.
 TEST(OfflineURLUtilsTest, IsOfflineURL) {
   EXPECT_FALSE(reading_list::IsOfflineURL(GURL()));
   EXPECT_FALSE(reading_list::IsOfflineURL(GURL("chrome://")));
   EXPECT_FALSE(reading_list::IsOfflineURL(GURL("chrome://offline-foobar")));
   EXPECT_FALSE(reading_list::IsOfflineURL(GURL("http://offline/")));
   EXPECT_FALSE(reading_list::IsOfflineURL(GURL("http://chrome://offline/")));
   EXPECT_TRUE(reading_list::IsOfflineURL(GURL("chrome://offline")));
   EXPECT_TRUE(reading_list::IsOfflineURL(GURL("chrome://offline/")));
   EXPECT_TRUE(reading_list::IsOfflineURL(GURL("chrome://offline/foobar")));
   EXPECT_TRUE(
       reading_list::IsOfflineURL(GURL("chrome://offline/foobar?foo=bar")));
 }
+
+// Checks that https:// scheme is correctly removed by
+// StripSchemeFromOnlineURLTest.
+TEST(OfflineURLUtilsTest, StripSchemeFromOnlineURLTest) {
+  size_t removed_size;
+  base::string16 empty_url;
+  EXPECT_EQ(reading_list::StripSchemeFromOnlineURL(empty_url, &removed_size),
+            empty_url);
+  EXPECT_EQ(removed_size, 0u);
+
+  base::string16 https_url = base::UTF8ToUTF16("https://www.chromium.org/");
+  base::string16 trimmed_https_url = base::UTF8ToUTF16("www.chromium.org/");
+  EXPECT_EQ(reading_list::StripSchemeFromOnlineURL(https_url, &removed_size),
+            trimmed_https_url);
+  EXPECT_EQ(removed_size, 8u);
+  base::string16 http_url = base::UTF8ToUTF16("http://www.chromium.org/");
+  EXPECT_DCHECK_DEATH(
+      reading_list::StripSchemeFromOnlineURL(http_url, &removed_size));
+
+  base::string16 other_scheme_url =
+      base::UTF8ToUTF16("scheme://www.chromium.org/");
+  EXPECT_EQ(
+      reading_list::StripSchemeFromOnlineURL(other_scheme_url, &removed_size),
+      other_scheme_url);
+  EXPECT_EQ(removed_size, 0u);
+
+  base::string16 no_scheme_url = base::UTF8ToUTF16("www.chromium.org/");
+  EXPECT_EQ(
+      reading_list::StripSchemeFromOnlineURL(no_scheme_url, &removed_size),
+      no_scheme_url);
+  EXPECT_EQ(removed_size, 0u);
+}