DumpRenderTree is printing the URL triggering an auth challenge in the didReceiveAuthenticationChallengeInFrame callback. WebKitTestRunner is not. This should be made consistent. Once consistent, some WK2 specific test expectations could be removed.
I don't think that this can be done. WebKit1 and WebKit2 have different APIs, and we need to test both. WebKit2 does not have the URL in authentication challenge at all.
(In reply to comment #1) > I don't think that this can be done. WebKit1 and WebKit2 have different APIs, and we need to test both. > > WebKit2 does not have the URL in authentication challenge at all. We would need to add a new API for WK2 to get access to the URL through the auth challenge. Alternatively, we could remove URL printing for WK1.
> We would need to add a new API for WK2 to get access to the URL through the auth challenge. It's intentional that WebKit2 does not expose it, because that's not always meaningful. Some servers are authenticated per connection, not per resource (notably NTLM authentication). > Alternatively, we could remove URL printing for WK1. In this case, we won't be testing WebKit1 API, which is not good.
(In reply to comment #3) > > We would need to add a new API for WK2 to get access to the URL through the auth challenge. > > It's intentional that WebKit2 does not expose it, because that's not always meaningful. Some servers are authenticated per connection, not per resource (notably NTLM authentication). > > > Alternatively, we could remove URL printing for WK1. > > In this case, we won't be testing WebKit1 API, which is not good. I guess that for these tests then, WK2 test expectations should be made the norm and WK1 the exception.
Yes, this seems reasonable to me.
> > It's intentional that WebKit2 does not expose it, because that's not always meaningful. Some servers are authenticated per connection, not per resource (notably NTLM authentication). An improvement would be for WK2 to log the meaningful information, i.e the protection space host and port IIUC. > I guess that for these tests then, WK2 test expectations should be made the norm and WK1 the exception. If there is no objection, I will prepare a patch that would improve WK2 logging and rationalize test expectations for the impacted tests.
Created attachment 238362 [details] Patch
> I guess that for these tests then, WK2 test expectations should be made the norm and WK1 the exception. The patch is factorizing modified EFL/GTK/MAC WK2 expected files in platform/wk2. I did not make WK2 test expectations the norm since it may have an impact on WK1 win port. To empty progressively platform/wk2, a platform/wk1 folder may be created to factorize WK1 mac and win ports.
Comment on attachment 238362 [details] Patch Clearing flags on attachment: 238362 Committed r173760: <http://trac.webkit.org/changeset/173760>
All reviewed patches have been landed. Closing bug.