Implement new console output frontend behind a pref

RESOLVED FIXED in Firefox 49

Status

P2
normal
RESOLVED FIXED
3 years ago
10 months ago

People

(Reporter: bgrins, Assigned: linclark)

Tracking

({meta})

46 Branch
Firefox 49
Dependency tree / graph

Firefox Tracking Flags

(firefox49 fixed)

Details

Attachments

(11 attachments)

(Reporter)

Description

3 years ago
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.
(Reporter)

Updated

3 years ago
Blocks: 1243802
(Reporter)

Updated

3 years ago
Blocks: 1088360
(Assignee)

Comment 1

3 years ago
Assignee: nobody → lclark
Status: NEW → ASSIGNED
Attachment #8747256 - Flags: review?(bgrinstead)
(Assignee)

Comment 2

3 years ago
Attachment #8747257 - Flags: review?(bgrinstead)
(Assignee)

Comment 3

3 years ago
Attachment #8747259 - Flags: review?(bgrinstead)
(Assignee)

Comment 4

3 years ago
Attachment #8747261 - Flags: review?(bgrinstead)
(Assignee)

Comment 5

3 years ago
Attachment #8747262 - Flags: review?(bgrinstead)
(Assignee)

Comment 6

3 years ago
Attachment #8747263 - Flags: review?(bgrinstead)
(Assignee)

Comment 7

3 years ago
Attachment #8747264 - Flags: review?(bgrinstead)
(Assignee)

Comment 8

3 years ago
Attachment #8747265 - Flags: review?(bgrinstead)
(Assignee)

Comment 9

3 years ago
Attachment #8747266 - Flags: review?(bgrinstead)
(Assignee)

Comment 10

3 years ago
Attachment #8747267 - Flags: review?(bgrinstead)
(Assignee)

Comment 11

3 years ago
Attachment #8747269 - Flags: review?(bgrinstead)
(Reporter)

Comment 13

3 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

3 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

3 years ago
Attachment #8747256 - Flags: review?(bgrinstead) → review+
(Reporter)

Updated

3 years ago
Attachment #8747257 - Flags: review?(bgrinstead) → review+
(Reporter)

Updated

3 years ago
Attachment #8747259 - Flags: review?(bgrinstead) → review+
(Reporter)

Updated

3 years ago
Attachment #8747261 - Flags: review?(bgrinstead) → review+
(Reporter)

Updated

3 years ago
Attachment #8747262 - Flags: review?(bgrinstead) → review+
(Reporter)

Updated

3 years ago
Attachment #8747263 - Flags: review?(bgrinstead) → review+
(Reporter)

Updated

3 years ago
Attachment #8747264 - Flags: review?(bgrinstead) → review+
(Reporter)

Updated

3 years ago
Attachment #8747265 - Flags: review?(bgrinstead) → review+
(Reporter)

Updated

3 years ago
Attachment #8747266 - Flags: review?(bgrinstead) → review+
(Reporter)

Updated

3 years ago
Attachment #8747267 - Flags: review?(bgrinstead) → review+
(Reporter)

Updated

3 years ago
Attachment #8747269 - Flags: review?(bgrinstead) → review+

Updated

10 months ago
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.