From 350663aa419b93a33ce1bf79528dbe49e61240eb Mon Sep 17 00:00:00 2001 From: Alan Orozco Date: Fri, 6 Nov 2020 15:08:01 -0600 Subject: [PATCH] =?UTF-8?q?=F0=9F=9A=AE=20Remove=20iOS=20specific=20unpaus?= =?UTF-8?q?able=20workaround=20(#31035)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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. --- css/video-autoplay.css | 6 ------ src/service/video/autoplay.js | 6 ------ 2 files changed, 12 deletions(-) diff --git a/css/video-autoplay.css b/css/video-autoplay.css index 623aa98f8f5b..b3f0455f2fe5 100644 --- a/css/video-autoplay.css +++ b/css/video-autoplay.css @@ -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; } diff --git a/src/service/video/autoplay.js b/src/service/video/autoplay.js index 34fd47b72000..363be0f37456 100644 --- a/src/service/video/autoplay.js +++ b/src/service/video/autoplay.js @@ -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'; @@ -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; }