1554913 - Simplify devtools/server/actors/network-monitor/network-observer.js and remove eslint-disable complexity

Status: RESOLVED FIXED

(DevTools :: Netmonitor, task, P3)

Description

[Patrick Brosset <:pbro>]

This file contains a special eslint comment to temporarily disable the complexity eslint rule since it contains one or more functions that exceed the maximum complexity threshold. The code should be simplified, and the comment removed. Link to the code in question: https://searchfox.org/mozilla-central/search?q=eslint-disable+complexity&case=false&regexp=false&path=devtools%2Fserver%2Factors%2Fnetwork-monitor%2Fnetwork-observer.js

Comment 1

[Ksenia Berezina [:ksenia]]

I would like to work on this

Comment 2

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Thanks for helping with the cleanup!

Honza

Comment 3

[Ksenia Berezina [:ksenia]]

Attachment: Bug 1554913 - Remove eslint-disable-next-line complexity from _setupHarTimings

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/48295741798b
Remove eslint-disable-next-line complexity from _setupHarTimings r=Honza

Comment 5

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/48295741798b