Windows widget AudioSession::Start not working in Metro

RESOLVED FIXED in mozilla17

Status

()

Core
Widget: Win32
RESOLVED FIXED
5 years ago
3 years ago

People

(Reporter: bbondy, Assigned: bbondy)

Tracking

Trunk
mozilla17
x86_64
Windows 8.1
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: completed-elm)

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
AudioSession::Start is failing on Metro because COM is already initialized and returning an error.
(Assignee)

Comment 1

5 years ago
Created attachment 650670 [details] [diff] [review]
Patch v1.

This is part of the stuff needed to get flash working in Metro but I separated this out since it can land on m-c.  CoInitialize is returning S_FALSE which causes the audio session to stay in an uninitialized state.

The other small change isn't needed but is more correct since the assertion is ignored in release mode and it would lead to a crash if uuidgen was ever not available.
Attachment #650670 - Flags: review?(jmathies)
(Assignee)

Comment 2

5 years ago
http://hg.mozilla.org/projects/elm/rev/0f5f696f4f46
Whiteboard: completed-elm
(Assignee)

Updated

5 years ago
Blocks: 781653
(Assignee)

Updated

5 years ago
Blocks: 781649

Updated

5 years ago
Attachment #650670 - Flags: review?(jmathies) → review+
(Assignee)

Comment 3

5 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/edb166cc85b9
https://hg.mozilla.org/mozilla-central/rev/edb166cc85b9
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.