Bug 42162 – [Qt]REGRESSION?(r63191): http/tests/inspector/resource-har-conversion.html fails

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Andras Becsi]

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.

[Andras Becsi]

Test skipped in r63200.

[Csaba Osztrogonác]

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]

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]

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]

Created attachment 196047 [details]
Patch

[Seokju Kwon]

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 on attachment 196047 [details]
Patch

rs=me

[WebKit Review Bot]

Comment on attachment 196047 [details]
Patch

Clearing flags on attachment: 196047

Committed r147405: <http://trac.webkit.org/changeset/147405>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.