Closed Bug 1260283 Opened 8 years ago Closed 8 years ago

Implement new console output frontend behind a pref

Categories

(DevTools :: Console, defect, P2)

46 Branch
defect

Tracking

(firefox49 fixed)

RESOLVED FIXED
Firefox 49
Tracking Status
firefox49 --- fixed

People

(Reporter: bgrins, Assigned: linclark)

References

Details

(Keywords: meta)

Attachments

(11 files)

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.
Blocks: 1243802
Blocks: 1088360
Assignee: nobody → lclark
Status: NEW → ASSIGNED
Attachment #8747256 - Flags: review?(bgrinstead)
Attachment #8747257 - Flags: review?(bgrinstead)
Attachment #8747259 - Flags: review?(bgrinstead)
Attachment #8747261 - Flags: review?(bgrinstead)
Attachment #8747262 - Flags: review?(bgrinstead)
Attachment #8747263 - Flags: review?(bgrinstead)
Attachment #8747264 - Flags: review?(bgrinstead)
Attachment #8747265 - Flags: review?(bgrinstead)
Attachment #8747266 - Flags: review?(bgrinstead)
Attachment #8747267 - Flags: review?(bgrinstead)
Attachment #8747269 - Flags: review?(bgrinstead)
(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)
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
Attachment #8747256 - Flags: review?(bgrinstead) → review+
Attachment #8747257 - Flags: review?(bgrinstead) → review+
Attachment #8747259 - Flags: review?(bgrinstead) → review+
Attachment #8747261 - Flags: review?(bgrinstead) → review+
Attachment #8747262 - Flags: review?(bgrinstead) → review+
Attachment #8747263 - Flags: review?(bgrinstead) → review+
Attachment #8747264 - Flags: review?(bgrinstead) → review+
Attachment #8747265 - Flags: review?(bgrinstead) → review+
Attachment #8747266 - Flags: review?(bgrinstead) → review+
Attachment #8747267 - Flags: review?(bgrinstead) → review+
Attachment #8747269 - Flags: review?(bgrinstead) → review+
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: