960915 - (emergencycall-as-app) [LockScreen][EmergencyCall] Make EmergencyCall as an app

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::System::Lockscreen, defect)

Description

[Greg Weng [:snowmantw][:gweng][:λ]]

Since we're implement SecureWindow to instead the embedded iframe, the emergency panel should be an standalone app, and follow the app creation/displaying/destroying logic to be managed.

Comment 1

[Greg Weng [:snowmantw][:gweng][:λ]]

Attachment: Patch

* Move the original system/emergency-call from System to an individual app.
* Use SecureWindow mechanism to launch it in LockScreen.

Comment 2

[Greg Weng [:snowmantw][:gweng][:λ]]

Attachment: Patch

New patch to bubble-tea because of the 1.4 landing policy.

Comment 3

[Greg Weng [:snowmantw][:gweng][:λ]]

Travis is green: https://travis-ci.org/mozilla-b2g/gaia/builds/19561555

I'll fix some minor issues and waiting review again.

Comment 4

[Greg Weng [:snowmantw][:gweng][:λ]]

Tim, since this bug is on the original v1.4 plan and should be done before 2/28, can you give me some feedback? Thanks.

Comment 5

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

(In reply to Greg Weng [:snowmantw][:gweng][:λ] from comment #4)
> Tim, since this bug is on the original v1.4 plan and should be done before
> 2/28, can you give me some feedback? Thanks.

I would worry this is to much of a change for bubble-tea, but I might be wrong. Let's discuss this offline.

Comment 6

[Greg Weng [:snowmantw][:gweng][:λ]]

I'll land it after rebase and wait for the tests.

Comment 7

[Greg Weng [:snowmantw][:gweng][:λ]]

Gareth's six questions:

- Is this a good idea?
> Yes. Since we have SecureWindow in LockScreen, we can finally remove all embedded iframes and to make them launched as an real app. While the camera is already an app, we need to make the emergency call as an app, too.

- What problem does it solve (and for whom)?
> As above.

- Is it high priority?
> Yes. Because this blocks the LockScreen refactoring plan I mentioned in dev-gaia.

- Are there any risks involved in doing this?
> I've tested with Travis (still on going) and manually tried it on my ZTE Open. But maybe there would be some regressions.

- Who should I ask for input?
> Still don't know what the 'input' is...

- Who should at least know that this is happening?
> Tim Guan-tin Chien

Comment 8

[Greg Weng [:snowmantw][:gweng][:λ]]

Oh, sorry for the inverted format. I'll correct it at the next bug.

Comment 9

[Greg Weng [:snowmantw][:gweng][:λ]]

Comment on attachment 8380437 [details] [review]
Patch

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

Comment 10

[Greg Weng [:snowmantw][:gweng][:λ]]

Attachment: New Patch

Since it's only different at the branch I want to merge, I keep the original one to show the r+ flag.

Comment 11

[Greg Weng [:snowmantw][:gweng][:λ]]

And I must mention about the opening and closing animation: Rob had discussed this with me, that he felt the old animation (from the right to left) would be confused with the Haida's sheet animation. So we should change secure apps' animation as it perform now (fade in/out).

Comment 12

[Greg Weng [:snowmantw][:gweng][:λ]]

The Travis is green:

https://travis-ci.org/mozilla-b2g/gaia/builds/21741014

Comment 13

[Greg Weng [:snowmantw][:gweng][:λ]]

master: https://github.com/mozilla-b2g/gaia/commit/8d9ba52b0340504acc024d0bfdd8717d674ab490

Hope it would be backed out.

Comment 14

[Greg Weng [:snowmantw][:gweng][:λ]]

LOL would *NOT* be backed out.

Comment 15

[Zac C (:zac)]

lol, it's passed on the device CI Greg!
Thanks heaps for updating the tests :)

Comment 16

[Greg Weng [:snowmantw][:gweng][:λ]]

Great! And thanks for the information!