| Index: chrome/browser/page_load_metrics/observers/from_gws_page_load_metrics_observer_unittest.cc
|
| diff --git a/chrome/browser/page_load_metrics/observers/from_gws_page_load_metrics_observer_unittest.cc b/chrome/browser/page_load_metrics/observers/from_gws_page_load_metrics_observer_unittest.cc
|
| index 299b3ae3b8963b9772383084076b886cc34ec4e8..3b590ab2680e96b1e24dfa12637dc4b883140ca3 100644
|
| --- a/chrome/browser/page_load_metrics/observers/from_gws_page_load_metrics_observer_unittest.cc
|
| +++ b/chrome/browser/page_load_metrics/observers/from_gws_page_load_metrics_observer_unittest.cc
|
| @@ -712,21 +712,22 @@ TEST_F(FromGWSPageLoadMetricsLoggerTest, IsGoogleSearchHostname) {
|
| bool expected_result;
|
| const char* url;
|
| } test_cases[] = {
|
| - {true, "www.google.com"},
|
| - {true, "www.google.co.uk"},
|
| - {true, "www.google.co.in"},
|
| - {false, "other.google.com"},
|
| - {false, "other.www.google.com"},
|
| - {false, "www.other.google.com"},
|
| - {false, "www.www.google.com"},
|
| - {false, "www.google.appspot.com"},
|
| - {false, "www.google.example.com"},
|
| + {true, "https://www.google.com/"},
|
| + {true, "https://www.google.co.uk/"},
|
| + {true, "https://www.google.co.in/"},
|
| + {false, "https://other.google.com/"},
|
| + {false, "https://other.www.google.com/"},
|
| + {false, "https://www.other.google.com/"},
|
| + {false, "https://www.www.google.com/"},
|
| + {false, "https://www.google.appspot.com/"},
|
| + {false, "https://www.google.example.com/"},
|
| // Search results are not served from the bare google.com domain.
|
| - {false, "google.com"},
|
| + {false, "https://google.com/"},
|
| };
|
| for (const auto& test : test_cases) {
|
| - EXPECT_EQ(test.expected_result,
|
| - FromGWSPageLoadMetricsLogger::IsGoogleSearchHostname(test.url))
|
| + EXPECT_EQ(
|
| + test.expected_result,
|
| + FromGWSPageLoadMetricsLogger::IsGoogleSearchHostname(GURL(test.url)))
|
| << "for URL: " << test.url;
|
| }
|
| }
|
|
|