1312801 - Navigating to about:srcdoc throws a "Firefox cannot open the page" alert

Status: RESOLVED FIXED

(Firefox for iOS :: Browser, defect, P1)

Description

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

http://mobile.nytimes.com/2016/09/29/technology/yahoo-data-breach-hacking.html

"Firefox cannot open the page - Firefox cannot open the page because it has an invalid address."

I put a breakpoint in BrowserViewController.webView(webView:, decidePolicyForNavigationAction:, decisionHandler:) and it looks like we have some new code in there that shows this alert if openURL() fails. This was done as part of bug 1212472

In this case of mobile.nyt.com the URL being passed to openURL is "about:docsrc". I am not entirely sure what that means but maybe we should do something similar that we do for about:blank URLs. We have this at the first check in the handler:

        // Fixes 1261457 - Rich text editor fails because requests to about:blank are blocked
        if url.scheme == "about" && url.resourceSpecifier == "blank" {
            decisionHandler(WKNavigationActionPolicy.Allow)
            return
        }

This is pretty annoying so I am marking this as tracking 6.0+ already.

Comment 1

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

about:srcdoc - not about:docsrc

Comment 2

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

Attachment: PR: https://github.com/mozilla-mobile/firefox-ios/pull/2192

This patch passes `about:srcdoc` URLs to WebKit instead if trying to handle it within the app and eventually passing them to `openURL()`.