Bug 42162 - [Qt]REGRESSION?(r63191): http/tests/inspector/resource-har-conversion.html fails
Summary: [Qt]REGRESSION?(r63191): http/tests/inspector/resource-har-conversion.html fails
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords: Qt, QtTriaged
Depends on:
Blocks: 79666 79668
  Show dependency treegraph
 
Reported: 2010-07-13 06:35 PDT by Andras Becsi
Modified: 2013-04-02 01:47 PDT (History)
8 users (show)

See Also:


Attachments
Patch (8.93 KB, patch)
2013-04-01 18:07 PDT, Seokju Kwon
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andras Becsi 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.
Comment 1 Andras Becsi 2010-07-13 06:43:34 PDT
Test skipped in r63200.
Comment 2 Csaba Osztrogonác 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"
         }
     ]
 }
Comment 3 Csaba Osztrogonác 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>
Comment 4 Csaba Osztrogonác 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.
Comment 5 Seokju Kwon 2013-04-01 18:07:30 PDT
Created attachment 196047 [details]
Patch
Comment 6 Seokju Kwon 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)
Comment 7 Csaba Osztrogonác 2013-04-02 01:32:53 PDT
Comment on attachment 196047 [details]
Patch

rs=me
Comment 8 WebKit Review Bot 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>
Comment 9 WebKit Review Bot 2013-04-02 01:47:51 PDT
All reviewed patches have been landed.  Closing bug.