Merge lp://qastaging/~daker/webbrowser-app/fix.1317428 into lp://qastaging/webbrowser-app

11 + if (invalid_hostname_characters.match(address)){
12 + return false
13 + }

It doesn’t seem correct to me to apply the invalid_hostname_characters RE to the entire address: you’d want to apply it to the host part only. But since we don’t really know whether address is a valid URl yet, there’s no easy way to extract the host part. So I’m not sure this makes sense.

If we really want to be strict about URL validation, I think we shouldn’t reinvent the wheel. QUrl provides us everything we need to parse a URL from a string and check whether it’s valid. Unfortunately it’s not exposed to QML (the url type in QML doesn’t match our needs), so we’d need to expose a C++ helper to QML to validate a given URL.