In order to fix a bug with handling unknown protocols in iFrames[0], we're
going to be calling NavigationDelegate.onLoadRequest for iFrame navigations in
addition to the top-level navigations for which we currently call it. To make
this usable, the LoadRequest class will be getting an additional field,
isTopLevel, which indicates whether the request is top-level or not. To
maintain your present behavior once this change has landed, you can simply
check if a request is non-top-level and allow it if so at the beginning of your
onLoadRequest implementation, eg:

@Override
GeckoResult<AllowOrDeny> onLoadRequest(GeckoSession session, LoadRequest req) {
    if (!req.isTopLevel) {
        return GeckoResult.fromValue(AllowOrDeny.ALLOW);
    }
    // Handle top-level load requests here.
}

This patch should be landing in nightly (78) in the next few days; I will send
a follow-up message to this mailing list once it does. Please let me know if
you have any questions/concerns by responding to this email or asking on Matrix
at https://chat.mozilla.org/#/room/#geckoview:mozilla.org

[0]: https://bugzilla.mozilla.org/show_bug.cgi?id=1596825

Regards,
Dylan Roeh
_______________________________________________
mobile-firefox-dev mailing list
mobile-firefox-dev@mozilla.org
https://mail.mozilla.org/listinfo/mobile-firefox-dev

Reply via email to