Currently Activity is managed in Window Manager. We would like to implement an OO class "Activity Window", which inherits from AppWindow class. And then remove the startInlineActivity and stopInlineActivity from WindowManager.

WIP https://github.com/alivedise/gaia/compare/bugzilla;911053;activity-window-v2?expand=1

Proposed Multi chaining activity model https://docs.google.com/drawings/d/1I3Fvg1PHuft82rxBl0ITTpl2HGcdiN1fxPc5qhIqFHw/edit?usp=sharing

WIP today, https://github.com/alivedise/gaia/commit/c76c23a6a85be88dfda73e446524c27f7886dc6d Nearly done, need to improve test.

I found getIconSplash is broken in activity(current), file another bug to track.

Also current orientation strategy of activity is not perfect, file another bug to refine.

WIP today, https://github.com/alivedise/gaia/commit/a748377114150634eb8212414e316b55a15b1ce2

Implement ActivityWindow Implement ActivityWindowFactory New VisibilityManager New OrientationManager Unit Tests

Part.2 https://github.com/alivedise/gaia/commit/b289bded15764013f8c54ac31653cab7b84f61b1 Implement AppModalDialog + BaseUI Refine logging Remove invalid tests Travis is green except test_bookmark. https://travis-ci.org/mozilla-b2g/gaia/builds/13080095

Sent an email to qa-b2g@mozilla.com, if no response today I will land this. (Still trying to find out significant issues)

(In reply to Alive Kuo [:alive] from comment #9) > Sent an email to qa-b2g@mozilla.com, if no response today I will land this. > (Still trying to find out significant issues) Sorry for the lack of response - let me ping the tw-qa guys to help out.

Thanks! (In reply to Jason Smith [:jsmith] from comment #10) > (In reply to Alive Kuo [:alive] from comment #9) > > Sent an email to qa-b2g@mozilla.com, if no response today I will land this. > > (Still trying to find out significant issues) > > Sorry for the lack of response - let me ping the tw-qa guys to help out.

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

You inverted the aria-hide attribute. It should be the other way around, when we unlock, aria-hidden should be false.

What is this change for?

James, The bugs(such as 937508 ,938090) related to activity window are already fixed in V1.3. Do still you want such fixes included in fugu (v1.2f)?

(In reply to pcheng from comment #16) > James, > The bugs(such as 937508 ,938090) related to activity window are already > fixed in V1.3. Do still you want such fixes included in fugu (v1.2f)? v1.3 is ok. Thanks!

(In reply to pcheng from comment #16) > James, > The bugs(such as 937508 ,938090) related to activity window are already > fixed in V1.3. Do still you want such fixes included in fugu (v1.2f)? Oh, sorry, we should merge it back to v1.2. It cause several major bugs in our bugzilla.

Alive, Why is this needed in fugu? Can it wait till 1.4?

It's not me setting fugu stuff. I have the same question and I tend to modify the title back.

Need v1.2f patch.

Comment on attachment 8346440 [details] [diff] [review] 911053.patch I have no idea why you submit a patch for a refactoring issue which has been resolved. Please note this bug isn't for a specific bug but a piece of window manager refactoring stuff. I know this work indirectly resolves some issues happen on fugu, but we can't uplift (because a bunch of dependency) nor accept workaround patch here. For those issues (indirectly resolved by this bug) on v1.2, we decided to resolve them separately. They are tracking in bug 949854 and bug 949860. Please check them out.

To confirm, this commit does not uplift cleanly to v1.2f. As stated in comment 22, someone will need to adapt the patch for this branch.