Bug 1943952 - update uses of sessionStorage and localStorage in core to be wrapped in a try/catch

[jeddai]

There are no changes to functionality for clients, so there are similarly no changes to tests or documentation.

If we feel it would be imperative to add tests to cover these cases, please let me know!

Pull Request checklist

• Quality: Make sure this PR builds and runs cleanly.
  • Inside the glean/ folder, run:
    • npm run test Runs all tests
    • npm run lint Runs all linters
• Tests: This PR includes thorough tests or an explanation of why it does not
• Changelog: This PR includes a changelog entry to CHANGELOG.md or an explanation of why it does not need one
• Documentation: This PR includes documentation changes, an explanation of why it does not need that or a follow-up bug has been filed to do that work
  • Documentation should be added to The Glean Book when making changes to Glean's user facing API
    • When changes to the Glean Book are necessary, link to the corresponding PR on the mozilla/glean repository
  • Documentation should be added to the Glean.js developer documentation when making internal code changes

[badboy]

No tests is fine, the try/catch is explicitly very narrow around the calls that can throw.