Improve network event update stub generation

NEW
Unassigned

Status

()

Firefox
Developer Tools: Console
P2
enhancement
a year ago
2 months ago

People

(Reporter: linclark, Unassigned)

Tracking

(Blocks: 2 bugs)

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [newconsole-mvp])

(Reporter)

Description

a year ago
Originally posted by:linclark

see https://github.com/devtools-html/gecko-dev/issues/312

Network event update stubs are hard to generate. We have a good start in place in #283, but there are a couple of flaws still.

1. It's hard to come up with a unique name for update stubs. Currently we do a pretty good job, but there are still places where we use the same key for multiple stubs.
2. It's hard to determine how many packets we should listen for. Currently it's hardcoded to 8, but that number could easily change on the backend. Perhaps we should listen for a set amount of time in the stub generator instead.
3. Because the object being sent from the client is mutated in the client when new packets come in, it's likely that the stubs for certain messages are inaccurate... the object will have been mutated before we write it. We could potentially solve this by changing the client. In the client, we could break out the code that formats the packet in `_onNetworkEvent`/`_onNetworkEventUpdate`. Then it could be reused in the test. Eventually, I think we might want to rearrange the client so that it doesn't handle this at all, but that will impact net monitor so will require more planning.
(Reporter)

Updated

a year ago
Priority: -- → P2
Whiteboard: new-console
Blocks: 1308219
Flags: qe-verify-
Priority: P2 → P3
Whiteboard: new-console → [reserve-new-console]

Updated

11 months ago
Blocks: 1333131

Updated

9 months ago
Whiteboard: [reserve-new-console] → [reserve-console-html]

Updated

6 months ago
Priority: P3 → P4

Updated

2 months ago
Flags: qe-verify-
Priority: P4 → P2
Whiteboard: [reserve-console-html] → [newconsole-mvp]
You need to log in before you can comment on or make changes to this bug.