|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
58 bytes, text/x-review-board-request
|Details | Review|
Comment on attachment 8792906 [details] Bug 1302062 - Use React on performance recording list; https://reviewboard.mozilla.org/r/79770/#review78798 Looks good! Great comments in the RecordingsView too, w/r/t migration.
Comment on attachment 8792907 [details] Bug 1302062 - Remove RecordingsView from tests; https://reviewboard.mozilla.org/r/79772/#review78800 This looks like it was a fun one, heh. LGTM
Comment on attachment 8792908 [details] Bug 1302062 - Rewrite browser_perf-console-record-08.js to be easier to understand; https://reviewboard.mozilla.org/r/79774/#review78802 LGTM!! ::: devtools/client/performance/test/browser_perf-console-record-08.js:30 (Diff revision 2) > +// Represents a recorded profile from console.profile(). > +const CONSOLE = 1; > +// Represents a profile that is currently recording. > +const RECORDING = 2; > +// Represents a profile that is currently selected. > +const SELECTED = 4; Nice, much cleaner than before
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/c651f7174f37 Use React on performance recording list; r=jsantell