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

Unified Diff: chrome/browser/chrome_content_browser_client.cc

Issue 2851173004: Eliminate bind callback that doesn't take a BindSourceInfo parameter. (Closed)
Patch Set: . Created 3 years, 8 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/chrome_content_browser_client.cc
diff --git a/chrome/browser/chrome_content_browser_client.cc b/chrome/browser/chrome_content_browser_client.cc
index 408eb38d735328067d9aaa3322e7aa0d04354c84..2c2bd18cc28940c3ec945dd1eaae7f5fe842742c 100644
--- a/chrome/browser/chrome_content_browser_client.cc
+++ b/chrome/browser/chrome_content_browser_client.cc
@@ -885,9 +885,9 @@ AppLoadedInTabSource ClassifyAppLoadedInTabSource(
}
#endif // BUILDFLAG(ENABLE_EXTENSIONS)
-void CreateUsbDeviceManager(
- RenderFrameHost* render_frame_host,
- mojo::InterfaceRequest<device::mojom::UsbDeviceManager> request) {
+void CreateUsbDeviceManager(RenderFrameHost* render_frame_host,
+ const service_manager::BindSourceInfo& source_info,
+ device::mojom::UsbDeviceManagerRequest request) {
WebContents* web_contents =
WebContents::FromRenderFrameHost(render_frame_host);
if (!web_contents) {
@@ -902,7 +902,8 @@ void CreateUsbDeviceManager(
void CreateWebUsbChooserService(
RenderFrameHost* render_frame_host,
- mojo::InterfaceRequest<device::mojom::UsbChooserService> request) {
+ const service_manager::BindSourceInfo& source_info,
+ device::mojom::UsbChooserServiceRequest request) {
WebContents* web_contents =
WebContents::FromRenderFrameHost(render_frame_host);
if (!web_contents) {
@@ -1675,6 +1676,7 @@ void MaybeAppendBlinkSettingsSwitchForFieldTrial(
#if defined(OS_ANDROID)
void ForwardInstalledAppProviderRequest(
base::WeakPtr<service_manager::InterfaceProvider> interface_provider,
+ const service_manager::BindSourceInfo& source_info,
blink::mojom::InstalledAppProviderRequest request) {
if (!interface_provider ||
ChromeOriginTrialPolicy().IsFeatureDisabled("InstalledApp")) {
@@ -1685,6 +1687,7 @@ void ForwardInstalledAppProviderRequest(
void ForwardShareServiceRequest(
base::WeakPtr<service_manager::InterfaceProvider> interface_provider,
+ const service_manager::BindSourceInfo& source_info,
blink::mojom::ShareServiceRequest request) {
if (!interface_provider ||
ChromeOriginTrialPolicy().IsFeatureDisabled("WebShare")) {
« no previous file with comments | « chrome/browser/budget_service/budget_service_impl.cc ('k') | chrome/browser/chromeos/attestation/platform_verification_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698