Bug 38456 - [Qt] QtTestBrowser does not support websites which requires HTTP Authentication via dialogs
Summary: [Qt] QtTestBrowser does not support websites which requires HTTP Authenticati...
Status: CLOSED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 420+
Hardware: All All
: P2 Major
Assignee: Diego Gonzalez
URL:
Keywords: Qt, QtTriaged
: 38748 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-05-03 05:55 PDT by Markus Goetz
Modified: 2011-04-19 05:15 PDT (History)
11 users (show)

See Also:

Attachments
Patch (4.27 KB, patch)
2010-05-19 11:14 PDT, Diego Gonzalez 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Markus Goetz 2010-05-03 05:55:08 PDT 
Tested with current Qt 4.6 and webkit trunk.

Some SSL websites work, some report handshake failed.

HTTP auth does not work at all.
Comment 1 Jesus Sanchez-Palencia 2010-05-12 12:50:57 PDT 
*** Bug 38748 has been marked as a duplicate of this bug. ***
Comment 2 Diego Gonzalez 2010-05-17 12:23:47 PDT 
Do you have some list of sites which fails and which not fails? I cannot reproduce it on Linux
Comment 3 Diego Gonzalez 2010-05-17 13:32:20 PDT 
Does in your test pages is required any authentication dialog?
Comment 4 Diego Gonzalez 2010-05-18 12:27:39 PDT 
Is necessary to implement these kind of dialog in QtTestBrowser?
We have this in other Qt based browsers, e.g. Arora. so, is possible this kind of authentication working.
Comment 5 Diego Gonzalez 2010-05-19 11:14:16 PDT 
Created attachment 56502 [details]
Patch
Comment 6 Kenneth Rohde Christiansen 2010-05-19 11:27:35 PDT 
Comment on attachment 56502 [details]
Patch

LGTM, though it might make more sense implementing the dialog on the view site and just call that from the WebPage.
Comment 7 WebKit Commit Bot 2010-05-20 19:02:43 PDT 
Comment on attachment 56502 [details]
Patch

Clearing flags on attachment: 56502

Committed r59896: <http://trac.webkit.org/changeset/59896>
Comment 8 WebKit Commit Bot 2010-05-20 19:02:51 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 9 WebKit Review Bot 2010-05-20 19:35:31 PDT 
http://trac.webkit.org/changeset/59896 might have broken SnowLeopard Intel Release (Tests)
Comment 10 Simon Hausmann 2010-05-21 01:18:41 PDT 
Including in the release branch to make QA's life easier
Comment 11 Simon Hausmann 2010-05-21 01:19:50 PDT 
Revision r59896 cherry-picked into qtwebkit-2.0 with commit 1242653b36579fa8c05175424c6e7af9cbec12bd