Bug 103069

Summary: [Qt] REGRESSION(r135507): It made 13 tests fail
Product: WebKit Reporter: Csaba Osztrogonác <ossy>
Component: New BugsAssignee: Allan Sandfeld Jensen <allan.jensen>
Status: RESOLVED FIXED    
Severity: Normal CC: allan.jensen, hausmann, ossy
Priority: P2 Keywords: Qt, QtTriaged
Version: 420+   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 79666, 102667    
Attachments:
Description Flags
Patch
none
Patch hausmann: review+

Csaba Osztrogonác
Reported 2012-11-22 07:51:11 PST
http://build.webkit.sed.hu/results/x86-64%20Linux%20Qt%20Release/r135508%20%2845331%29/results.html fast/preloader/script.html http/tests/navigation/reload-subframe-object.html mhtml/check_domain.mht mhtml/multi_frames_binary.mht mhtml/multi_frames_ie.mht mhtml/multi_frames_unmht.mht mhtml/page_with_css_and_js_ie.mht mhtml/page_with_css_and_js_unmht.mht mhtml/page_with_image_ie.mht mhtml/page_with_image_unmht.mht mhtml/shared_buffer_bug.mht mhtml/simple_page_ie.mht mhtml/simple_page_unmht.mht
Attachments
Patch (3.13 KB, patch)
2012-11-22 08:17 PST, Allan Sandfeld Jensen
no flags
Patch (3.40 KB, patch)
2012-11-23 03:19 PST, Allan Sandfeld Jensen
hausmann: review+
Allan Sandfeld Jensen
Comment 1 2012-11-22 08:17:16 PST
Csaba Osztrogonác
Comment 2 2012-11-22 08:27:14 PST
I skipped them to paint the bots green - r135526. Please unskip them with the fix.
Allan Sandfeld Jensen
Comment 3 2012-11-23 02:09:50 PST
The issue with http/tests/navigation/reload-subframe-object.html is more complicated that the rest, though also more important to fix. It will require similar api changes as those I introduce to fix bug #103056, so I will fix it with the same or a related patch.
Allan Sandfeld Jensen
Comment 4 2012-11-23 02:31:25 PST
Csaba Osztrogonác
Comment 5 2012-11-23 02:34:29 PST
Reopen, because http/tests/navigation/reload-subframe-object.html is still skipped.
Allan Sandfeld Jensen
Comment 6 2012-11-23 03:19:21 PST
Allan Sandfeld Jensen
Comment 7 2012-11-23 03:20:56 PST
Since we do not call getMIMETypeForExtension from getMIMETypeForPath, it turned out I could fix this without changing the general API.
Allan Sandfeld Jensen
Comment 8 2012-11-23 03:28:27 PST
Note You need to log in before you can comment on or make changes to this bug.