Skip to content

Fix an amp-instagram race condition#8192

Merged
aghassemi merged 2 commits intoampproject:masterfrom
aghassemi:instarace
Mar 27, 2017
Merged

Fix an amp-instagram race condition#8192
aghassemi merged 2 commits intoampproject:masterfrom
aghassemi:instarace

Conversation

@aghassemi
Copy link
Copy Markdown
Contributor

Fixing a small race condition where vsync ran after component was unlayedout (setting this.iframe_ to null)

@ampprojectbot
Copy link
Copy Markdown
Member

Hi, ampproject bot here! Here are a list of the owners that can approve your files.

You may leave an issue comment stating "@ampprojectbot retry!" to force me to re-evaluate this Pull Request's status

/to cramforce dvoytenko jridgewell

  • extensions/amp-instagram/0.1/amp-instagram.js

For any issues please file a bug at https://github.com/google/github-owners-bot/issues

@aghassemi aghassemi requested a review from camelburrito March 16, 2017 19:52
@aghassemi aghassemi merged commit 459ab07 into ampproject:master Mar 27, 2017
lironzluf pushed a commit to lironzluf/amphtml that referenced this pull request Mar 28, 2017
* master: (34 commits)
  Prevent amp-carousel next/previous icons fade away on desktop (ampproject#8428)
  Turn on flag slidescroll-disable-css-snap” (ampproject#8436)
  Revert "temporarily turn off yarn (ampproject#8356)" (ampproject#8384)
  initial commit (ampproject#8404)
  Upgrades for Index Exchange amp-ad tags to report load statistics (ampproject#8054)
  amp-bind validation tweak (ampproject#8414)
  Fix an amp-instagram race condition (ampproject#8192)
  Use whitelist to restrict urlReplacement for scoped analytics element (ampproject#8360)
  Report active experiments in error logs (ampproject#8108)
  amp-bind: Catch exceptions in mutatedAttributesCallback (ampproject#8383)
  Fixing custom scroll-snap on IOS (ampproject#8391)
  Add experiment for using AmpContext class in integration.js (ampproject#8348)
  add (ampproject#8349)
  swipe api (ampproject#8357)
  skip 3 flaky tests (ampproject#8388)
  amp-bind: Expression complexity limit (ampproject#8321)
  add margin-bottom (ampproject#8350)
  Flying carpet: make container full viewport and center content (ampproject#8292)
  Service Registration: Document Click (ampproject#7882)
  Add a8ad (ampproject#8036)
  ...
mrjoro pushed a commit to mrjoro/amphtml that referenced this pull request Apr 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants