Bug 101765 - [Qt] Can not load MHTML documents
Summary: [Qt] Can not load MHTML documents
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Page Loading (show other bugs)
Version: 420+
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Allan Sandfeld Jensen
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-11-09 08:53 PST by Allan Sandfeld Jensen
Modified: 2012-11-12 05:12 PST (History)
5 users (show)

See Also:

Attachments
Patch (2.60 KB, patch)
2012-11-09 08:57 PST, Allan Sandfeld Jensen 		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 Allan Sandfeld Jensen 2012-11-09 08:53:44 PST 
The Qt platform does not currently have MHTML enabled, and even if enabled, it will not run the tests or open .mht documents. The only reason it doesn't work is because we do not recognize the .mht extension.
Comment 1 Allan Sandfeld Jensen 2012-11-09 08:57:31 PST 
Created attachment 173317 [details]
Patch
Comment 2 Allan Sandfeld Jensen 2012-11-09 08:59:04 PST 
I have one uncertainty about the patch. We are not skipping the mhtml tests in TestExpectations, so I am not sure were they are automatically skipped or unskipped, but they run and pass.
Comment 3 Early Warning System Bot 2012-11-09 09:10:51 PST 
Comment on attachment 173317 [details]
Patch

Attachment 173317 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/14779451
Comment 4 Allan Sandfeld Jensen 2012-11-12 03:01:34 PST 
Comment on attachment 173317 [details]
Patch

That error does not look related to this patch in any way.
Comment 5 WebKit Review Bot 2012-11-12 05:12:14 PST 
Comment on attachment 173317 [details]
Patch

Clearing flags on attachment: 173317

Committed r134214: <http://trac.webkit.org/changeset/134214>
Comment 6 WebKit Review Bot 2012-11-12 05:12:18 PST 
All reviewed patches have been landed.  Closing bug.