804396 - [system] screenshot of apps that never brought to foreground is distorted

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Description

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Attachment: screenshot

## Environment :
Otoro phone, build 2012-10-22
Taken from default.xml in b2g-distro: 
* "platform_build" revision= 0d6d050bc37d3167cc82a1885fd7660456bb0f4e
* "gaia" revision= cf533e66c4c589478f5addc53cab2d9a4853402b 
* "releases-mozilla-aurora" revision= 471232068fc202ba54b12eaf5b7de70aef4a079e
* "gonk-misc" revision= da6e50508245d023166e0c04369200e5bb898220

  
## Repro :
1. launch clock app
2. set alarm for 1 min into future
3. long tap home button and swipe clock app closed
4. wait for alarm
5. long tap home button

## Expected :
1. full shot of clock app

## Actual :
1. part of the clock app enlarged

## Note :
1. see screenshot

Comment 1

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Update title -- we we need to ensure the size of the iframe is set properly as we create the frame -- not until we launch it.

Comment 2

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Attachment: Pointer https://github.com/mozilla-b2g/gaia/pull/5971

https://github.com/mozilla-b2g/gaia/pull/5971

[Approval Request Comment]
Bug caused by (feature/regressing bug #): launching app with system message mozChromeEvent; not bring app to foreground as we launch it
User impact if declined: as described in STR
Testing completed: testing done on Otoro
Risk to taking this patch (and alternatives if risky): there shouldn't be any risk.

Comment 3

[David Flanagan [:djf]]

Comment on attachment 674520 [details]
Pointer https://github.com/mozilla-b2g/gaia/pull/5971

r- See the comment on github

Comment 4

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Attachment: Pointer https://github.com/mozilla-b2g/gaia/pull/5971

Thanks for the comment, I didn't trace the calls closely enough. Some comment is addressed on the same pull request.

Comment 5

[David Flanagan [:djf]]

Comment on attachment 674948 [details]
Pointer https://github.com/mozilla-b2g/gaia/pull/5971

looks good now

Comment 6

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Comment on attachment 674948 [details]
Pointer https://github.com/mozilla-b2g/gaia/pull/5971

a=me because this is a small js change for a big ugly issue but you may want to ask blocking-basecamp? next time :)

Comment 7

[Tim Guan-tin Chien [:timdream] (please needinfo)]

https://github.com/mozilla-b2g/gaia/pull/5971

Comment 8

[Rebecca Billings [:rbillings]]

Verified on the 11/12 otoro us build.

Comment 9

[Rebecca Billings [:rbillings]]

Verified on the 11/12 otoro us build.