Bug 78792 - [Qt][WK2] Implement proxy authentication handling.
Summary: [Qt][WK2] Implement proxy authentication handling.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Michael Brüning
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-02-16 02:15 PST by Michael Brüning
Modified: 2012-02-22 07:11 PST (History)
1 user (show)

See Also:


Attachments
Patch (22.69 KB, patch)
2012-02-16 02:38 PST, Michael Brüning
no flags Details | Formatted Diff | Diff
Patch (25.60 KB, patch)
2012-02-16 02:56 PST, Michael Brüning
no flags Details | Formatted Diff | Diff
Patch (26.25 KB, patch)
2012-02-20 03:54 PST, Michael Brüning
no flags Details | Formatted Diff | Diff
Patch (3.76 KB, patch)
2012-02-22 05:53 PST, Michael Brüning
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Brüning 2012-02-16 02:15:50 PST
[Qt][WK2] Implement proxy authentication handling.
Comment 1 Michael Brüning 2012-02-16 02:38:03 PST
Created attachment 127337 [details]
Patch
Comment 2 Michael Brüning 2012-02-16 02:42:09 PST
Comment on attachment 127337 [details]
Patch

Forgot to add the QML property. Sorry...
Comment 3 Michael Brüning 2012-02-16 02:56:09 PST
Created attachment 127344 [details]
Patch
Comment 4 Michael Brüning 2012-02-20 03:54:03 PST
Created attachment 127802 [details]
Patch
Comment 5 WebKit Review Bot 2012-02-20 04:25:30 PST
Comment on attachment 127802 [details]
Patch

Clearing flags on attachment: 127802

Committed r108232: <http://trac.webkit.org/changeset/108232>
Comment 6 WebKit Review Bot 2012-02-20 04:25:34 PST
All reviewed patches have been landed.  Closing bug.
Comment 7 Michael Brüning 2012-02-22 05:44:50 PST
Reopening as there was two things wrong with the original implementation:
   - it wasn't connecting the signal proxuAuthenticationRequired to the slot that was implemented for it
   - the slot that was meant to deal with it had the wrong parameter list (it was the one for authenticationRequired).

Updated patch coming up...
Comment 8 Michael Brüning 2012-02-22 05:53:15 PST
Created attachment 128189 [details]
Patch
Comment 9 WebKit Review Bot 2012-02-22 07:11:19 PST
Comment on attachment 128189 [details]
Patch

Clearing flags on attachment: 128189

Committed r108499: <http://trac.webkit.org/changeset/108499>
Comment 10 WebKit Review Bot 2012-02-22 07:11:24 PST
All reviewed patches have been landed.  Closing bug.