679733 - Widget Qt should send system notifications (display, taskswitcher, flight modes)

Status: RESOLVED FIXED

(Core Graveyard :: Widget: Qt, defect)

Description

[Oleg Romashin (:romaxa)]

We need to listen qmsystem2 (in future QtMobility API) for displayState, Activity, device mode notifications and generate related observer notifications.

Comment 1

[Oleg Romashin (:romaxa)]

Attachment: Listen device system states and send observe notifications

I've reused old notification states from maemo5, and added ts-background, ts-foreground notifications which notify us when browser goes into taskswitcher mode (in that case we should make last active tab/docshell inactive)

Comment 2

[Oleg Romashin (:romaxa)]

Comment on attachment 553792 [details] [diff] [review]
Listen device system states and send observe notifications

if you have better observe name than "ts-background", otherwise it good to go as system part

Comment 3

[Alon Zakai (:azakai)]

Who for example would be listening to these notifications?

Comment 4

[Oleg Romashin (:romaxa)]

1) ts-background/foreground - right now when fennec moved to background, top fennec docshell is staying active, and we should add listener possibly in fennec UI or in DocShell and set current visible tab as inactive (that should cause animations suspend et.c.)

2) As an option we can completely suspend child process when fennec goes to taskswitcher in order to suspend all possible JS activity on page.

3) system-active/inactive - practically fennec child process should be suspended

2-3) suspension can be canceled in case if there are some audio activity on the page...

Comment 5

[Oleg Romashin (:romaxa)]

Attachment: Deactivate last visible tab when fennec go into bg, or system inactive

Here is example which is deactivating last selected tab when fennec goes into background or system go to locked/inactive state or when display is off...

Comment 6

[Oleg Romashin (:romaxa)]

Comment on attachment 553942 [details] [diff] [review]
Deactivate last visible tab when fennec go into bg, or system inactive

new bug created for this implementation

Comment 7

[Oleg Romashin (:romaxa)]

Attachment: Child process suspend/resume API (SIGSTOP/SIGCONT)

Another example is suspend child process by hard (SIGSTOP) but only if it is not playing any audio stuff (case with lastfm et.c.)

And feedback on this?

Comment 8

[Alon Zakai (:azakai)]

(In reply to Oleg Romashin (:romaxa) from comment #7)
> Created attachment 553973 [details] [diff] [review]
> Child process suspend/resume API (SIGSTOP/SIGCONT)
> 
> Another example is suspend child process by hard (SIGSTOP) but only if it is
> not playing any audio stuff (case with lastfm et.c.)
> 
> And feedback on this?

There was opposition to this when we considered it last year, due to a lot of potential problems with it. It can probably be done, but is risky - probably better to figure out why we need to suspend the child and fix that problem (are there unnecessary timers, etc.).

Comment 9

[Oleg Romashin (:romaxa)]

> There was opposition to this when we considered it last year, due to a lot
> of potential problems with it. It can probably be done, but is risky -
> probably better to figure out why we need to suspend the child and fix that
> problem (are there unnecessary timers, etc.).

Practically this is short term fix for all timers in content process (Page JS mostly).

Possibly more elegant fix for that is to call Window->Suspend/Resume timeouts, that suspend all JS timers.

For me both ways are having + and -
SIGSTOP:
 + will not cause some conflicts in content process caused by components which are trying to use/create JS timers and those are not working.
 - May cause some undiscovered problems of Fenenc which is trying to interact with STOPPED child process

JS Suspend/Resume timeouts:
 + Keep child process in alive state, helps to catch other timers for long term fixing
 + problem described in SIGSTOP as "+" here is minus... such pages like gmail intentionally broken after such suspend..

Comment 10

[Doug Turner (:dougt)]

Comment on attachment 553792 [details] [diff] [review]
Listen device system states and send observe notifications

please adjust the copyright date.

Comment 11

[Doug Turner (:dougt)]

Comment on attachment 553973 [details] [diff] [review]
Child process suspend/resume API (SIGSTOP/SIGCONT)

please put this in a different bug (one fix per bug).

Comment 12

[Oleg Romashin (:romaxa)]

this was example of possible usage
> please put this in a different bug (one fix per bug).
will do that next

Comment 13

[Oleg Romashin (:romaxa)]

Attachment: [TOPUSH] Listen device system states and send observe notifications, fixed copyright

Comment 14

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

http://hg.mozilla.org/integration/mozilla-inbound/rev/fb72354db342

Comment 15

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

And backed out for Qt bustage:

http://tbpl.allizom.org/php/getParsedLog.php?id=6029466
http://hg.mozilla.org/integration/mozilla-inbound/rev/1cf660434af7
http://hg.mozilla.org/integration/mozilla-inbound/rev/87e7c8dbba63

Comment 16

[Oleg Romashin (:romaxa)]

Attachment: [TOPUSH] Listen device system states and send observe notifications, fixed copyright

Ok, this compiles fine on other Qt builds
http://tbpl.allizom.org/?tree=Try&usebuildbot=1&rev=86f78f470d87

Comment 17

[Ed Morley [:emorley]]

http://hg.mozilla.org/integration/mozilla-inbound/rev/b51e7241dd13

Comment 18

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

http://hg.mozilla.org/mozilla-central/rev/b51e7241dd13

Comment 19

[Oleg Romashin (:romaxa)]

This patch does not make big sense without 679923

Comment 20

[Oleg Romashin (:romaxa)]

Comment on attachment 554737 [details] [diff] [review]
[TOPUSH] Listen device system states and send observe notifications, fixed copyright

This patch is needed as backend for bug 679923, affecting only Qt Maemo6 port, more details about whole change is in https://bugzilla.mozilla.org/show_bug.cgi?id=679923#c37

Comment 21

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 554737 [details] [diff] [review]
[TOPUSH] Listen device system states and send observe notifications, fixed copyright

Leaving in fx9.