Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(538)

Unified Diff: third_party/WebKit/Source/core/html/MediaCustomControlsFullscreenDetectorTest.cpp

Issue 2780403004: Create core/html/media/ and move auxiliary media files in it. (Closed)
Patch Set: actually add autoplay files Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/html/MediaCustomControlsFullscreenDetectorTest.cpp
diff --git a/third_party/WebKit/Source/core/html/MediaCustomControlsFullscreenDetectorTest.cpp b/third_party/WebKit/Source/core/html/MediaCustomControlsFullscreenDetectorTest.cpp
deleted file mode 100644
index 9a1eea922fbec95f4bb1a5e57d2f9af583d28eb4..0000000000000000000000000000000000000000
--- a/third_party/WebKit/Source/core/html/MediaCustomControlsFullscreenDetectorTest.cpp
+++ /dev/null
@@ -1,186 +0,0 @@
-// Copyright 2017 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "core/html/MediaCustomControlsFullscreenDetector.h"
-
-#include "core/EventTypeNames.h"
-#include "core/html/HTMLVideoElement.h"
-#include "core/testing/DummyPageHolder.h"
-#include "platform/RuntimeEnabledFeatures.h"
-#include "platform/geometry/IntRect.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-namespace blink {
-
-namespace {
-
-struct TestParam {
- String description;
- IntRect targetRect;
- bool expectedResult;
-};
-
-} // anonymous namespace
-
-class MediaCustomControlsFullscreenDetectorTest : public ::testing::Test {
- protected:
- void SetUp() override {
- m_originalVideoFullscreenDetectionEnabled =
- RuntimeEnabledFeatures::videoFullscreenDetectionEnabled();
-
- RuntimeEnabledFeatures::setVideoFullscreenDetectionEnabled(true);
-
- m_pageHolder = DummyPageHolder::create();
- m_newPageHolder = DummyPageHolder::create();
- }
-
- void TearDown() override {
- RuntimeEnabledFeatures::setVideoFullscreenDetectionEnabled(
- m_originalVideoFullscreenDetectionEnabled);
- }
-
- HTMLVideoElement* videoElement() const {
- return toHTMLVideoElement(document().querySelector("video"));
- }
-
- static MediaCustomControlsFullscreenDetector* fullscreenDetectorFor(
- HTMLVideoElement* videoElement) {
- return videoElement->m_customControlsFullscreenDetector;
- }
-
- MediaCustomControlsFullscreenDetector* fullscreenDetector() const {
- return fullscreenDetectorFor(videoElement());
- }
-
- Document& document() const { return m_pageHolder->document(); }
- Document& newDocument() const { return m_newPageHolder->document(); }
-
- bool checkEventListenerRegistered(EventTarget& target,
- const AtomicString& eventType,
- EventListener* listener) {
- EventListenerVector* listeners = target.getEventListeners(eventType);
- if (!listeners)
- return false;
-
- for (const auto& registeredListener : *listeners) {
- if (registeredListener.listener() == listener)
- return true;
- }
- return false;
- }
-
- static bool computeIsDominantVideo(const IntRect& targetRect,
- const IntRect& rootRect,
- const IntRect& intersectionRect) {
- return MediaCustomControlsFullscreenDetector::
- computeIsDominantVideoForTests(targetRect, rootRect, intersectionRect);
- }
-
- private:
- std::unique_ptr<DummyPageHolder> m_pageHolder;
- std::unique_ptr<DummyPageHolder> m_newPageHolder;
- Persistent<HTMLVideoElement> m_video;
-
- bool m_originalVideoFullscreenDetectionEnabled;
-};
-
-TEST_F(MediaCustomControlsFullscreenDetectorTest, computeIsDominantVideo) {
- // TestWithParam cannot be applied here as IntRect needs the memory allocator
- // to be initialized, but the array of parameters is statically initialized,
- // which is before the memory allocation initialization.
- TestParam testParams[] = {
- {"xCompleteFill", {0, 0, 100, 50}, true},
- {"yCompleteFill", {0, 0, 50, 100}, true},
- {"xyCompleteFill", {0, 0, 100, 100}, true},
- {"xIncompleteFillTooSmall", {0, 0, 84, 50}, false},
- {"yIncompleteFillTooSmall", {0, 0, 50, 84}, false},
- {"xIncompleteFillJustRight", {0, 0, 86, 50}, true},
- {"yIncompleteFillJustRight", {0, 0, 50, 86}, true},
- {"xVisibleProportionTooSmall", {-26, 0, 100, 100}, false},
- {"yVisibleProportionTooSmall", {0, -26, 100, 100}, false},
- {"xVisibleProportionJustRight", {-24, 0, 100, 100}, true},
- {"yVisibleProportionJustRight", {0, -24, 100, 100}, true},
- };
-
- IntRect rootRect(0, 0, 100, 100);
-
- for (const TestParam& testParam : testParams) {
- const IntRect& targetRect = testParam.targetRect;
- IntRect intersectionRect = intersection(targetRect, rootRect);
- EXPECT_EQ(testParam.expectedResult,
- computeIsDominantVideo(targetRect, rootRect, intersectionRect))
- << testParam.description << " failed";
- }
-}
-
-TEST_F(MediaCustomControlsFullscreenDetectorTest,
- hasNoListenersBeforeAddingToDocument) {
- HTMLVideoElement* video =
- toHTMLVideoElement(document().createElement("video"));
-
- EXPECT_FALSE(checkEventListenerRegistered(document(),
- EventTypeNames::fullscreenchange,
- fullscreenDetectorFor(video)));
- EXPECT_FALSE(checkEventListenerRegistered(
- document(), EventTypeNames::webkitfullscreenchange,
- fullscreenDetectorFor(video)));
- EXPECT_FALSE(checkEventListenerRegistered(
- *video, EventTypeNames::loadedmetadata, fullscreenDetectorFor(video)));
-}
-
-TEST_F(MediaCustomControlsFullscreenDetectorTest,
- hasListenersAfterAddToDocumentByScript) {
- HTMLVideoElement* video =
- toHTMLVideoElement(document().createElement("video"));
- document().body()->appendChild(video);
-
- EXPECT_TRUE(checkEventListenerRegistered(
- document(), EventTypeNames::fullscreenchange, fullscreenDetector()));
- EXPECT_TRUE(checkEventListenerRegistered(
- document(), EventTypeNames::webkitfullscreenchange,
- fullscreenDetector()));
- EXPECT_TRUE(checkEventListenerRegistered(
- *videoElement(), EventTypeNames::loadedmetadata, fullscreenDetector()));
-}
-
-TEST_F(MediaCustomControlsFullscreenDetectorTest,
- hasListenersAfterAddToDocumentByParser) {
- document().body()->setInnerHTML("<body><video></video></body>");
-
- EXPECT_TRUE(checkEventListenerRegistered(
- document(), EventTypeNames::fullscreenchange, fullscreenDetector()));
- EXPECT_TRUE(checkEventListenerRegistered(
- document(), EventTypeNames::webkitfullscreenchange,
- fullscreenDetector()));
- EXPECT_TRUE(checkEventListenerRegistered(
- *videoElement(), EventTypeNames::loadedmetadata, fullscreenDetector()));
-}
-
-TEST_F(MediaCustomControlsFullscreenDetectorTest,
- hasListenersAfterDocumentMove) {
- HTMLVideoElement* video =
- toHTMLVideoElement(document().createElement("video"));
- document().body()->appendChild(video);
-
- newDocument().body()->appendChild(videoElement());
-
- EXPECT_FALSE(checkEventListenerRegistered(document(),
- EventTypeNames::fullscreenchange,
- fullscreenDetectorFor(video)));
- EXPECT_FALSE(checkEventListenerRegistered(
- document(), EventTypeNames::webkitfullscreenchange,
- fullscreenDetectorFor(video)));
-
- EXPECT_TRUE(checkEventListenerRegistered(newDocument(),
- EventTypeNames::fullscreenchange,
- fullscreenDetectorFor(video)));
- EXPECT_TRUE(checkEventListenerRegistered(
- newDocument(), EventTypeNames::webkitfullscreenchange,
- fullscreenDetectorFor(video)));
-
- EXPECT_TRUE(checkEventListenerRegistered(
- *video, EventTypeNames::loadedmetadata, fullscreenDetectorFor(video)));
-}
-
-} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698