Attachment 362721 Details for Bug 194899 – Patch

[patch] Patch

bug-194899-20190222162753.patch (text/plain), 21.07 KB, created by Miguel Gomez on 2019-02-22 07:27:55 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Miguel Gomez

Created: 2019-02-22 07:27:55 PST

Size: 21.07 KB

patch

obsolete

>Subversion Revision: 241945
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index c7b81ce05f6067872c43d516124eac69ff6bf398..f0739c879bf3b076a80898923f465cdd6bf139c6 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,38 @@
>+2019-02-22  Miguel Gomez  <magomez@igalia.com>
>+
>+        [WPE] Add support for holepunch using an external video player
>+        https://bugs.webkit.org/show_bug.cgi?id=194899
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Implement the holepunch feature to allow playback using an external player. This creates
>+        a new dummy MediaPlayerPrivateHolePunch that accepts only the type "video/holepunch", and
>+        whose goal is to just draw a transparent rectangle in the position where the video should be.
>+        This can be used to allow a player placed on a lower plane than the browser to become visible.
>+
>+        Added ManualTest wpe/video-player-holepunch-external.html to test the feature.
>+
>+        * PlatformWPE.cmake:
>+        * platform/HolePunch.cmake: Added.
>+        * platform/graphics/MediaPlayer.cpp:
>+        (WebCore::buildMediaEnginesVector):
>+        * platform/graphics/holepunch/MediaPlayerPrivateHolePunch.cpp: Added.
>+        (WebCore::MediaPlayerPrivateHolePunch::MediaPlayerPrivateHolePunch):
>+        (WebCore::MediaPlayerPrivateHolePunch::~MediaPlayerPrivateHolePunch):
>+        (WebCore::MediaPlayerPrivateHolePunch::platformLayer const):
>+        (WebCore::MediaPlayerPrivateHolePunch::naturalSize const):
>+        (WebCore::MediaPlayerPrivateHolePunch::pushNextHolePunchBuffer):
>+        (WebCore::MediaPlayerPrivateHolePunch::swapBuffersIfNeeded):
>+        (WebCore::MediaPlayerPrivateHolePunch::proxy const):
>+        (WebCore::mimeTypeCache):
>+        (WebCore::MediaPlayerPrivateHolePunch::getSupportedTypes):
>+        (WebCore::MediaPlayerPrivateHolePunch::supportsType):
>+        (WebCore::MediaPlayerPrivateHolePunch::registerMediaEngine):
>+        (WebCore::MediaPlayerPrivateHolePunch::notifyReadyState):
>+        * platform/graphics/holepunch/MediaPlayerPrivateHolePunch.h: Added.
>+        * platform/graphics/texmap/TextureMapperPlatformLayerBuffer.cpp:
>+        (WebCore::TextureMapperPlatformLayerBuffer::paintToTextureMapper):
>+
> 2019-02-22  Eric Carlson  <eric.carlson@apple.com>
> 
>         Update some media logging
>diff --git a/Source/WebCore/PlatformWPE.cmake b/Source/WebCore/PlatformWPE.cmake
>index 54b12e9fa0f9689c5c3386e49afc557fe5c22edb..143d1348bcd5517add906af741f4cde84e068c75 100644
>--- a/Source/WebCore/PlatformWPE.cmake
>+++ b/Source/WebCore/PlatformWPE.cmake
>@@ -5,6 +5,10 @@ include(platform/GStreamer.cmake)
> include(platform/ImageDecoders.cmake)
> include(platform/TextureMapper.cmake)
> 
>+if (USE_EXTERNAL_HOLEPUNCH)
>+    include(platform/HolePunch.cmake)
>+endif ()
>+
> list(APPEND WebCore_UNIFIED_SOURCE_LIST_FILES
>     "SourcesWPE.txt"
> 
>diff --git a/Source/WebCore/platform/HolePunch.cmake b/Source/WebCore/platform/HolePunch.cmake
>new file mode 100644
>index 0000000000000000000000000000000000000000..4ac1631435f86d1738032189797629b7d3698859
>--- /dev/null
>+++ b/Source/WebCore/platform/HolePunch.cmake
>@@ -0,0 +1,6 @@
>+list(APPEND WebCore_INCLUDE_DIRECTORIES
>+    "${WEBCORE_DIR}/platform/graphics/holepunch"
>+)
>+list(APPEND WebCore_SOURCES
>+    platform/graphics/holepunch/MediaPlayerPrivateHolePunch.cpp
>+)
>diff --git a/Source/WebCore/platform/graphics/MediaPlayer.cpp b/Source/WebCore/platform/graphics/MediaPlayer.cpp
>index bda0711f6b55c26d48af638cc8b2f9a578eb778e..8bff2d7f074d1d03a22e4301cfa56644ffbdc77a 100644
>--- a/Source/WebCore/platform/graphics/MediaPlayer.cpp
>+++ b/Source/WebCore/platform/graphics/MediaPlayer.cpp
>@@ -84,6 +84,10 @@
> #include "MediaPlayerPrivateAVFoundationCF.h"
> #endif
> 
>+#if USE(EXTERNAL_HOLEPUNCH)
>+#include "MediaPlayerPrivateHolePunch.h"
>+#endif
>+
> namespace WebCore {
> 
> #if !RELEASE_LOG_DISABLED
>@@ -255,6 +259,10 @@ static void buildMediaEnginesVector()
>         MediaPlayerPrivateGStreamerMSE::registerMediaEngine(addMediaEngine);
> #endif
> 
>+#if USE(EXTERNAL_HOLEPUNCH)
>+    MediaPlayerPrivateHolePunch::registerMediaEngine(addMediaEngine);
>+#endif
>+
>     haveMediaEnginesVector() = true;
> }
> 
>diff --git a/Source/WebCore/platform/graphics/holepunch/MediaPlayerPrivateHolePunch.cpp b/Source/WebCore/platform/graphics/holepunch/MediaPlayerPrivateHolePunch.cpp
>new file mode 100644
>index 0000000000000000000000000000000000000000..1172458410e9a1453b635b0f8d91b9fb4a696c58
>--- /dev/null
>+++ b/Source/WebCore/platform/graphics/holepunch/MediaPlayerPrivateHolePunch.cpp
>@@ -0,0 +1,154 @@
>+/*
>+ * Copyright (C) 2019 Igalia S.L
>+ * Copyright (C) 2019 Metrological Group B.V.
>+ *
>+ * This library is free software; you can redistribute it and/or
>+ * modify it under the terms of the GNU Library General Public
>+ * License as published by the Free Software Foundation; either
>+ * version 2 of the License, or (at your option) any later version.
>+ *
>+ * This library is distributed in the hope that it will be useful,
>+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
>+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
>+ * Library General Public License for more details.
>+ *
>+ * You should have received a copy of the GNU Library General Public License
>+ * aint with this library; see the file COPYING.LIB.  If not, write to
>+ * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
>+ * Boston, MA 02110-1301, USA.
>+ */
>+
>+#include "config.h"
>+#include "MediaPlayerPrivateHolePunch.h"
>+
>+#if USE(EXTERNAL_HOLEPUNCH)
>+#include "MediaPlayer.h"
>+#include "TextureMapperPlatformLayerBuffer.h"
>+#include "TextureMapperPlatformLayerProxy.h"
>+
>+namespace WebCore {
>+
>+static const FloatSize s_holePunchDefaultFrameSize(1280, 720);
>+
>+MediaPlayerPrivateHolePunch::MediaPlayerPrivateHolePunch(MediaPlayer* player)
>+    : m_player(player)
>+    , m_readyTimer(RunLoop::main(), this, &MediaPlayerPrivateHolePunch::notifyReadyState)
>+#if USE(NICOSIA)
>+    , m_nicosiaLayer(Nicosia::ContentLayer::create(Nicosia::ContentLayerTextureMapperImpl::createFactory(*this)))
>+#else
>+    , m_platformLayerProxy(adoptRef(new TextureMapperPlatformLayerProxy()))
>+#endif
>+{
>+    pushNextHolePunchBuffer();
>+
>+    // Delay the configuration of the HTMLMediaElement, as during this stage this is not
>+    // the MediaPlayer private yet and calls from HTMLMediaElement won't reach this.
>+    m_readyTimer.startOneShot(0_s);
>+}
>+
>+MediaPlayerPrivateHolePunch::~MediaPlayerPrivateHolePunch()
>+{
>+#if USE(NICOSIA)
>+    downcast<Nicosia::ContentLayerTextureMapperImpl>(m_nicosiaLayer->impl()).invalidateClient();
>+#endif
>+}
>+
>+PlatformLayer* MediaPlayerPrivateHolePunch::platformLayer() const
>+{
>+#if USE(NICOSIA)
>+    return m_nicosiaLayer.ptr();
>+#else
>+    return const_cast<MediaPlayerPrivateHolePunch*>(this);
>+#endif
>+}
>+
>+FloatSize MediaPlayerPrivateHolePunch::naturalSize() const
>+{
>+    // When using the holepuch we may not be able to get the video frames size, so we can't use
>+    // it. But we need to report some non empty naturalSize for the player's GraphicsLayer
>+    // to be properly created.
>+    return s_holePunchDefaultFrameSize;
>+}
>+
>+void MediaPlayerPrivateHolePunch::pushNextHolePunchBuffer()
>+{
>+    auto proxyOperation =
>+        [this](TextureMapperPlatformLayerProxy& proxy)
>+        {
>+            LockHolder holder(proxy.lock());
>+            std::unique_ptr<TextureMapperPlatformLayerBuffer> layerBuffer = std::make_unique<TextureMapperPlatformLayerBuffer>(0, m_size, TextureMapperGL::ShouldNotBlend, GL_DONT_CARE);
>+            proxy.pushNextBuffer(WTFMove(layerBuffer));
>+        };
>+
>+#if USE(NICOSIA)
>+    proxyOperation(downcast<Nicosia::ContentLayerTextureMapperImpl>(m_nicosiaLayer->impl()).proxy());
>+#else
>+    proxyOperation(*m_platformLayerProxy);
>+#endif
>+}
>+
>+void MediaPlayerPrivateHolePunch::swapBuffersIfNeeded()
>+{
>+    pushNextHolePunchBuffer();
>+}
>+
>+#if !USE(NICOSIA)
>+RefPtr<TextureMapperPlatformLayerProxy> MediaPlayerPrivateHolePunch::proxy() const
>+{
>+    return m_platformLayerProxy.copyRef();
>+}
>+#endif
>+
>+static HashSet<String, ASCIICaseInsensitiveHash>& mimeTypeCache()
>+{
>+    static NeverDestroyed<HashSet<String, ASCIICaseInsensitiveHash>> cache;
>+    static bool typeListInitialized = false;
>+
>+    if (typeListInitialized)
>+        return cache;
>+
>+    const char* mimeTypes[] = {
>+        "video/holepunch"
>+    };
>+
>+    for (unsigned i = 0; i < (sizeof(mimeTypes) / sizeof(*mimeTypes)); ++i)
>+        cache.get().add(String(mimeTypes[i]));
>+
>+    typeListInitialized = true;
>+
>+    return cache;
>+}
>+
>+void MediaPlayerPrivateHolePunch::getSupportedTypes(HashSet<String, ASCIICaseInsensitiveHash>& types)
>+{
>+    types = mimeTypeCache();
>+}
>+
>+MediaPlayer::SupportsType MediaPlayerPrivateHolePunch::supportsType(const MediaEngineSupportParameters& parameters)
>+{
>+    auto containerType = parameters.type.containerType();
>+
>+    // Spec says we should not return "probably" if the codecs string is empty.
>+    if (!containerType.isEmpty() && mimeTypeCache().contains(containerType)) {
>+        if (parameters.type.codecs().isEmpty())
>+            return MediaPlayer::MayBeSupported;
>+
>+        return MediaPlayer::IsSupported;
>+    }
>+
>+    return MediaPlayer::IsNotSupported;
>+}
>+
>+void MediaPlayerPrivateHolePunch::registerMediaEngine(MediaEngineRegistrar registrar)
>+{
>+    registrar([](MediaPlayer* player) { return std::make_unique<MediaPlayerPrivateHolePunch>(player); },
>+        getSupportedTypes, supportsType, nullptr, nullptr, nullptr, nullptr);
>+}
>+
>+void MediaPlayerPrivateHolePunch::notifyReadyState()
>+{
>+    // Notify the ready state so the GraphicsLayer gets created.
>+    m_player->readyStateChanged();
>+}
>+}
>+#endif // USE(EXTERNAL_HOLEPUNCH)
>diff --git a/Source/WebCore/platform/graphics/holepunch/MediaPlayerPrivateHolePunch.h b/Source/WebCore/platform/graphics/holepunch/MediaPlayerPrivateHolePunch.h
>new file mode 100644
>index 0000000000000000000000000000000000000000..f0d04d4c63d93120bce805ab8fea287a01e0442a
>--- /dev/null
>+++ b/Source/WebCore/platform/graphics/holepunch/MediaPlayerPrivateHolePunch.h
>@@ -0,0 +1,119 @@
>+/*
>+ * Copyright (C) 2019 Igalia S.L
>+ * Copyright (C) 2019 Metrological Group B.V.
>+ *
>+ * This library is free software; you can redistribute it and/or
>+ * modify it under the terms of the GNU Library General Public
>+ * License as published by the Free Software Foundation; either
>+ * version 2 of the License, or (at your option) any later version.
>+ *
>+ * This library is distributed in the hope that it will be useful,
>+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
>+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
>+ * Library General Public License for more details.
>+ *
>+ * You should have received a copy of the GNU Library General Public License
>+ * aint with this library; see the file COPYING.LIB.  If not, write to
>+ * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
>+ * Boston, MA 02110-1301, USA.
>+ */
>+
>+#pragma once
>+
>+#if USE(EXTERNAL_HOLEPUNCH)
>+
>+#include "MediaPlayerPrivate.h"
>+#include "PlatformLayer.h"
>+#include <wtf/RunLoop.h>
>+#include <wtf/WeakPtr.h>
>+
>+#if USE(NICOSIA)
>+#include "NicosiaContentLayerTextureMapperImpl.h"
>+#else
>+#include "TextureMapperPlatformLayerProxyProvider.h"
>+#endif
>+
>+namespace WebCore {
>+
>+class TextureMapperPlatformLayerProxy;
>+
>+class MediaPlayerPrivateHolePunch : public MediaPlayerPrivateInterface, public CanMakeWeakPtr<MediaPlayerPrivateHolePunch>
>+#if USE(NICOSIA)
>+    , public Nicosia::ContentLayerTextureMapperImpl::Client
>+#else
>+    , public PlatformLayer
>+#endif
>+{
>+
>+public:
>+    MediaPlayerPrivateHolePunch(MediaPlayer*);
>+    ~MediaPlayerPrivateHolePunch();
>+
>+    static void registerMediaEngine(MediaEngineRegistrar);
>+
>+    void load(const String&) final { };
>+#if ENABLE(MEDIA_SOURCE)
>+    void load(const String&, MediaSourcePrivateClient*) final { };
>+#endif
>+#if ENABLE(MEDIA_STREAM)
>+    void load(MediaStreamPrivate&) final { };
>+#endif
>+    void cancelLoad() final { };
>+
>+    void play() final { };
>+    void pause() final { };
>+
>+    PlatformLayer* platformLayer() const final;
>+
>+    FloatSize naturalSize() const final;
>+
>+    bool hasVideo() const final { return false; };
>+    bool hasAudio() const final { return false; };
>+
>+    void setVisible(bool) final { };
>+
>+    bool seeking() const final { return false; }
>+
>+    bool paused() const final { return false; };
>+
>+    MediaPlayer::NetworkState networkState() const final { return MediaPlayer::Empty; };
>+    MediaPlayer::ReadyState readyState() const final { return MediaPlayer::HaveMetadata; };
>+
>+    std::unique_ptr<PlatformTimeRanges> buffered() const final { return std::make_unique<PlatformTimeRanges>(); };
>+
>+    bool didLoadingProgress() const final { return false; };
>+
>+    void setSize(const IntSize& size) final { m_size = size; };
>+
>+    void paint(GraphicsContext&, const FloatRect&) final { };
>+
>+    bool supportsAcceleratedRendering() const final { return true; }
>+
>+    bool shouldIgnoreIntrinsicSize() final { return true; }
>+
>+    void pushNextHolePunchBuffer();
>+    void swapBuffersIfNeeded() final;
>+#if !USE(NICOSIA)
>+    RefPtr<TextureMapperPlatformLayerProxy> proxy() const final;
>+#endif
>+
>+private:
>+    static void getSupportedTypes(HashSet<String, ASCIICaseInsensitiveHash>&);
>+    static MediaPlayer::SupportsType supportsType(const MediaEngineSupportParameters&);
>+
>+    void notifyReadyState();
>+
>+    MediaPlayer* m_player;
>+    IntSize m_size;
>+    RunLoop::Timer<MediaPlayerPrivateHolePunch> m_readyTimer;
>+#if USE(TEXTURE_MAPPER_GL)
>+#if USE(NICOSIA)
>+    Ref<Nicosia::ContentLayer> m_nicosiaLayer;
>+#else
>+    RefPtr<TextureMapperPlatformLayerProxy> m_platformLayerProxy;
>+#endif
>+#endif
>+
>+};
>+}
>+#endif // USE(EXTERNAL_HOLEPUNCH)
>diff --git a/Source/WebCore/platform/graphics/texmap/TextureMapperPlatformLayerBuffer.cpp b/Source/WebCore/platform/graphics/texmap/TextureMapperPlatformLayerBuffer.cpp
>index 963bdc4864a7c661608e2a419b514588062654f1..01f0d2897338f4ad37f5bc18641fea2883729331 100644
>--- a/Source/WebCore/platform/graphics/texmap/TextureMapperPlatformLayerBuffer.cpp
>+++ b/Source/WebCore/platform/graphics/texmap/TextureMapperPlatformLayerBuffer.cpp
>@@ -78,9 +78,10 @@ void TextureMapperPlatformLayerBuffer::paintToTextureMapper(TextureMapper& textu
>         return;
>     }
> 
>-    if (m_holePunchClient && m_extraFlags & TextureMapperGL::ShouldNotBlend) {
>+    if (m_extraFlags & TextureMapperGL::ShouldNotBlend) {
>         ASSERT(!m_texture);
>-        m_holePunchClient->setVideoRectangle(enclosingIntRect(modelViewMatrix.mapRect(targetRect)));
>+        if (m_holePunchClient)
>+            m_holePunchClient->setVideoRectangle(enclosingIntRect(modelViewMatrix.mapRect(targetRect)));
>         texmapGL.drawSolidColor(targetRect, modelViewMatrix, Color(0, 0, 0, 0), false);
>         return;
>     }
>diff --git a/Source/cmake/OptionsWPE.cmake b/Source/cmake/OptionsWPE.cmake
>index 678aa1a467919b693674e4bbe49b23641519f528..05c5d2918d764934b112dce8420e7539864c4648 100644
>--- a/Source/cmake/OptionsWPE.cmake
>+++ b/Source/cmake/OptionsWPE.cmake
>@@ -70,6 +70,7 @@ WEBKIT_OPTION_DEFINE(ENABLE_WPE_QT_API "Whether to enable support for the Qt5/QM
> # Private options specific to the WPE port.
> WEBKIT_OPTION_DEFINE(USE_OPENVR "Whether to use OpenVR as WebVR backend." PRIVATE ${ENABLE_EXPERIMENTAL_FEATURES})
> WEBKIT_OPTION_DEFINE(USE_GSTREAMER_HOLEPUNCH "Whether to enable GStreamer holepunch" PRIVATE OFF)
>+WEBKIT_OPTION_DEFINE(USE_EXTERNAL_HOLEPUNCH "Whether to enable external holepunch" PRIVATE OFF)
> 
> if (CMAKE_SYSTEM_NAME MATCHES "Linux")
>     WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_MEMORY_SAMPLER PRIVATE ON)
>@@ -94,6 +95,7 @@ if (("${PC_CAIRO_VERSION}" VERSION_GREATER "1.16.0" OR "${PC_CAIRO_VERSION}" STR
> endif ()
> 
> WEBKIT_OPTION_DEPEND(USE_GSTREAMER_HOLEPUNCH ENABLE_VIDEO)
>+WEBKIT_OPTION_DEPEND(USE_EXTERNAL_HOLEPUNCH ENABLE_VIDEO)
> 
> include(GStreamerDependencies)
> 
>diff --git a/ChangeLog b/ChangeLog
>index 8afc9813bcc4cf076f15de5795f324fbee51a5ea..a92af1799fc1e0989f6155870927f5dbe57e17cf 100644
>--- a/ChangeLog
>+++ b/ChangeLog
>@@ -1,3 +1,17 @@
>+2019-02-22  Miguel Gomez  <magomez@igalia.com>
>+
>+        [WPE] Add support for holepunch using an external video player
>+        https://bugs.webkit.org/show_bug.cgi?id=194899
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Add EXTERNAL_HOLEPUNCH option to the WPE port. Add a manual test to check the
>+        feature. Rename and update the test for GStreamer holepunch to avoid confusion.
>+
>+        * ManualTests/wpe/video-player-holepunch-external.html: Added.
>+        * ManualTests/wpe/video-player-holepunch-gstreamer.html: Renamed from ManualTests/wpe/video-player-holepunch.html.
>+        * Source/cmake/OptionsWPE.cmake:
>+
> 2019-02-22  Adrian Perez de Castro  <aperez@igalia.com>
> 
>         Unreviewed. Bump version numbers
>diff --git a/ManualTests/wpe/video-player-holepunch-external.html b/ManualTests/wpe/video-player-holepunch-external.html
>new file mode 100644
>index 0000000000000000000000000000000000000000..ffbb7453e334b6106aa46c3251b2a050b6f93e7f
>--- /dev/null
>+++ b/ManualTests/wpe/video-player-holepunch-external.html
>@@ -0,0 +1,56 @@
>+<html>
>+  <head>
>+    <title>WPE external holepunch test</title>
>+    <style>
>+      video {
>+          width: 400px;
>+          height: 400px;
>+      }
>+    </style>
>+  </head>
>+  <body>
>+    <p>
>+      <strong>WPE external holepunch test</strong>
>+    </p>
>+    <p>
>+      This test checks whether the external holepunch feature is working on WPE.
>+    </p>
>+    <p>
>+      There is a video player of 400x400 below this text. It will  play a valid video for 5 seconds, then a holepunch one
>+      for another 5 seconds and then the first one again.
>+    </p>
>+    <p>
>+      If everything is working fine, there should be the valid video for 5 seconds, then a transparent rectangle of 400x400
>+      for 5 seconds, and then again the valid video.
>+    </p>
>+    <p>
>+      If GStreamer holepunch is enabled when running this test, the first and third video will show a transparent
>+      rectangle instead of the video content.
>+    </p>
>+    <video id="video">
>+      <source id="source" src="../../LayoutTests/media/content/long-test.mp4" type="video/mp4">
>+    </video>
>+    <div id="log"></div>
>+    <script type="text/javascript">
>+      var v = document.getElementById("video");
>+      var s = document.getElementById("source");
>+      var l = document.getElementById("log");
>+      log.innerHTML = "Playing first video. The video (or a transparent rectangle if GStreamer holepunch is enabled) should be visible";
>+      v.play();
>+
>+      setTimeout(function() {
>+        s.type = "video/holepunch";
>+        v.load();
>+        v.play();
>+        log.innerHTML = "Playing second video. A transparent rectangle should be visible";
>+      }, 5000);
>+
>+      setTimeout(function() {
>+        s.type = "video/mp4";
>+        v.load();
>+        v.play();
>+        log.innerHTML = "Playing third video. The video (or a transparent rectangle if GStreamer holepunch is enabled) should be visible";
>+      }, 10000);
>+    </script>
>+  </body>
>+</html>
>diff --git a/ManualTests/wpe/video-player-holepunch-gstreamer.html b/ManualTests/wpe/video-player-holepunch-gstreamer.html
>new file mode 100644
>index 0000000000000000000000000000000000000000..1f4ff36e82bd5c37d90980fc114058e133f964c6
>--- /dev/null
>+++ b/ManualTests/wpe/video-player-holepunch-gstreamer.html
>@@ -0,0 +1,33 @@
>+<html>
>+  <head>
>+    <title>WPE GStreamer holepunch test</title>
>+    <style>
>+      video {
>+          width: 400px;
>+          height: 400px;
>+      }
>+    </style>
>+  </head>
>+  <body>
>+    <p>
>+      <strong>WPE GStreamer holepunch test</strong>
>+    </p>
>+    <p>
>+      This test checks whether the GStreamer holepunch feature is working on WPE.
>+    </p>
>+    <p>
>+      There is a video player of 400x400 placed below this text. If the video playback is visible,
>+      then the holepunch option is disabled or it's not working properly.
>+    </p>
>+    <p>
>+      If everything is working fine, there should be a transparent rectangle of 400x400 just
>+      below this.
>+    </p>
>+    <video id="video"></video>
>+    <script type="text/javascript">
>+      var v = document.getElementById("video");
>+      v.src = "../../LayoutTests/media/content/long-test.mp4";
>+      v.play();
>+    </script>
>+  </body>
>+</html>
>diff --git a/ManualTests/wpe/video-player-holepunch.html b/ManualTests/wpe/video-player-holepunch.html
>deleted file mode 100644
>index f69d08d4c80c7f910cd6416818e0bba3da83e6ad..0000000000000000000000000000000000000000
>--- a/ManualTests/wpe/video-player-holepunch.html
>+++ /dev/null
>@@ -1,33 +0,0 @@
>-<html>
>-  <head>
>-    <title>WPE holepunch test</title>
>-    <style>
>-      video {
>-          width: 400px;
>-          height: 400px;
>-      }
>-    </style>
>-  </head>
>-  <body>
>-    <p>
>-      <strong>WPE holepunch test</strong>
>-    </p>
>-    <p>
>-      This test checks whether the holepunch feature is working on WPE.
>-    </p>
>-    <p>
>-      There is a video player of 400x400 placed below this text. If the video playback is visible,
>-      then the holepunch option is disabled or it's not working properly.
>-    </p>
>-    <p>
>-      If everything is working fine, there should be a transparent rectangle of 400x400 just
>-      below this.
>-    </p>
>-    <video id="video"></video>
>-    <script type="text/javascript">
>-      var v = document.getElementById("video");
>-      v.src = "../../LayoutTests/media/content/long-test.mp4";
>-      v.play();
>-    </script>
>-  </body>
>-</html>

Actions: View | Formatted Diff | Diff

Attachments on bug 194899: 362608 | 362721