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
Fix friendly embedding on amp-ad, and reenable test-amp-ad-fake.js #22504
Fix friendly embedding on amp-ad, and reenable test-amp-ad-fake.js #22504
Conversation
redirecting to @lannka |
add @rsimha for reviewing as it might be related to the recent closure compiler change |
The Travis run still shows flakiness on Firefox.
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.
One comment below.
/cc @calebcordry @lannka
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.
Approving bundle size increase. In this case, it's a good sign, since we were missing some runtime code earlier.
…mpproject#22504) * Reenable test * Bandaid to fix installInEmbedWindow failing in compiled local testing. * Add nocollapse to allowed tags * Skip test on Firefox (like before the change). The Travis run still shows flakiness on Firefox. * Unskip the remaining test
See #22572. Seems like it's now failing due to a regression (function being stripped from compiled code).