IPC VolumeManager on Gonk

RESOLVED FIXED in mozilla17

Status

()

Core
Widget: Gonk
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: dougt, Assigned: dougt)

Tracking

Trunk
mozilla17
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(blocking-basecamp:+)

Details

Attachments

(1 attachment, 2 obsolete attachments)

11.28 KB, patch
Ben Turner (not reading bugmail, use the needinfo flag!)
: review+
Details | Diff | Splinter Review
(Assignee)

Description

5 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

5 years ago
Blocks: 780691
(Assignee)

Comment 1

5 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

5 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

5 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+
(Assignee)

Comment 5

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/5334f92a9f34

Comment 6

5 years ago
I rearranged the ContentChild constructor to fix warnings-as-errors:

https://hg.mozilla.org/integration/mozilla-inbound/rev/14c4b37f9a0e

Comment 7

5 years ago
https://hg.mozilla.org/mozilla-central/rev/5334f92a9f34
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
And the follow-up:
https://hg.mozilla.org/mozilla-central/rev/14c4b37f9a0e
You need to log in before you can comment on or make changes to this bug.