Bug 42162

Summary: [Qt]REGRESSION?(r63191): http/tests/inspector/resource-har-conversion.html fails
Product: WebKit Reporter: Andras Becsi <abecsi>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: hausmann, japhet, kadam, ossy, tonyg, vestbo, webkit.review.bot, zarvai
Priority: P2 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Bug Depends on:    
Bug Blocks: 79666, 79668    
Attachments:
Description Flags
Patch none

Andras Becsi
Reported 2010-07-13 06:35:29 PDT
http/tests/inspector/resource-har-conversion.html fails on the 32bit and 64bit Qt Linux Release bots, but does not fail on the Debug builds. I'll skip the test until a solution for the problem is found.
Attachments
Patch (8.93 KB, patch)
2013-04-01 18:07 PDT, Seokju Kwon 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Andras Becsi
Comment 1 2010-07-13 06:43:34 PDT
Test skipped in r63200.
Csaba Osztrogonác
Comment 2 2012-05-17 04:24:05 PDT
The bug is still valid, here is the diff: --- /home/oszi/WebKit/WebKitBuild/Release/layout-test-results/http/tests/inspector/resource-har-conversion-expected.txt +++ /home/oszi/WebKit/WebKitBuild/Release/layout-test-results/http/tests/inspector/resource-har-conversion-actual.txt @@ -11,7 +11,7 @@ { startedDateTime : <plausible> id : <string> - title : "" + title : "http://127.0.0.1:8000/inspector/resource-har-conversion.html" pageTimings : { onContentLoad : <number> onLoad : <number> @@ -20,7 +20,6 @@ ] entries : [ { - pageref : "http://127.0.0.1:8000/inspector/resource-har-conversion.html" startedDateTime : <plausible> time : <number> request : { @@ -98,18 +97,18 @@ ] content : { size : <number> - mimeType : "application/x-javascript" - } - redirectURL : "" - headersSize : <number> - bodySize : <number> - } - cache : { - } - timings : <object> - } - { - pageref : "http://127.0.0.1:8000/inspector/resource-har-conversion.html" + mimeType : "text/plain" + } + redirectURL : "" + headersSize : <number> + bodySize : <number> + } + cache : { + } + timings : <object> + pageref : "page_1" + } + { startedDateTime : <plausible> time : <number> request : { @@ -143,9 +142,9 @@ cache : { } timings : <object> - } - { - pageref : "http://127.0.0.1:8000/inspector/resource-har-conversion.html" + pageref : "page_1" + } + { startedDateTime : <plausible> time : <number> request : { @@ -169,18 +168,18 @@ ] content : { size : <number> - mimeType : "application/x-javascript" - } - redirectURL : "" - headersSize : <number> - bodySize : <number> - } - cache : { - } - timings : <object> - } - { - pageref : "http://127.0.0.1:8000/inspector/resource-har-conversion.html" + mimeType : "text/plain" + } + redirectURL : "" + headersSize : <number> + bodySize : <number> + } + cache : { + } + timings : <object> + pageref : "page_1" + } + { startedDateTime : <plausible> time : <number> request : { @@ -218,6 +217,7 @@ cache : { } timings : <object> + pageref : "page_1" } ] }
Csaba Osztrogonác
Comment 3 2012-11-21 03:10:25 PST
It still fails, but with different diff: --- /home/oszi/WebKit/WebKitBuild/Release/layout-test-results/http/tests/inspector/resource-har-conversion-expected.txt +++ /home/oszi/WebKit/WebKitBuild/Release/layout-test-results/http/tests/inspector/resource-har-conversion-actual.txt @@ -10,7 +10,7 @@ { cache : { } - pageref : "http://127.0.0.1:8000/inspector/resource-har-conversion.html" + pageref : "page_1" request : { bodySize : <number> cookies : [ @@ -53,7 +53,7 @@ response : { bodySize : <number> content : { - mimeType : "application/x-javascript" + mimeType : "text/plain" size : <number> } cookies : [ @@ -99,7 +99,7 @@ { cache : { } - pageref : "http://127.0.0.1:8000/inspector/resource-har-conversion.html" + pageref : "page_1" request : { bodySize : <number> cookies : [ @@ -135,7 +135,7 @@ { cache : { } - pageref : "http://127.0.0.1:8000/inspector/resource-har-conversion.html" + pageref : "page_1" request : { bodySize : <number> cookies : [ @@ -151,7 +151,7 @@ response : { bodySize : <number> content : { - mimeType : "application/x-javascript" + mimeType : "text/plain" size : <number> } cookies : [ @@ -170,7 +170,7 @@ { cache : { } - pageref : "http://127.0.0.1:8000/inspector/resource-har-conversion.html" + pageref : "page_1" request : { bodySize : <number> cookies : [ @@ -216,7 +216,7 @@ onLoad : <number> } startedDateTime : <plausible> - title : "" + title : "http://127.0.0.1:8000/inspector/resource-har-conversion.html" } ] version : <string>
Csaba Osztrogonác
Comment 4 2012-11-21 03:14:11 PST
It isn't release/debug specific. It fails in relase and debug mode too with same diff. But it fails on WK2 on a different way - with notifyDone timeout: -Tests conversion of Inspector's resource representation into HAR format. +FAIL: Timed out waiting for notifyDone to be called -Page reloaded.
Seokju Kwon
Comment 5 2013-04-01 18:07:30 PDT
Created attachment 196047 [details] Patch
Seokju Kwon
Comment 6 2013-04-01 18:07:50 PDT
Comment on attachment 196047 [details] Patch - Add QT expected results for resource-har-conversion.html (only difference from the cross-platform results is that the .js resources are now given a MIME type of "text/plain") - Need Web Inspector in WK2 (skip it in qt-5.0-wk2/TestExpectation)
Csaba Osztrogonác
Comment 7 2013-04-02 01:32:53 PDT
Comment on attachment 196047 [details] Patch rs=me
WebKit Review Bot
Comment 8 2013-04-02 01:47:47 PDT
Comment on attachment 196047 [details] Patch Clearing flags on attachment: 196047 Committed r147405: <http://trac.webkit.org/changeset/147405>
WebKit Review Bot
Comment 9 2013-04-02 01:47:51 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.