Closed Bug 834263 Opened 7 years ago Closed 7 years ago

consolidate ClonedMessageData unpacking code

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla21

People

(Reporter: froydnj, Assigned: froydnj)

Details

Attachments

(1 file)

No description provided.
This is the counterpart of bug 834106, only on the receiving side.  Unfortunately,
these have to be free functions, because all the callsites don't have visibility
on MessageManagerCallback's methods.  nsFrameMessageManager.h seemed like a
reasonable place to put them, given what we already have.

The explicit spelling out of mozilla::dom::ipc::UnpackClonedMessageDataFor* seems
to be necessary to satisfy the linker.

There's still some cleanup opportunities between here and the IndexedDB code.
Attachment #705877 - Flags: review?(bugs)
Comment on attachment 705877 [details] [diff] [review]
consolidate ClonedMessageData unpacking code


>+    for (uint32_t index = 0; index < length; index++) {
++index please
Attachment #705877 - Flags: review?(bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/cc0081ab6c4a
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
Assignee: nobody → nfroyd
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.