Closed
Bug 1260283
Opened 8 years ago
Closed 8 years ago
Implement new console output frontend behind a pref
Categories
(DevTools :: Console, defect, P2)
Tracking
(firefox49 fixed)
RESOLVED
FIXED
Firefox 49
Tracking | Status | |
---|---|---|
firefox49 | --- | fixed |
People
(Reporter: bgrins, Assigned: linclark)
References
Details
(Keywords: meta)
Attachments
(11 files)
7.80 KB,
patch
|
bgrins
:
review+
|
Details | Diff | Splinter Review |
10.24 KB,
patch
|
bgrins
:
review+
|
Details | Diff | Splinter Review |
5.77 KB,
patch
|
bgrins
:
review+
|
Details | Diff | Splinter Review |
41.87 KB,
patch
|
bgrins
:
review+
|
Details | Diff | Splinter Review |
2.32 KB,
patch
|
bgrins
:
review+
|
Details | Diff | Splinter Review |
40.88 KB,
patch
|
bgrins
:
review+
|
Details | Diff | Splinter Review |
6.60 KB,
patch
|
bgrins
:
review+
|
Details | Diff | Splinter Review |
2.25 KB,
patch
|
bgrins
:
review+
|
Details | Diff | Splinter Review |
44.61 KB,
patch
|
bgrins
:
review+
|
Details | Diff | Splinter Review |
1.37 KB,
patch
|
bgrins
:
review+
|
Details | Diff | Splinter Review |
493 bytes,
patch
|
bgrins
:
review+
|
Details | Diff | Splinter Review |
We have been doing development on a fork on github so we don't need to fill up the tree with experimental changes. Originally at https://github.com/bgrins/console-output-experiment/ and now at https://github.com/bgrins/gecko-dev/tree/console-frontend. This is not meant to replace the 'jsterm' input field, nor the Variables View. This is still an experiment, but some of the goals are to have a frontend that we can swap out with the current one and is easier to unit test, easier to extend for new message types, removes XUL, and hopefully be more performant.
Assignee | ||
Comment 1•8 years ago
|
||
Assignee | ||
Comment 2•8 years ago
|
||
Attachment #8747257 -
Flags: review?(bgrinstead)
Assignee | ||
Comment 3•8 years ago
|
||
Attachment #8747259 -
Flags: review?(bgrinstead)
Assignee | ||
Comment 4•8 years ago
|
||
Attachment #8747261 -
Flags: review?(bgrinstead)
Assignee | ||
Comment 5•8 years ago
|
||
Attachment #8747262 -
Flags: review?(bgrinstead)
Assignee | ||
Comment 6•8 years ago
|
||
Attachment #8747263 -
Flags: review?(bgrinstead)
Assignee | ||
Comment 7•8 years ago
|
||
Attachment #8747264 -
Flags: review?(bgrinstead)
Assignee | ||
Comment 8•8 years ago
|
||
Attachment #8747265 -
Flags: review?(bgrinstead)
Assignee | ||
Comment 9•8 years ago
|
||
Attachment #8747266 -
Flags: review?(bgrinstead)
Assignee | ||
Comment 10•8 years ago
|
||
Attachment #8747267 -
Flags: review?(bgrinstead)
Assignee | ||
Comment 11•8 years ago
|
||
Attachment #8747269 -
Flags: review?(bgrinstead)
Reporter | ||
Comment 12•8 years ago
|
||
New try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=14527d195b75
Reporter | ||
Comment 13•8 years ago
|
||
(this includes changes to head.js to copy in some bits of shared/webconsole/test/common.js rather than loading it with subscript loader, since that wouldn't work unless if the shared directory already ran, and there was an issue with redeclaration of const Services against fx-team)
Reporter | ||
Comment 14•8 years ago
|
||
Bulk r+ coming, although keep an eye on what actually lands and not this patch series since there are some changes / folding as noted in Comment 13
Reporter | ||
Updated•8 years ago
|
Attachment #8747256 -
Flags: review?(bgrinstead) → review+
Reporter | ||
Updated•8 years ago
|
Attachment #8747257 -
Flags: review?(bgrinstead) → review+
Reporter | ||
Updated•8 years ago
|
Attachment #8747259 -
Flags: review?(bgrinstead) → review+
Reporter | ||
Updated•8 years ago
|
Attachment #8747261 -
Flags: review?(bgrinstead) → review+
Reporter | ||
Updated•8 years ago
|
Attachment #8747262 -
Flags: review?(bgrinstead) → review+
Reporter | ||
Updated•8 years ago
|
Attachment #8747263 -
Flags: review?(bgrinstead) → review+
Reporter | ||
Updated•8 years ago
|
Attachment #8747264 -
Flags: review?(bgrinstead) → review+
Reporter | ||
Updated•8 years ago
|
Attachment #8747265 -
Flags: review?(bgrinstead) → review+
Reporter | ||
Updated•8 years ago
|
Attachment #8747266 -
Flags: review?(bgrinstead) → review+
Reporter | ||
Updated•8 years ago
|
Attachment #8747267 -
Flags: review?(bgrinstead) → review+
Reporter | ||
Updated•8 years ago
|
Attachment #8747269 -
Flags: review?(bgrinstead) → review+
Comment 15•8 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/62a8a77385e5 https://hg.mozilla.org/integration/fx-team/rev/a50d06b44505 https://hg.mozilla.org/integration/fx-team/rev/44d8df5d3904 https://hg.mozilla.org/integration/fx-team/rev/93182f27abcd https://hg.mozilla.org/integration/fx-team/rev/09e7ae1dc4c9 https://hg.mozilla.org/integration/fx-team/rev/508da9fe1904 https://hg.mozilla.org/integration/fx-team/rev/4a3f21438a3e https://hg.mozilla.org/integration/fx-team/rev/463082706368 https://hg.mozilla.org/integration/fx-team/rev/992077cdabaf
Comment 16•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/62a8a77385e5 https://hg.mozilla.org/mozilla-central/rev/a50d06b44505 https://hg.mozilla.org/mozilla-central/rev/44d8df5d3904 https://hg.mozilla.org/mozilla-central/rev/93182f27abcd https://hg.mozilla.org/mozilla-central/rev/09e7ae1dc4c9 https://hg.mozilla.org/mozilla-central/rev/508da9fe1904 https://hg.mozilla.org/mozilla-central/rev/4a3f21438a3e https://hg.mozilla.org/mozilla-central/rev/463082706368 https://hg.mozilla.org/mozilla-central/rev/992077cdabaf
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox49:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•