1401467 - (netmonitor-cdp) [meta] Support for CDP

Status: NEW

(DevTools :: Netmonitor, enhancement, P3)

Description

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

The Network monitor should support connecting to Chrome browser over CDP.

This is meta bug covering this effort.

-

First part of the work has been done in bug 1390037.

* New CDP connector implemented in: devtools/client/netmonitor/src/connector
* Check out devtools/client/netmonitor/src/connector/README.md to see how to test it.

Next steps (follow ups for bug 1390037):

* Cookies
HTTP cookies are not currently supported since `getCookies` method doesn't have request ID.

* Security
See more details:
https://github.com/locke12456/ChromeConnector/blob/develop/chrome/response.js#L46

The security information doesn't contain fingerprint from CDP. CDP doesn't seem to support it properly in the current version.

* Timings
See more details:
https://github.com/locke12456/ChromeConnector/blob/develop/chrome/response.js#L124

CDP provides many timing information. The current support uses only:
- connect
- send
- dns 
- receiveHeaders

We might use other timings and show them in Netmonitor UI.

* Response content
We need to verify usage of `encodedDataLength` map to `transferredSize`.

* Request
XHRs are currently not distinguished from other requests. It's not clear how/if the CDP protocol supports this.

Honza

Comment 1

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

Some code clean up yet needed:

* Rename `netmonitor/src/connector` to `netmonitor/src/connectors`
* Rename `netmonitor/src/connector/firefox-data-provider` to `netmonitor/src/connector/firefox/data-provider`
* Rename `netmonitor/src/connector/chrome/cdp-connector` to `netmonitor/src/connector/chrome/data-provider`

Honza

Comment 2

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

Original prototype implemented by Locke Chen:
https://github.com/locke12456/ChromeConnector

Honza

Comment 3

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

We might use info from:
https://github.com/locke12456/ChromeConnector/blob/develop/README.md

...and update:
netmonitor/src/connector/README.md 

Honza