IPC VolumeManager on Gonk

RESOLVED FIXED in mozilla17

Status

RESOLVED FIXED
7 years ago
3 months ago

People

(Reporter: dougt, Assigned: dougt)

Tracking

Trunk
mozilla17
x86
Mac OS X

Firefox Tracking Flags

(blocking-basecamp:+)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

7 years ago
Created attachment 649370 [details] [diff] [review]
patch v.1

This patch forwards notifications from the volume manager to the child process.
It also allow you to synchronously learn about the status of a give named volume.

David and I both touched this patch.  Kyle, can you look at this?
Attachment #649370 - Flags: review?(kyle)
(Assignee)

Updated

7 years ago
Blocks: 780691
(Assignee)

Comment 1

7 years ago
Comment on attachment 649370 [details] [diff] [review]
patch v.1

not the right patch.
Attachment #649370 - Flags: review?(kyle) → review-
(Assignee)

Comment 2

7 years ago
Created attachment 649409 [details] [diff] [review]
patch v.1
Attachment #649370 - Attachment is obsolete: true
Attachment #649409 - Flags: review?(bent.mozilla)
blocking-basecamp: --- → +
(Assignee)

Comment 3

7 years ago
Created attachment 650678 [details] [diff] [review]
patch v.2

without that sync method
Attachment #649409 - Attachment is obsolete: true
Attachment #649409 - Flags: review?(bent.mozilla)
Attachment #650678 - Flags: review?(bent.mozilla)
Comment on attachment 650678 [details] [diff] [review]
patch v.2

Review of attachment 650678 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/system/gonk/Makefile.in
@@ +69,5 @@
>    $(NULL)
>  # for our local copy of AudioSystem.h
>  LOCAL_INCLUDES += -I$(topsrcdir)/media/libsydneyaudio/src
> +
> +EXPORTS     = \

Nit: too much whitespace there.
Attachment #650678 - Flags: review?(bent.mozilla) → review+
I rearranged the ContentChild constructor to fix warnings-as-errors:

https://hg.mozilla.org/integration/mozilla-inbound/rev/14c4b37f9a0e
https://hg.mozilla.org/mozilla-central/rev/5334f92a9f34
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17

Updated

3 months ago
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.