1004104 - Disable caching when running netmonitor tests to hopefully fix some timeouts

Status: RESOLVED FIXED

(DevTools :: Netmonitor, defect)

Description

[Victor Porof [:vporof][:vp]]

This function in head.js [0] expects certain events to be fired to decide whether or not a requests was performed and received by the frontend. Sometimes, when requests are cached, one or more of those events [1] aren't fired, because there's no corresponding network activity to trigger them.

Disabling cache might fix those types of oranges.

[0] http://dxr.mozilla.org/mozilla-central/source/browser/devtools/netmonitor/test/head.js#146
[1] http://dxr.mozilla.org/mozilla-central/source/browser/devtools/netmonitor/test/head.js#168

Comment 1

[Ryan VanderMeulen [:RyanVM]]

Any luck with this, Victor? :)

Comment 2

[Rob Campbell [:rc] (:robcee)]

victor busy. he will see someday.

Comment 3

[Victor Porof [:vporof][:vp]]

Attachment: v1

Going through try.

Comment 4

[Victor Porof [:vporof][:vp]]

Try was green: https://tbpl.mozilla.org/?tree=Try&rev=ad1bde86a7cd

Comment 5

[Victor Porof [:vporof][:vp]]

Attachment: v2

Reruns look promising. This patch is even more aggressive and turns off caching on a per-request basis.

Comment 6

[Victor Porof [:vporof][:vp]]

Attachment: v2

Didn't qref properly before.

Comment 7

[Victor Porof [:vporof][:vp]]

New try run: https://tbpl.mozilla.org/?tree=Try&rev=73e5b8eab97a

Comment 8

[Victor Porof [:vporof][:vp]]

https://hg.mozilla.org/integration/fx-team/rev/78abcb1bf52f

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/78abcb1bf52f

Comment 10

[Ryan VanderMeulen [:RyanVM]]

Uplifted to beta32 to simplify uplifting bug 1036895.

https://hg.mozilla.org/releases/mozilla-beta/rev/00f079c876f6