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
A4a 3p remove load from init #5936
Conversation
The previous change was made in error (wrong branch).
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.
LGTM
@@ -124,23 +132,17 @@ export class AmpAdXOriginIframeHandler { | |||
// post message depending on if A4A flow. |
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.
let's remove the comment here.
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.
Done. Please merge and thanks!
* Update amp-ad-network-doubleclick-impl.md * Reverting doc changes The previous change was made in error (wrong branch). * Remove load listener from AmpAdXOriginIframeHandler.init * Correct conflict * PR feedback
* Update amp-ad-network-doubleclick-impl.md * Reverting doc changes The previous change was made in error (wrong branch). * Remove load listener from AmpAdXOriginIframeHandler.init * Correct conflict * PR feedback
No longer require load event to remove visibility hidden from frame (in fact A4A flow never sets visibility hidden)