Closed Bug 974989 Opened 6 years ago Closed 5 years ago

Add an ipc channel message loop for winrt

Categories

(Core :: IPC, defect, P1)

x86_64
Windows 8.1
defect

Tracking

()

RESOLVED WONTFIX

People

(Reporter: jimm, Unassigned)

References

Details

(Whiteboard: p=2)

Attachments

(1 file, 8 obsolete files)

Land just some basics, pref'd off so we can start working on this.

todo:

- gecko thread creation and registration
- simple test protocols
- thread check helpers
Blocks: 911133
Hmm, unfortunately I just realized this can't be tied to a pref since we need to know about the switch before gecko is initialized.
Attached patch wip (obsolete) — Splinter Review
Attached patch wip (obsolete) — Splinter Review
Attachment #8379304 - Attachment is obsolete: true
Depends on: 974901
Attached patch part 1- message loop (obsolete) — Splinter Review
Attachment #8379314 - Attachment is obsolete: true
Blocks: 975452
No longer blocks: 911133
Fixed some build errors when building with this patch set and the updated patch in bug 974901.
Attachment #8379813 - Attachment is obsolete: true
Attachment #8380203 - Attachment is obsolete: true
Moving the winrt work I was doing here up to bug 975452.
Summary: Land basic input thread separation framework pref'd off → Add an ipc channel message loop for winrt
Attached patch ipc patch (obsolete) — Splinter Review
Attachment #8379812 - Attachment is obsolete: true
Attached patch ipc patch (obsolete) — Splinter Review
Attachment #8382463 - Attachment is obsolete: true
Attached patch IPC Patch v2 (obsolete) — Splinter Review
Updated to apply cleanly when starting from this:
  170933:4cfb6c61b137
  https://bug975452.bugzilla.mozilla.org/attachment.cgi?id=8383035
  https://bug975452.bugzilla.mozilla.org/attachment.cgi?id=8383037
Attachment #8383033 - Attachment is obsolete: true
Comment on attachment 8383168 [details] [diff] [review]
IPC Patch v2

This doesn't compile. Also the previous comment was incorrect; the patch applies cleanly on top of updated versions of the patches listed that I haven't posted yet.

Will fix compilation issues and re-submit.
Attachment #8383168 - Attachment is obsolete: true
Priority: -- → P1
Component: Widget: WinRT → IPC
Assignee: jmathies → nobody
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.