[Offline pages] Fixing OfflinePageUtis::EqualsIgnoringFragment

chrome/browser/android/offline_pages/offline_page_utils_unittest.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 <stdint.h>
 #include <utility>
 
 #include "base/callback.h"
@@ skipping 244 matching lines @@
   // This one should be missing
   OfflinePageUtils::CheckExistenceOfRequestsWithURL(
       profile(), kDownloadNamespace, kTestPage1Url,
       base::Bind(&HasDuplicatesCallback, base::Unretained(&has_duplicates),
                  base::Unretained(&latest_saved_time)));
   RunUntilIdle();
   EXPECT_FALSE(has_duplicates);
   EXPECT_TRUE(latest_saved_time.is_null());
 }
 
+TEST_F(OfflinePageUtilsTest, EqualsIgnoringFragment) {
+  EXPECT_TRUE(OfflinePageUtils::EqualsIgnoringFragment(
+      GURL("http://example.com/"), GURL("http://example.com/")));
+  EXPECT_TRUE(OfflinePageUtils::EqualsIgnoringFragment(

[jianli, 2016/11/28 22:18:56]

Please add more coverage like the followings:
  EXPECT_TRUE(OfflinePageUtils::EqualsIgnoringFragment(
      GURL("http://example.com/#test"), GURL("http://example.com/")));
  EXPECT_TRUE(OfflinePageUtils::EqualsIgnoringFragment(
      GURL("http://example.com/#test"), GURL("http://example.com/#test2")));

[fgorski, 2016/11/28 23:14:50]

Done.

+      GURL("http://example.com/"), GURL("http://example.com/#test")));
+  EXPECT_FALSE(OfflinePageUtils::EqualsIgnoringFragment(
+      GURL("http://example.com/"), GURL("http://test.com/#test")));
+}
+
 }  // namespace offline_pages