Implement a component for shared (virtualized) net panel content space

NEW
Unassigned

Status

P4
normal
2 years ago
8 months ago

People

(Reporter: rickychien, Unassigned)

Tracking

(Blocks: 1 bug)

Trunk
Dependency tree / graph
Bug Flags:
qe-verify +

Firefox Tracking Flags

(firefox52 affected)

Details

(Whiteboard: [netmonitor-reserve])

Implement a generic component to take over every visualized features. This is a break down task for bug 1308425.

- Decouple Performance Statistics module into a reusable generic component for visualized features.
Blocks: 1307743
No longer blocks: 1308425
Depends on: 1308425
Whiteboard: [netmonitor]
Flags: qe-verify+
QA Contact: ciprian.georgiu
The idea behind this report is implementing something like xul:deck [1]

Deck is a component that displays only one of its children at a time. The selectedIndex property determines which child is displayed.

In context of the Network panel. Here is an example of the main Net panel layout:

<NetPanel>
  <Toolbar />
  <RequestListView />
</NetPanel>


But, the Net panel content can be also switched to display performance statistics instead (by clicking on the 'Toggle performance analysis' button in the toolbar). Later, we want to integrate WebSocket Monitor extension [2] and this will likely require the entire space to be used for displaying details related to Web sockets (There is also WebRTC, etc.)

So, the overall layout would change as follows:


<NetPanel>
  <Toolbar />*
  <Deck>
    <RequestListView />
    <PerformanceStatisticsView />
    <WebSocketsView />
  </Deck>
</NetPanel>

* The toolbar might be shared or every view can have its own toolbar

And the <Deck> component ensures that only one of its child views is displayed at a time.


Honza


[1] https://developer.mozilla.org/en-US/docs/Mozilla/Tech/XUL/deck
[2] https://addons.mozilla.org/en-US/firefox/addon/websocket-monitor/
Priority: -- → P2
Priority: P2 → P3
Whiteboard: [netmonitor] → [netmonitor-reserve]
Priority: P3 → P4

Updated

8 months ago
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.