Closed Bug 1668026 Opened 4 years ago Closed 4 years ago

Update child process index and length when modifying those in parent process

Categories

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

defect

Tracking

()

RESOLVED FIXED
83 Branch
Fission Milestone M6b
Tracking Status
firefox83 --- fixed

People

(Reporter: smaug, Assigned: smaug)

References

Details

Attachments

(1 file)

nsISHistory.purgeHistory() for example modifies the session history in parent process, but index and length aren't updated to child process in that case.
At least browser/base/content/test/sanitize/browser_purgehistory_clears_sh.js is testing this.

The patch is utilizing SHistoryChangeNotifier which was added for testing.
CallerWillNotifyHistoryIndexAndLengthChanges is needed to avoid extra index/length updates so that
the child side doesn't get update but have still pending updates in it.

Severity: -- → S2
Status: NEW → ASSIGNED
Fission Milestone: --- → M6b
Priority: -- → P2
Pushed by opettay@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/85126aab8664
Update child process index and length when modifying those in parent process, r=peterv
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 83 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: