Remove unnecessary calls to GURL()

chrome/browser/ui/webui/browsing_history_handler_unittest.cc

 // Copyright (c) 2013 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/ui/webui/browsing_history_handler.h"
 
 #include <stdint.h>
 #include <memory>
 #include <set>
 
@@ skipping 54 matching lines @@
 }
 
 // Returns true if |result| matches the test data given by |correct_result|,
 // otherwise returns false.
 bool ResultEquals(
     const BrowsingHistoryHandler::HistoryEntry& result,
     const TestResult& correct_result) {
   base::Time correct_time =
       baseline_time + base::TimeDelta::FromHours(correct_result.hour_offset);
 
-  return result.time == correct_time && result.url == GURL(correct_result.url);
+  return result.time == correct_time && result.url == correct_result.url;
 }
 
 void IgnoreBoolAndDoNothing(bool ignored_argument) {}
 
 class TestSyncService : public browser_sync::TestProfileSyncService {
  public:
   explicit TestSyncService(Profile* profile)
       : browser_sync::TestProfileSyncService(
             CreateProfileSyncServiceParamsForTest(profile)),
         sync_active_(true) {}
@@ skipping 96 matching lines @@
 // Fails on Android.  http://crbug.com/2345
 #if defined(OS_ANDROID)
 #define MAYBE_MergeDuplicateResults DISABLED_MergeDuplicateResults
 #else
 #define MAYBE_MergeDuplicateResults MergeDuplicateResults
 #endif
 TEST_F(BrowsingHistoryHandlerTest, MAYBE_MergeDuplicateResults) {
   {
     // Basic test that duplicates on the same day are removed.
     TestResult test_data[] = {
-      { "http://google.com", 0 },
-      { "http://google.de", 1 },
-      { "http://google.com", 2 },
-      { "http://google.com", 3 }  // Most recent.
+      { "http://google.com/", 0 },
+      { "http://google.de/", 1 },
+      { "http://google.com/", 2 },
+      { "http://google.com/", 3 }  // Most recent.
     };
     std::vector<BrowsingHistoryHandler::HistoryEntry> results;
     AddQueryResults(test_data, arraysize(test_data), &results);
     BrowsingHistoryHandler::MergeDuplicateResults(&results);
 
     ASSERT_EQ(2U, results.size());
     EXPECT_TRUE(ResultEquals(results[0], test_data[3]));
     EXPECT_TRUE(ResultEquals(results[1], test_data[1]));
   }
 
   {
     // Test that a duplicate URL on the next day is not removed.
     TestResult test_data[] = {
-      { "http://google.com", 0 },
-      { "http://google.com", 23 },
-      { "http://google.com", 24 },  // Most recent.
+      { "http://google.com/", 0 },
+      { "http://google.com/", 23 },
+      { "http://google.com/", 24 },  // Most recent.
     };
     std::vector<BrowsingHistoryHandler::HistoryEntry> results;
     AddQueryResults(test_data, arraysize(test_data), &results);
     BrowsingHistoryHandler::MergeDuplicateResults(&results);
 
     ASSERT_EQ(2U, results.size());
     EXPECT_TRUE(ResultEquals(results[0], test_data[2]));
     EXPECT_TRUE(ResultEquals(results[1], test_data[1]));
   }
 
   {
     // Test multiple duplicates across multiple days.
     TestResult test_data[] = {
       // First day.
-      { "http://google.de", 0 },
-      { "http://google.com", 1 },
-      { "http://google.de", 2 },
-      { "http://google.com", 3 },
+      { "http://google.de/", 0 },
+      { "http://google.com/", 1 },
+      { "http://google.de/", 2 },
+      { "http://google.com/", 3 },
 
       // Second day.
-      { "http://google.de", 24 },
-      { "http://google.com", 25 },
-      { "http://google.de", 26 },
-      { "http://google.com", 27 },  // Most recent.
+      { "http://google.de/", 24 },
+      { "http://google.com/", 25 },
+      { "http://google.de/", 26 },
+      { "http://google.com/", 27 },  // Most recent.
     };
     std::vector<BrowsingHistoryHandler::HistoryEntry> results;
     AddQueryResults(test_data, arraysize(test_data), &results);
     BrowsingHistoryHandler::MergeDuplicateResults(&results);
 
     ASSERT_EQ(4U, results.size());
     EXPECT_TRUE(ResultEquals(results[0], test_data[7]));
     EXPECT_TRUE(ResultEquals(results[1], test_data[6]));
     EXPECT_TRUE(ResultEquals(results[2], test_data[3]));
     EXPECT_TRUE(ResultEquals(results[3], test_data[2]));
   }
 
   {
     // Test that timestamps for duplicates are properly saved.
     TestResult test_data[] = {
-      { "http://google.com", 0 },
-      { "http://google.de", 1 },
-      { "http://google.com", 2 },
-      { "http://google.com", 3 }  // Most recent.
+      { "http://google.com/", 0 },
+      { "http://google.de/", 1 },
+      { "http://google.com/", 2 },
+      { "http://google.com/", 3 }  // Most recent.
     };
     std::vector<BrowsingHistoryHandler::HistoryEntry> results;
     AddQueryResults(test_data, arraysize(test_data), &results);
     BrowsingHistoryHandler::MergeDuplicateResults(&results);
 
     ASSERT_EQ(2U, results.size());
     EXPECT_TRUE(ResultEquals(results[0], test_data[3]));
     EXPECT_TRUE(ResultEquals(results[1], test_data[1]));
     EXPECT_EQ(3u, results[0].all_timestamps.size());
     EXPECT_EQ(1u, results[1].all_timestamps.size());
@@ skipping 96 matching lines @@
   const base::DictionaryValue* first_entry;
   ASSERT_TRUE(arg2->GetDictionary(0, &first_entry));
 
   base::string16 title;
   ASSERT_TRUE(first_entry->GetString("title", &title));
 
   ASSERT_EQ(0u, title.find(base::ASCIIToUTF16("http://loooo")));
   EXPECT_EQ(300u, title.size());
 }
 #endif