Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(476)

Unified Diff: components/ntp_snippets/physical_web_pages/physical_web_page_suggestions_provider_unittest.cc

Issue 2565023002: Use GURLs in Physical Web data source (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/ntp_snippets/physical_web_pages/physical_web_page_suggestions_provider_unittest.cc
diff --git a/components/ntp_snippets/physical_web_pages/physical_web_page_suggestions_provider_unittest.cc b/components/ntp_snippets/physical_web_pages/physical_web_page_suggestions_provider_unittest.cc
index 70d4cc97d19062ed4635db8450e245720d0d9ad7..e6d8be94e0a632092357f82acbdaff17ef68d826 100644
--- a/components/ntp_snippets/physical_web_pages/physical_web_page_suggestions_provider_unittest.cc
+++ b/components/ntp_snippets/physical_web_pages/physical_web_page_suggestions_provider_unittest.cc
@@ -95,15 +95,15 @@ class PhysicalWebPageSuggestionsProviderTest : public testing::Test {
KnownCategories::PHYSICAL_WEB_PAGES);
}
- void FireUrlFound(const std::string& url) {
+ void FireUrlFound(const GURL& url) {
physical_web_data_source_.NotifyOnFound(url);
}
- void FireUrlLost(const std::string& url) {
+ void FireUrlLost(const GURL& url) {
physical_web_data_source_.NotifyOnLost(url);
}
- void FireUrlDistanceChanged(const std::string& url, double new_distance) {
+ void FireUrlDistanceChanged(const GURL& url, double new_distance) {
physical_web_data_source_.NotifyOnDistanceChanged(url, new_distance);
}

Powered by Google App Engine
This is Rietveld 408576698