836766 - Drop arbitrary restriction that chat windows opened by the worker are minimized

Status: RESOLVED FIXED

(Firefox Graveyard :: SocialAPI, defect)

Description

[Mark Hammond [:markh] [:mhammond]]

There is no reasonable requirement for chat windows opened by the worker to be opened minimized.  It does nothing to prevent abuse by the provider and just adds an extra step for the user when they want to see the content of the incoming chat message.  So we should just drop this arbitrary restriction.

Later we could consider an optional way for a chat window to be opened minimized (whether by the worker of via the navigator api), but there doesn't seem to be a current use-case for this - so we should only do this when such a use-case can be identified.

Comment 1

[Shane Caraveo (:mixedpuppy)]

Attachment: remove forced minimize mode

https://tbpl.mozilla.org/?tree=Try&rev=08842555faab

Comment 2

[Shane Caraveo (:mixedpuppy)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/3d1434e45bd9

Comment 3

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/3d1434e45bd9

Comment 4

[Shane Caraveo (:mixedpuppy)]

Comment on attachment 756856 [details] [diff] [review]
remove forced minimize mode

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 784535 
User impact if declined: incoming call/chat requests are minimized making them largely unnoticeable by users.  this is especially problematic for incoming webrtc calls in talkilla
Testing completed (on m-c, etc.): on m-c
Risk to taking this patch (and alternatives if risky): low
String or IDL/UUID changes made by this patch: none

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/6d5e8bc3b065

Comment 6

[u279076]

Deprioritizing for verification since it looks like this has in-testsuite test coverage. Please drop the [qa-] tag and add the verifyme keyword if QA verification is desired.