RESOLVED INVALID 126114
[SOUP] ResourceHandle doesn't implement continueShouldUseCredentialStorage() 
https://bugs.webkit.org/show_bug.cgi?id=126114
Summary [SOUP] ResourceHandle doesn't implement continueShouldUseCredentialStorage()
Carlos Garcia Campos
Reported 2013-12-21 00:44:51 PST
Clients using the async methods like willSendRequestAsync or didReceiveResponseAsync use a method to continue like continueWillSendRequest or continueDidReceiveResponse. We are not implementing those yet, even though nwe are calling the async methods. In the case of willSendRequest it's clear we need to implement continueWillSendRequest because the request might have changed, but in the other cases we still don't know whether they are required or not.
Attachments
Add attachment proposed patch, testcase, etc.
Carlos Garcia Campos
Comment 1 2013-12-27 03:14:04 PST
Found a use case where continueDidReceiveResponse is required, when the main resource load is converted into a download by the policy checker. Patch attached to bug #126131 implements continueDidReceiveResponse.
Carlos Garcia Campos
Comment 2 2013-12-31 01:35:37 PST
continueWillSendRequest() is now implemented too, so I'm renaming the bug.
Pratik Solanki
Comment 3 2014-05-03 17:19:36 PDT
My fix for bug 132308 (r168232) removed the continueShouldUseCredentialStorage() call. I think there's anything more to do in this bug now.
Pratik Solanki
Comment 4 2014-05-03 17:20:09 PDT
(In reply to comment #3) > I think there's anything more to do in this bug now. I mean.. I don't think there's anything more to do in this bug now.
Note You need to log in before you can comment on or make changes to this bug.