Closed Bug 1277705 Opened 8 years ago Closed 8 years ago

Remove more Chromium IPC code

Categories

(Core :: IPC, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla50
Tracking Status
firefox50 --- fixed

People

(Reporter: billm, Assigned: billm)

References

Details

(Whiteboard: btpp-active)

Attachments

(10 files)

(deleted), patch
dvander
: review+
Details | Diff | Splinter Review
(deleted), patch
dvander
: review+
Details | Diff | Splinter Review
(deleted), patch
dvander
: review+
Details | Diff | Splinter Review
(deleted), patch
dvander
: review+
Details | Diff | Splinter Review
(deleted), patch
dvander
: review+
Details | Diff | Splinter Review
(deleted), patch
dvander
: review+
Details | Diff | Splinter Review
(deleted), patch
dvander
: review+
Details | Diff | Splinter Review
(deleted), patch
dvander
: review+
Details | Diff | Splinter Review
(deleted), patch
dvander
: review+
Details | Diff | Splinter Review
(deleted), patch
dvander
: review+
Details | Diff | Splinter Review
Lots of process-related stuff can be eliminated.
It's unused.
Attachment #8759404 - Flags: review?(dvander)
Attached patch remove notification service (deleted) — Splinter Review
No one actually listens to these notifications.
Attachment #8759406 - Flags: review?(dvander)
Attached patch remove waitable event watcher (deleted) — Splinter Review
The purpose of all this code is to call OnWaitableEventSignaled at the right time. But all the implementations of these methods are no-ops.
Attachment #8759407 - Flags: review?(dvander)
No one uses it.
Attachment #8759408 - Flags: review?(dvander)
We only send messages on a channel, so this interface is pointless.
Attachment #8759409 - Flags: review?(dvander)
Attached patch remove child_process_info (deleted) — Splinter Review
No one uses this info.
Attachment #8759410 - Flags: review?(dvander)
These are unused now that the notification service is gone.
Attachment #8759411 - Flags: review?(dvander)
Attached patch remove result codes (deleted) — Splinter Review
Only one of these is ever used.
Attachment #8759413 - Flags: review?(dvander)
Attached patch simplify child_thread (deleted) — Splinter Review
More unused stuff here.
Attachment #8759414 - Flags: review?(dvander)
Attachment #8759404 - Flags: review?(dvander) → review+
Attachment #8759406 - Flags: review?(dvander) → review+
Attachment #8759407 - Flags: review?(dvander) → review+
Comment on attachment 8759408 [details] [diff] [review] remove child process host iterator Review of attachment 8759408 [details] [diff] [review]: ----------------------------------------------------------------- I was wondering what this was for.
Attachment #8759408 - Flags: review?(dvander) → review+
Attachment #8759409 - Flags: review?(dvander) → review+
Attachment #8759410 - Flags: review?(dvander) → review+
Attachment #8759411 - Flags: review?(dvander) → review+
Attachment #8759412 - Flags: review?(dvander) → review+
Attachment #8759413 - Flags: review?(dvander) → review+
Attachment #8759414 - Flags: review?(dvander) → review+
Whiteboard: btpp-active
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: