Closed Bug 1549168 Opened 6 years ago Closed 6 years ago

Remove use of principal SerializationHelper and replace with E10S.serializePrincipal

Categories

(Core :: DOM: Security, enhancement, P1)

65 Branch
enhancement

Tracking

()

RESOLVED FIXED
mozilla68
Tracking Status
firefox68 --- fixed

People

(Reporter: jkt, Assigned: jkt)

References

Details

(Whiteboard: [domsecurity-active])

Attachments

(1 file)

As part of Bug 1508939 there are some places in the code that are relying on a hard coded serializationHelper. As we plan to move how principals are serialized this should happen first so that the Write method can be removed from principals.

  • browser/base/content/browser.js
  • browser/base/content/pageinfo/pageInfo.js
  • browser/base/content/test/pageinfo/browser_pageinfo_firstPartyIsolation.js
  • browser/components/sessionstore/test/browser_old_favicon.js
  • dom/base/nsContentUtils.cpp

Due to the cpp changes I think we can only make the browser.js changes here.

Whiteboard: [domsecurity-active]
Pushed by jkingston@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/5fe8a5adca36 Remove use of principal SerializationHelper in browser.js and replace with E10S.serializePrincipal. r=ckerschb
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: