1125378 - Flag in remote-troubleshooting indicating e10s status

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[[:Cww]]

While we wait for bug 1091944 to resolve, we'd like to have some indicator of the e10s status of the user in the remote-troubleshooting packet. This way, we can quickly determine if the user's feedback in Input (or problem in Support) is caused by e10s.

Comment 1

[:Gavin Sharp [email: [e-mail]]]

We should add a boolean flag to the payload reflecting Services.appinfo.browserTabsRemoteAutostart.

Comment 2

[:Gijs (he/him)]

Why is "numRemoteWindows" vs. "numTotalWindows" not good enough?

Comment 3

[:Gavin Sharp [email: [e-mail]]]

That doesn't reveal whether e10s is "enabled" or not, on a default basis - we want to expose Services.appinfo.browserTabsRemoteAutostart directly.

Worth considering how this value is interpreted, though, given the presence of "new e10s/non-e10s window".

Comment 4

[:Gijs (he/him)]

(In reply to :Gavin Sharp [email: [e-mail]] from comment #3)
> That doesn't reveal whether e10s is "enabled" or not, on a default basis -
> we want to expose Services.appinfo.browserTabsRemoteAutostart directly.

Right, but...

> Worth considering how this value is interpreted, though, given the presence
> of "new e10s/non-e10s window".

Indeed. Seems like it's more useful to know how many windows are actually using e10s or not, but if we're sure we want the other info, it's not hard to implement. Will try to have a patch in a bit.

Comment 5

[:Gijs (he/him)]

Attachment: indicate default e10s state in remote troubleshooting,

Comment 6

[:Gijs (he/him)]

remote:   https://hg.mozilla.org/integration/fx-team/rev/9c7ebf496752

Comment 7

[:Gijs (he/him)]

Annnnd some bustage followup. I did wonder why there wasn't a test in toolkit for this. :-\

https://hg.mozilla.org/integration/fx-team/rev/65f735ba947e

Comment 8

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/9c7ebf496752
https://hg.mozilla.org/mozilla-central/rev/65f735ba947e

Comment 9

[[SV Manager] Florin Mezei, QA (:FlorinMezei)]

This seems to be covered automatically so setting qe-verify-. Please set back if any manual testing is needed.