Closed Bug 1480893 Opened 6 years ago Closed 5 years ago

Consider making the browsing context ID a struct rather than an integer

Categories

(Core :: DOM: Core & HTML, task, P3)

task

Tracking

()

RESOLVED WONTFIX
Fission Milestone M4

People

(Reporter: nika, Unassigned)

References

(Blocks 1 open bug)

Details

This would make it more clear what the object was, as well as giving easy access to the process which created the object, without changing the size at all. If we make it a IPDL struct, we even get serialization :-). For example, consider:

```
struct BrowsingContextId {
  uint32_t mOrigProcId;
  uint32_t mLocalId;
};
```

In the parent process, we could check when receiving an ID for the first time that the mOrigProcId field actually matched the ID of the process which sent us the ID, so that compromised processes couldn't stomp on other processes IDs.

We can even check that the process which sends us an ID actually knows about each ID which it didn't create itself, otherwise it shouldn't be able to send it to us.
Priority: -- → P3
Fission Milestone: --- → M3
Blocks: improve-bc
No longer blocks: browsingcontext
Component: DOM → DOM: Core & HTML
Type: enhancement → task
Fission Milestone: M3 → M4

We've decided we won't do this, especially because frontend will need it to be an integer.

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.