Closed
Bug 1260283
Opened 10 years ago
Closed 10 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•10 years ago
|
||
| Assignee | ||
Comment 2•10 years ago
|
||
Attachment #8747257 -
Flags: review?(bgrinstead)
| Assignee | ||
Comment 3•10 years ago
|
||
Attachment #8747259 -
Flags: review?(bgrinstead)
| Assignee | ||
Comment 4•10 years ago
|
||
Attachment #8747261 -
Flags: review?(bgrinstead)
| Assignee | ||
Comment 5•10 years ago
|
||
Attachment #8747262 -
Flags: review?(bgrinstead)
| Assignee | ||
Comment 6•10 years ago
|
||
Attachment #8747263 -
Flags: review?(bgrinstead)
| Assignee | ||
Comment 7•10 years ago
|
||
Attachment #8747264 -
Flags: review?(bgrinstead)
| Assignee | ||
Comment 8•10 years ago
|
||
Attachment #8747265 -
Flags: review?(bgrinstead)
| Assignee | ||
Comment 9•10 years ago
|
||
Attachment #8747266 -
Flags: review?(bgrinstead)
| Assignee | ||
Comment 10•10 years ago
|
||
Attachment #8747267 -
Flags: review?(bgrinstead)
| Assignee | ||
Comment 11•10 years ago
|
||
Attachment #8747269 -
Flags: review?(bgrinstead)
| Reporter | ||
Comment 12•10 years ago
|
||
| Reporter | ||
Comment 13•10 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•10 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•10 years ago
|
Attachment #8747256 -
Flags: review?(bgrinstead) → review+
| Reporter | ||
Updated•10 years ago
|
Attachment #8747257 -
Flags: review?(bgrinstead) → review+
| Reporter | ||
Updated•10 years ago
|
Attachment #8747259 -
Flags: review?(bgrinstead) → review+
| Reporter | ||
Updated•10 years ago
|
Attachment #8747261 -
Flags: review?(bgrinstead) → review+
| Reporter | ||
Updated•10 years ago
|
Attachment #8747262 -
Flags: review?(bgrinstead) → review+
| Reporter | ||
Updated•10 years ago
|
Attachment #8747263 -
Flags: review?(bgrinstead) → review+
| Reporter | ||
Updated•10 years ago
|
Attachment #8747264 -
Flags: review?(bgrinstead) → review+
| Reporter | ||
Updated•10 years ago
|
Attachment #8747265 -
Flags: review?(bgrinstead) → review+
| Reporter | ||
Updated•10 years ago
|
Attachment #8747266 -
Flags: review?(bgrinstead) → review+
| Reporter | ||
Updated•10 years ago
|
Attachment #8747267 -
Flags: review?(bgrinstead) → review+
| Reporter | ||
Updated•10 years ago
|
Attachment #8747269 -
Flags: review?(bgrinstead) → review+
Comment 15•10 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•10 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: 10 years ago
status-firefox49:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Updated•7 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•