Closed Bug 1546761 Opened 2 years ago Closed 2 years ago

Consolidate IPC calls to nsISHistory inside of nsDocShell::OnNewURI

Categories

(Core :: DOM: Navigation, task, P2)

Other Branch
task

Tracking

()

RESOLVED FIXED
Fission Milestone M3

People

(Reporter: annyG, Assigned: annyG)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

We would like to consolidate 4 IPC calls in the following code bit[1] into 1 IPC call.

[1] https://searchfox.org/mozilla-central/rev/f46e2bf881d522a440b30cbf5cf8d76fc212eaf4/docshell/base/nsDocShell.cpp#10764-10775

Fission Milestone: --- → M3

Inside of nsDocShell::OnNewURI there are 4 sync IPC calls
to nsSHistory that can be replaced with 1 sync IPC call
by adding a new method EnsureCorrectEntryAtCurrIndex to nsSHistory.

Status: NEW → ASSIGNED
Type: enhancement → task
Priority: -- → P2
Version: unspecified → Other Branch
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Pushed by pvanderbeken@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ed6458f39849
Consolidate IPC calls to nsISHistory inside of nsDocShell::OnNewURI, r=peterv, r=nika for adding sync IPC messages
You need to log in before you can comment on or make changes to this bug.