Closed Bug 1026424 Opened 7 years ago Closed 7 years ago

Remove explicit frame switching where activityWindow is used

Categories

(Firefox OS Graveyard :: Gaia::UI Tests, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: zcampbell, Assigned: zcampbell)

References

Details

Attachments

(6 obsolete files)

After bug 1024433 was merged the displayedApp atom can now represent an app inside an activityWindow.

That means we can remove a lot of frame switching and just rely on the displayedApp atom.
Assignee: nobody → zcampbell
Attached file Bookmark app object pr (obsolete) —
Attachment #8441946 - Flags: review?(florin.strugariu)
Comment on attachment 8441946 [details] [review]
Bookmark app object pr

>https://github.com/mozilla-b2g/gaia/pull/20661
Attachment #8441946 - Attachment description: github pr → Bookmark app object pr
Attached file Camera app object pr (obsolete) —
Attachment #8441948 - Flags: review?(florin.strugariu)
Attached file Wallpaper app/test pr (obsolete) —
Attachment #8441951 - Flags: review?(florin.strugariu)
Attachment #8441946 - Flags: review?(florin.strugariu) → review?(viorela.ioia)
Attachment #8441948 - Flags: review?(florin.strugariu)
Attached file github pr (obsolete) —
Attachment #8441969 - Flags: review?(viorela.ioia)
Comment on attachment 8441946 [details] [review]
Bookmark app object pr

Lgtm, r+
Attachment #8441946 - Flags: review?(viorela.ioia) → review+
Comment on attachment 8441969 [details] [review]
github pr

Lgtm, I just added a suggestion in the PR
Attachment #8441969 - Flags: review?(viorela.ioia) → review+
Comment on attachment 8441948 [details] [review]
Camera app object pr

Missed a bit before but just fixed it so r? again
Attachment #8441948 - Flags: review?(viorela.ioia)
Comment on attachment 8441951 [details] [review]
Wallpaper app/test pr

Nice and clean
Attachment #8441951 - Flags: review?(florin.strugariu) → review+
Comment on attachment 8441948 [details] [review]
Camera app object pr

Lgtm
Attachment #8441948 - Flags: review?(viorela.ioia) → review+
Attached file github pr (obsolete) —
Just 1 more after this!
Attachment #8445150 - Flags: review?(florin.strugariu)
Attachment #8445150 - Flags: review?(florin.strugariu) → review+
Attached file github pr (obsolete) —
Attachment #8446418 - Flags: review?(florin.strugariu)
Comment on attachment 8446418 [details] [review]
github pr

Clean as always
Attachment #8446418 - Flags: review?(florin.strugariu) → review+
Thanks Bebe,
we can close this now!
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.