Create JIT details view

RESOLVED FIXED in Firefox 41

Status

()

Firefox
Developer Tools: Performance Tools (Profiler/Timeline)
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jsantell, Assigned: jsantell)

Tracking

(Blocks: 1 bug)

41 Branch
Firefox 41
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox41 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Comment hidden (empty)
(Assignee)

Updated

3 years ago
Blocks: 1150299, 1174267
No longer depends on: 1150299
(Assignee)

Updated

3 years ago
Blocks: 1175662
(Assignee)

Updated

3 years ago
Depends on: 1175705
Created attachment 8624456 [details] [diff] [review]
1175650-jit-view.patch

Some small test failures, but wanted to get this out sooner than later
Assignee: nobody → jsantell
Status: NEW → ASSIGNED
Attachment #8624456 - Flags: review?(vporof)
Created attachment 8624468 [details] [diff] [review]
1175650-jit-view.patch

updated with working tests
Attachment #8624456 - Attachment is obsolete: true
Attachment #8624456 - Flags: review?(vporof)
Attachment #8624468 - Flags: review?(vporof)
https://treeherder.mozilla.org/#/jobs?repo=try&revision=bebc60cf5759
Comment on attachment 8624468 [details] [diff] [review]
1175650-jit-view.patch

Review of attachment 8624468 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/devtools/performance/performance.xul
@@ +342,5 @@
> +                  <tabs>
> +                    <tab id="optimizations-optimizations-tab"
> +                         label="Optimizations" />
> +                    <tab id="optimizations-frames-tab"
> +                         label="Frames" />

Localize?

@@ +345,5 @@
> +                    <tab id="optimizations-frames-tab"
> +                         label="Frames" />
> +                  </tabs>
> +                  <tabpanels flex="1">
> +                    <!-- Optimizations Panel -->

Nit: Add an newline before this comment to make it parsing this xul file easier on the eyes

@@ +360,5 @@
> +                        </toolbar>
> +                        <vbox id="jit-optimizations-raw-view"></vbox>
> +                      </vbox>
> +                    </tabpanel>
> +                    <tabpanel id="frames-tabpanel"

<-- Frames Panel --> comment goes here. Newline before

::: browser/devtools/performance/views/details-optimizations.js
@@ +23,5 @@
> +
> +    OptimizationsListView.initialize();
> +    FramesListView.initialize({ container: $("#frames-tabpanel") });
> +    FramesListView.on("select", this._onFramesListSelect);
> +    //PerformanceController.on(EVENTS.RECORDING_SELECTED, this.reset);

Commented out code?

@@ +35,5 @@
> +
> +    this.tabs = this._threadNode = this._frameNode = null;
> +    OptimizationsListView.destroy();
> +    FramesListView.destroy();
> +    DetailsSubview.destroy.call(this);

This should be the first method called.
Attachment #8624468 - Flags: review?(vporof) → review+
Comment on attachment 8624468 [details] [diff] [review]
1175650-jit-view.patch

Review of attachment 8624468 [details] [diff] [review]:
-----------------------------------------------------------------

Fixin' it up

::: browser/devtools/performance/performance.xul
@@ +342,5 @@
> +                  <tabs>
> +                    <tab id="optimizations-optimizations-tab"
> +                         label="Optimizations" />
> +                    <tab id="optimizations-frames-tab"
> +                         label="Frames" />

Want to hold off on localizing too much as I'd imagine we'll be changing a lot of names here for the prototype

Comment 6

3 years ago
https://hg.mozilla.org/integration/fx-team/rev/2fe76a95986b
https://hg.mozilla.org/integration/fx-team/rev/c3100b5ccfe8
(Assignee)

Updated

3 years ago
Whiteboard: [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/2fe76a95986b
https://hg.mozilla.org/mozilla-central/rev/c3100b5ccfe8
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-firefox41: affected → fixed
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 41
You need to log in before you can comment on or make changes to this bug.