Skip to content

Commit

Permalink
🚮 Remove iOS specific unpausable workaround (#31035)
Browse files Browse the repository at this point in the history
1. The iOS issue no longer applies to our support matrix.
2. Even if it did, after the `IntersectionObserver` change, videos playing while visible to 0% so we rarely see this state.
  • Loading branch information
alanorozco committed Nov 6, 2020
1 parent 3e391e6 commit 350663a
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 12 deletions.
6 changes: 0 additions & 6 deletions css/video-autoplay.css
Expand Up @@ -77,12 +77,6 @@ amp-story .amp-video-eq {
.amp-video-eq-col div {
animation-play-state: paused;
}
[unpausable] .amp-video-eq-col div {
animation-name: none;
}
[unpausable].amp-video-eq-play .amp-video-eq-col div {
animation-name: amp-video-eq-animation;
}
.amp-video-eq-play .amp-video-eq-col div {
animation-play-state: running;
}
Expand Down
6 changes: 0 additions & 6 deletions src/service/video/autoplay.js
Expand Up @@ -14,7 +14,6 @@
* limitations under the License.
*/

import {Services} from '../../services';
import {dev} from '../../log';
import {htmlFor} from '../../static-template';
import {removeElement} from '../../dom';
Expand Down Expand Up @@ -70,10 +69,5 @@ export function renderIcon(win, elOrDoc) {
// Remove seed column.
removeElement(firstCol);

if (Services.platformFor(win).isIos()) {
// iOS is unable to pause hardware accelerated animations.
icon.setAttribute('unpausable', '');
}

return icon;
}

0 comments on commit 350663a

Please sign in to comment.