Bug 795348 might need this unless that site can be evangelized. Either way, it will be good to have the ability to override navigator.userAgent up our sleeve. I suppose UserAgentOverrides.jsm needs to converted to an XPCOM service, or an XPCOM services utilizing UserAgentOverrides.jsm needs to be added, such that Navigator.cpp can use that. Does this sound reasonable?
Seems fine to me, yes.
Created attachment 670305 [details] [diff] [review] patch
Created attachment 670306 [details] [diff] [review] override for testing You can test this on <https://direktnet.raiffeisen.hu/rai/direktnet/home.do>. With the override applied, the login form should be present.
Comment on attachment 670305 [details] [diff] [review] patch >+++ b/dom/base/Navigator.cpp > Navigator::GetUserAgent(nsAString& aUserAgent) >+ NS_ENSURE_SUCCESS(rv, NS_OK); Second arg should be rv. >+ nsCOMPtr<nsIDocument> doc = do_QueryInterface(win->GetExtantDocument()); nsIDocument* doc = win->GetExtantDoc(); r=me with those.
Created attachment 670558 [details] [diff] [review] patch, review comments addressed
(In reply to Dão Gottwald [:dao] from comment #6) > https://bugzilla.mozilla.org/attachment.cgi?id=670558 wrong link... https://hg.mozilla.org/integration/mozilla-inbound/rev/9f28c28e988f
Comment on attachment 670558 [details] [diff] [review] patch, review comments addressed [Approval Request Comment] Extends our ability to react to sites that are broken due to bad UA sniffing. Bug caused by (feature/regressing bug #): n/a User impact if declined: n/a Testing completed (on m-c, etc.): on m-c Risk to taking this patch (and alternatives if risky): low, I think String or UUID changes made by this patch: none