Add Tab List implementation for the remote debugger

RESOLVED FIXED in Thunderbird 24.0

Status

Thunderbird
General
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: Fallen, Assigned: Fallen)

Tracking

unspecified
Thunderbird 24.0

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
Aside from the main process, the remote debugger supports "tabs", which in Firefox are the real window tabs. I think the closest we can get to this is to provide access to the content windows inside <browser> elements.
(Assignee)

Comment 1

4 years ago
Created attachment 762839 [details] [diff] [review]
Fix - v1
Attachment #762839 - Flags: review?(mconley)
Comment on attachment 762839 [details] [diff] [review]
Fix - v1

Review of attachment 762839 [details] [diff] [review]:
-----------------------------------------------------------------

This is really straight-forward. Thanks Philipp.

Just some formatting nits, and maybe use a Set checkedWindows instead of an Array.

::: mail/components/debugger/content/dbg-mail-actors.js
@@ +61,5 @@
> +  _mustNotify: false,
> +  _listeningToMediator: false,
> +
> +  // These windows should be checked for browser elements
> +  checkedWindows: ["mail:3pane", "mail:messageWindow"],

If we're going to store _actorByBrowser as a Map, we might as well go the full nine yards and store checkedWindows as a Set.  Then use checkedWindows.has() instead of indexOf > -1.

@@ +64,5 @@
> +  // These windows should be checked for browser elements
> +  checkedWindows: ["mail:3pane", "mail:messageWindow"],
> +
> +  get onListChanged() {
> +      return this._onListChanged;

2 space indentation please

@@ +76,5 @@
> +    this._checkListening();
> +  },
> +
> +  _checkListening: function() {
> +   let shouldListenToMediator =

2 space indentation
Attachment #762839 - Flags: review?(mconley) → review+
(Assignee)

Comment 3

4 years ago
Pushed to comm-central changeset a17d8582694a
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
(Assignee)

Comment 4

4 years ago
All comments taken care of, I went with the Set.
Target Milestone: --- → Thunderbird 24.0
You need to log in before you can comment on or make changes to this bug.