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

Updating SafeFrame version. #11519

Merged
merged 1 commit into from
Oct 2, 2017
Merged

Updating SafeFrame version. #11519

merged 1 commit into from
Oct 2, 2017

Conversation

glevitzky
Copy link
Contributor

@glevitzky glevitzky commented Oct 2, 2017

Updates the default SafeFrame version passed to each Fast Fetch ad request.

Without this update, Fluid ads will not render at all because they will be requesting an old version of SafeFrame which is incompatible with Fluid.

@keithwrightbos keithwrightbos merged commit b2e8473 into ampproject:master Oct 2, 2017
dreamofabear pushed a commit that referenced this pull request Oct 2, 2017
dreamofabear pushed a commit to dreamofabear/amphtml that referenced this pull request Oct 2, 2017
dreamofabear pushed a commit that referenced this pull request Oct 2, 2017
* Updating SafeFrame version. (#11519)

* Explicitly report past Performance Timineline values. (#11521)

Chrome's implementation does not yet support reporting on past values.
See
https://bugs.chromium.org/p/chromium/issues/detail?id=725567

The good news is that it appears as if we have only reported on fast loads so far :)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants