1309185 - Move sharable basic components from netmonitor to shared

Status: RESOLVED WONTFIX

(DevTools :: Netmonitor, defect)

Description

[Ricky Chien [:rickychien] (inactive)]

This is a breakdown task for bug 1308450 in order to integrate HTTP inspector into the Net panel.

- Copy HTTPi code into netmonitor/shared panel directory

Comment 1

[[e-mail]]

need update devtools/docs/http-inspector.md as well

Comment 2

[[e-mail]]

Follow the discussion from bug 1308450 comment 1,

On contrary to DataProvider from webconsole/net, all related backend request in netmonitor are done with gNetwork, which is a shortcut for NetmonitorController::NetworkEventsHandler. Since it still works well, we can focus on UI refactoring at this moment.

Then I found more difference in webconsole/net

1. layout difference
2. style are hard coded in webconsole/net components
3. 10n: webconsole/net components use Locale.$STR to get l10n strings

Therefore instead of `Copy` webconsole/net, 
I'll start from porting a smaller portion of webconsole/net components to netmonitor/components/net/ and replace the current UI. It means porting Cookie tab in Bug 1317649, which includes cookie-tab, net-info-group-list, net-info-group, spinner components, related styles and tests

Comment 3

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

(In reply to Fred Lin [:gasolin] from comment #2)
> I'll start from porting a smaller portion of webconsole/net components to
> netmonitor/components/net/ and replace the current UI. It means porting
> Cookie tab in Bug 1317649, which includes cookie-tab, net-info-group-list,
> net-info-group, spinner components, related styles and tests
Sounds good to me, thanks Fred.

Honza

Comment 4

[Ricky Chien [:rickychien] (inactive)]

Fred, 

After discussing, we've agreed that it makes sense to port HTTP inspector from webconsole in each sub-panel bug (as you mentioned on comment 2). 

In order to avoid confusing, would you close this bug? Perhaps I think the bug might be reusable for the last step to move HTTPi component from netmonitor to shared folder once the HTTPi is completed and becomes stable. If the second option makes sense to you, please unassign yourself and change the bug title to something like "Move HTTPi component from netmonitor to shared":)

Comment 5

[[e-mail]]

Good suggestion. Though in practice I think we should keep HTTPi related components inside the netmonitor and only move reusable basic components to /share.

Marco, as a result we can remove this from MVP bug and discuss what can be added in next Monday's meeting

Comment 6

[Marco Mucci [:MarcoM]]

Hi Fred.  To confirm, would you like this bug moved from the current release to the MVP backlog, removed to the reserve backlog or removed from the project altogether?

Comment 7

[[e-mail]]

I'd like move this into reserve backlog, thanks

Comment 8

[[e-mail]]

We have implemented all HTTP inspector panels in netmonitor