Page MenuHomePhabricator

VE educational popup should try/catch localStorage calls
Closed, ResolvedPublic1 Estimated Story Points

Description

I configured my Chrome preferences to not accept any sort of local storage and discovered that one of the tools makes calls on localStorage, without try/catch guarding.

Uncaught DOMException: Failed to read the 'localStorage' property from 'Window': Access is denied for this document.
    at VeUiMwLinkInspectorTool.VeUiMwEducationPopupTool

Event Timeline

Jdforrester-WMF moved this task from To Triage to TR0: Interrupt on the VisualEditor board.
Jdforrester-WMF set the point value for this task to 1.

Change 311834 had a related patch set uploaded (by Alex Monk):
Education popup: Put localStorage check inside try/catch

https://gerrit.wikimedia.org/r/311834

Change 311834 merged by jenkins-bot:
Education popup: Put localStorage check inside try/catch

https://gerrit.wikimedia.org/r/311834