Closed Bug 849015 Opened 8 years ago Closed 3 years ago

ContextUI.dismiss is async and should return a promise

Categories

(Firefox for Metro Graveyard :: App Bar, defect)

x86
Windows 8.1
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: sfoster, Unassigned)

References

Details

(Whiteboard: p=0)

Attachments

(1 file)

The css transition trigged by the presence/absence of the visible attribute on the #tray element causes dimiss to be async - the element is not its final height and reflow doesn't happen until later. If it returned a promise, tests (and possibly other code) could more reliably wait for the right time to run.
Blocks: 834370
Summary: ContextUI.dimiss is async and should return a promise → ContextUI.dismiss is async and should return a promise
Assignee: nobody → jmathies
OS: Windows 8 → Windows 8 Metro
Attached patch patchSplinter Review
Put this together in case it's needed. Now that we have yield hideContextUI() it may not be necessary.
No longer blocks: 834370
Assignee: jmathies → nobody
No longer blocks: 888919
Blocks: metrobacklog
Whiteboard: p=0
OS: Windows 8 Metro → Windows 8.1
Mass close of bugs in obsolete product https://bugzilla.mozilla.org/show_bug.cgi?id=1350354
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.