Bug 179781 – LayoutTest http/tests/inspector/network/har/har-page.html is a flaky failure

• Component: Web Inspector
• Status: RESOLVED FIXED

[Ryan Haddad]

LayoutTest http/tests/inspector/network/har/har-page.html is a flaky failure

https://build.webkit.org/results/Apple%20Sierra%20Release%20WK2%20(Tests)/r224920%20(5780)/results.html

https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=http%2Ftests%2Finspector%2Fnetwork%2Fhar%2Fhar-page.html

--- /Volumes/Data/slave/sierra-release-tests-wk2/build/layout-test-results/http/tests/inspector/network/har/har-page-expected.txt
+++ /Volumes/Data/slave/sierra-release-tests-wk2/build/layout-test-results/http/tests/inspector/network/har/har-page-actual.txt
@@ -70,7 +70,7 @@
         "request": {
           "method": "GET",
           "url": "http://127.0.0.1:8000/inspector/resources/inspector-test.js",
-          "httpVersion": "",
+          "httpVersion": "HTTP/1.1",
           "cookies": [],
           "headers": "<filtered>",
           "queryString": [],
@@ -80,11 +80,11 @@
         "response": {
           "status": 200,
           "statusText": "OK",
-          "httpVersion": "",
+          "httpVersion": "HTTP/1.1",
           "cookies": [],
           "headers": "<filtered>",
           "content": {
-            "size": 0,
+            "size": 6707,
             "compression": 0,
             "mimeType": "application/x-javascript",
             "text": "<filtered text (6707)>"

[Blaze Burg]

Thanks Ryan. Joe should take a look when he's back in.

[Joseph Pecoraro]

We can just filter out this value. I'll make that change now.

[Joseph Pecoraro]

Created attachment 327209 [details]
[PATCH] Proposed Fix

[Alexey Proskuryakov]

Comment on attachment 327209 [details]
[PATCH] Proposed Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=327209&action=review

r=me on filtering, but I don't understand why the size changes.

> LayoutTests/http/tests/inspector/network/har/har-page-expected.txt:46
> -            "size": 2915,
> +            "size": 3042,

I don't understand this part.

[Joseph Pecoraro]

> > LayoutTests/http/tests/inspector/network/har/har-page-expected.txt:46
> > -            "size": 2915,
> > +            "size": 3042,
> 
> I don't understand this part.

That is the size of the test page itself.

[WebKit Commit Bot]

Comment on attachment 327209 [details]
[PATCH] Proposed Fix

Clearing flags on attachment: 327209

Committed r224979: <https://trac.webkit.org/changeset/224979>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/35621576>

[Michael Catanzaro]

*** Bug 179173 has been marked as a duplicate of this bug. ***