fix: Uncaught session storage security error handling

[jooohhn]

Summary

• Fixes sessionStorage introduced in feat(privacy): Add a storage option #320
  • Reason is that storage APIs throw errors when cookies are disabled in certain browsers
• Change custom localStorage name to ampLocalStorage to remove confusion
• Closes Uncaught SecurityError: Failed to read the 'sessionStorage' property from 'Window #343

Checklist

• Does your PR title have the correct title format?
• Does your PR have a breaking change?: NO

[kelvin-lu]

is there the risk that localstorage has the same permissioning issues?

[jooohhn]

No because we have a localStorage.js file that manages error handling

I was confused by our naming of it and thought its usage in metadata-storage.js referred to the actual browser API so I changed it to ampLocalStorage

[kelvin-lu]

Super cool. I wonder if it makes sense to put session storage with some compat flags like local storage, but not a blocker. I feel like I don't understand the session storage api quite well enough to say here.