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
Revert "Make <amp-install-service-worker> a low priority element, and… #14293
Revert "Make <amp-install-service-worker> a low priority element, and… #14293
Conversation
… remove timer-based delay to installation (ampproject#13409)" This reverts commit 7a8be5f.
// introduced to avoid installing SWs for content that the user | ||
// only engaged with superficially. | ||
Services.timerFor(this.win).delay(() => { | ||
this.deferMutate(this.insertIframe_.bind(this)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jridgewell removed deferMutate, make sure you merge the changes from here into this revert: c4ef869#diff-d7bcc789dc4ca07564072200b4451bb3
expect(iframe.getAttribute('sandbox')).to.equal( | ||
'allow-same-origin allow-scripts'); | ||
let deferredMutate; | ||
implementation.deferMutate = fn => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above: c4ef869#diff-c7e92f692652c6ad5db6167bf7608b8e
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, you can move anything relating to LayoutPriority because it's no longer part of this extension
… remove timer-based delay to installation (#13409)"
This reverts commit 7a8be5f.
Fixes #14288 and reopens #12473.