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

Unified Diff: chrome/browser/ui/intents/web_intent_picker_controller.cc

Issue 8417043: Add webkit_glue namespace. Improve some variable and test names. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix merge error Created 9 years, 2 months 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: chrome/browser/ui/intents/web_intent_picker_controller.cc
diff --git a/chrome/browser/ui/intents/web_intent_picker_controller.cc b/chrome/browser/ui/intents/web_intent_picker_controller.cc
index 3d29e239b1f03f48196f9faab5a9be801efc7510..30208a8c22bd8c5d768b2ee3e76fc735333f60c1 100644
--- a/chrome/browser/ui/intents/web_intent_picker_controller.cc
+++ b/chrome/browser/ui/intents/web_intent_picker_controller.cc
@@ -58,7 +58,7 @@ class WebIntentPickerController::WebIntentDataFetcher
// WebIntentsRegistry::Consumer implementation.
virtual void OnIntentsQueryDone(
WebIntentsRegistry::QueryID,
- const std::vector<WebIntentServiceData>& intents) OVERRIDE;
+ const std::vector<webkit_glue::WebIntentServiceData>& services) OVERRIDE;
// A weak pointer to the picker controller.
WebIntentPickerController* controller_;
@@ -229,10 +229,10 @@ void WebIntentPickerController::OnCancelled() {
}
void WebIntentPickerController::OnWebIntentDataAvailable(
- const std::vector<WebIntentServiceData>& intent_data) {
+ const std::vector<webkit_glue::WebIntentServiceData>& services) {
urls_.clear();
- for (size_t i = 0; i < intent_data.size(); ++i) {
- urls_.push_back(intent_data[i].service_url);
+ for (size_t i = 0; i < services.size(); ++i) {
+ urls_.push_back(services[i].service_url);
}
// Tell the picker to initialize N urls to the default favicon
@@ -281,8 +281,8 @@ void WebIntentPickerController::WebIntentDataFetcher::Fetch(
void WebIntentPickerController::WebIntentDataFetcher::OnIntentsQueryDone(
WebIntentsRegistry::QueryID,
- const std::vector<WebIntentServiceData>& intents) {
- controller_->OnWebIntentDataAvailable(intents);
+ const std::vector<webkit_glue::WebIntentServiceData>& services) {
+ controller_->OnWebIntentDataAvailable(services);
query_id_ = -1;
}

Powered by Google App Engine
This is Rietveld 408576698