Commit ee4aefb6 authored by Matt Giuca's avatar Matt Giuca Committed by Commit Bot

Fix getInstalledRelatedApps on Android.

Mojo bindings were incorrectly registered since r480972, resulting in
calls to Navigator.getInstalledRelatedApps never resolving or rejecting.
This correctly registers with a RenderFrameHost, not WebContents.

Adds an end-to-end test for getInstalledRelatedApps on Android, which
would've caught this breakage. (The feature is already covered by Blink
Layout Tests and Android Java unit tests, but there was nothing that
ensures the whole system is working.)

Bug: 750348
Change-Id: Ife895f26c42c6344906ea842ffb48f73d881b236
Reviewed-on: https://chromium-review.googlesource.com/597608
Commit-Queue: Matt Giuca <mgiuca@chromium.org>
Reviewed-by: default avatarKen Rockot <rockot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#491924}
parent cdb1fbf9
...@@ -3335,7 +3335,7 @@ void ChromeContentBrowserClient::InitFrameInterfaces() { ...@@ -3335,7 +3335,7 @@ void ChromeContentBrowserClient::InitFrameInterfaces() {
#if defined(OS_ANDROID) #if defined(OS_ANDROID)
frame_interfaces_parameterized_->AddInterface(base::Bind( frame_interfaces_parameterized_->AddInterface(base::Bind(
&ForwardToJavaWebContentsRegistry<blink::mojom::InstalledAppProvider>)); &ForwardToJavaFrameRegistry<blink::mojom::InstalledAppProvider>));
frame_interfaces_parameterized_->AddInterface( frame_interfaces_parameterized_->AddInterface(
base::Bind(&ForwardToJavaFrameRegistry<payments::mojom::PaymentRequest>)); base::Bind(&ForwardToJavaFrameRegistry<payments::mojom::PaymentRequest>));
#else #else
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment