1972065 - Add missing session data types for viewport and geolocation overrides

Status: RESOLVED FIXED

(Remote Protocol :: Agent, task, P3)

Description

[Julian Descottes [:jdescottes]]

We should normally declare new Session Data types in https://searchfox.org/mozilla-central/rev/99f02d7a9456d4cfe2abd9c2a4a1eb8f3f41e22a/remote/shared/messagehandler/sessiondata/SessionData.sys.mjs#29-32

export const SessionDataCategory = {
  Event: "event",
  PreloadScript: "preload-script",
};

but we haven't done that for viewport and geolocation overrides, and we also aren't using it explicitly for preload scripts: https://searchfox.org/mozilla-central/search?q=category%3A+%22&path=remote%2F**%2Fmodules&case=false&regexp=false

Comment 1

[Khalid AlHaddad]

Attachment: Bug 1972065 - Add new categories to SessionData module to be used instead of hardcoded strings. r?jdescottes

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/adeb257729ee
https://hg.mozilla.org/integration/autoland/rev/f58f6dd58b91
Add new categories to SessionData module to be used instead of hardcoded strings. r=jdescottes

Comment 3

[agoloman]

https://hg.mozilla.org/mozilla-central/rev/f58f6dd58b91