Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error: Failed to read the 'sessionStorage' property from 'Window': Access is denied for this document. #3797

Closed
zhouyx opened this issue Jun 27, 2016 · 7 comments
Assignees

Comments

@zhouyx
Copy link
Contributor

zhouyx commented Jun 27, 2016

at Error (native)
at D (https://cdn.ampproject.org/rtv/001466185787499/v0/amp-accordion-0.1.js:12:287)
at B.buildCallback (https://cdn.ampproject.org/rtv/001466185787499/v0/amp-accordion-0.1.js:10:178)
at amp-accordion.d.build (https://cdn.ampproject.org/r
@zhouyx
Copy link
Contributor Author

zhouyx commented Jun 27, 2016

from amp-accordion.js line 102.
try {
const sessionStr = this.win_./*OK*/sessionStorage.getItem(this.id_);
return JSON.parse(sessionStr);
}

@erwinmombay
Copy link
Member

@zhouyx can you mark one as dupe and close the other one? we don't need to keep both

@zhouyx
Copy link
Contributor Author

zhouyx commented Jun 28, 2016

close #3770

@jridgewell
Copy link
Contributor

We already have this in a try-catch, this is expected.

@rudygalfi rudygalfi added this to the Current milestone Jun 29, 2016
@rudygalfi
Copy link
Contributor

Putting in Current — assume this is in progress?

@zhouyx
Copy link
Contributor Author

zhouyx commented Jun 30, 2016

#3813

@jridgewell
Copy link
Contributor

Closed by #5968.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants