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

Unified Diff: chrome/browser/banners/app_banner_manager.cc

Issue 2589663003: mojo:: Rename mojo::GetProxy() to mojo::MakeRequest() (Closed)
Patch Set: Rebase 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: chrome/browser/banners/app_banner_manager.cc
diff --git a/chrome/browser/banners/app_banner_manager.cc b/chrome/browser/banners/app_banner_manager.cc
index 2038125b69daf702be777f9f65cf2c6e1a7a9fb0..6dc577b2cebb27ebbaf42f2b83a96a4cc05f1fa5 100644
--- a/chrome/browser/banners/app_banner_manager.cc
+++ b/chrome/browser/banners/app_banner_manager.cc
@@ -135,7 +135,7 @@ void AppBannerManager::RequestAppBanner(const GURL& validated_url,
void AppBannerManager::OnInstall() {
blink::mojom::InstallationServicePtr installation_service;
web_contents()->GetMainFrame()->GetRemoteInterfaces()->GetInterface(
- mojo::GetProxy(&installation_service));
+ mojo::MakeRequest(&installation_service));
DCHECK(installation_service);
installation_service->OnInstall();
}
@@ -358,10 +358,10 @@ void AppBannerManager::SendBannerPromptRequest() {
event_request_id_ = ++gCurrentRequestID;
web_contents()->GetMainFrame()->GetRemoteInterfaces()->GetInterface(
- mojo::GetProxy(&controller_));
+ mojo::MakeRequest(&controller_));
controller_->BannerPromptRequest(
- binding_.CreateInterfacePtrAndBind(), mojo::GetProxy(&event_),
+ binding_.CreateInterfacePtrAndBind(), mojo::MakeRequest(&event_),
{GetBannerType()},
base::Bind(&AppBannerManager::OnBannerPromptReply, GetWeakPtr()));
}

Powered by Google App Engine
This is Rietveld 408576698