Implement new console output frontend behind a pref

RESOLVED FIXED in Firefox 49

Status

()

Firefox
Developer Tools: Console
P2
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: bgrins, Assigned: linclark)

Tracking

({meta})

46 Branch
Firefox 49
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox49 fixed)

Details

Attachments

(11 attachments)

(Reporter)

Description

2 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

2 years ago
Blocks: 1243802
(Reporter)

Updated

2 years ago
Blocks: 1088360
(Assignee)

Comment 1

2 years ago
Created attachment 8747256 [details] [diff] [review]
Bug1260283-1.patch
Assignee: nobody → lclark
Status: NEW → ASSIGNED
Attachment #8747256 - Flags: review?(bgrinstead)
(Assignee)

Comment 2

2 years ago
Created attachment 8747257 [details] [diff] [review]
Bug1260283-2.patch
Attachment #8747257 - Flags: review?(bgrinstead)
(Assignee)

Comment 3

2 years ago
Created attachment 8747259 [details] [diff] [review]
Bug1260283-3.patch
Attachment #8747259 - Flags: review?(bgrinstead)
(Assignee)

Comment 4

2 years ago
Created attachment 8747261 [details] [diff] [review]
Bug1260283-4.patch
Attachment #8747261 - Flags: review?(bgrinstead)
(Assignee)

Comment 5

2 years ago
Created attachment 8747262 [details] [diff] [review]
Bug1260283-5.patch
Attachment #8747262 - Flags: review?(bgrinstead)
(Assignee)

Comment 6

2 years ago
Created attachment 8747263 [details] [diff] [review]
Bug1260283-6.patch
Attachment #8747263 - Flags: review?(bgrinstead)
(Assignee)

Comment 7

2 years ago
Created attachment 8747264 [details] [diff] [review]
Bug1260283-7.patch
Attachment #8747264 - Flags: review?(bgrinstead)
(Assignee)

Comment 8

2 years ago
Created attachment 8747265 [details] [diff] [review]
Bug1260283-8.patch
Attachment #8747265 - Flags: review?(bgrinstead)
(Assignee)

Comment 9

2 years ago
Created attachment 8747266 [details] [diff] [review]
Bug1260283-9.patch
Attachment #8747266 - Flags: review?(bgrinstead)
(Assignee)

Comment 10

2 years ago
Created attachment 8747267 [details] [diff] [review]
Bug1260283-10.patch
Attachment #8747267 - Flags: review?(bgrinstead)
(Assignee)

Comment 11

2 years ago
Created attachment 8747269 [details] [diff] [review]
Bug1260283-11.patch
Attachment #8747269 - Flags: review?(bgrinstead)
(Reporter)

Comment 12

2 years ago
New try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=14527d195b75
(Reporter)

Comment 13

2 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

2 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

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

Updated

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

Updated

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

Updated

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

Updated

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

Updated

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

Updated

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

Updated

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

Updated

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

Updated

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

Updated

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

Comment 15

2 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

2 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
Last Resolved: 2 years ago
status-firefox49: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
You need to log in before you can comment on or make changes to this bug.