third_party/WebKit/Source/web/tests/VisualViewportTest.cpp - Issue 2654933003: platform/testing/{URL|Unit}TestHelpers improvements

Unified Diff: third_party/WebKit/Source/web/tests/VisualViewportTest.cpp

Issue 2654933003: platform/testing/{URL|Unit}TestHelpers improvements (Closed)

Patch Set: header changes Created 3 years, 10 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

« no previous file with comments | « third_party/WebKit/Source/web/tests/ViewportTest.cpp ('k') | third_party/WebKit/Source/web/tests/WebDocumentSubresourceFilterTest.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: third_party/WebKit/Source/web/tests/VisualViewportTest.cpp

diff --git a/third_party/WebKit/Source/web/tests/VisualViewportTest.cpp b/third_party/WebKit/Source/web/tests/VisualViewportTest.cpp

index 3070cc11c820a63b78b2c62e3b21e07e2eca1f1a..7d084c1db21b9a3a8d4c0f515429bf100b8d7f50 100644

--- a/third_party/WebKit/Source/web/tests/VisualViewportTest.cpp

+++ b/third_party/WebKit/Source/web/tests/VisualViewportTest.cpp

@@ -22,6 +22,7 @@

#include "platform/graphics/CompositorElementId.h"

#include "platform/testing/RuntimeEnabledFeaturesTestHelpers.h"

#include "platform/testing/URLTestHelpers.h"

+#include "platform/testing/UnitTestHelpers.h"

#include "public/platform/Platform.h"

#include "public/platform/WebCachePolicy.h"

#include "public/platform/WebInputEvent.h"

@@ -103,8 +104,8 @@ namespace {

typedef bool TestParamRootLayerScrolling;

class VisualViewportTest

- : public testing::Test,

- public testing::WithParamInterface<TestParamRootLayerScrolling>,

+ : public ::testing::Test,

+ public ::testing::WithParamInterface<TestParamRootLayerScrolling>,

private ScopedRootLayerScrollingForTest {

public:

VisualViewportTest()

@@ -143,9 +144,16 @@ class VisualViewportTest

}

void registerMockedHttpURLLoad(const std::string& fileName) {

- URLTestHelpers::registerMockedURLFromBaseURL(

- WebString::fromUTF8(m_baseURL.c_str()),

- WebString::fromUTF8(fileName.c_str()));

+ URLTestHelpers::registerMockedURLLoadFromBase(

+ WebString::fromUTF8(m_baseURL), blink::testing::webTestDataPath(),

+ WebString::fromUTF8(fileName));

+ }

+ void registerMockedHttpURLLoad(const std::string& url,

+ const std::string& fileName) {

+ URLTestHelpers::registerMockedURLLoad(

+ toKURL(url),

+ blink::testing::webTestDataPath(WebString::fromUTF8(fileName)));

}

WebLayer* getRootScrollLayer() {

@@ -2143,8 +2151,7 @@ TEST_P(VisualViewportTest, ResizeCompositedAndFixedBackground) {

webViewImpl->resizeWithBrowserControls(WebSize(pageWidth, pageHeight),

browserControlsHeight, false);

- URLTestHelpers::registerMockedURLLoad(toKURL("http://example.com/foo.png"),

- "white-1x1.png");

+ registerMockedHttpURLLoad("http://example.com/foo.png", "white-1x1.png");

WebURL baseURL = URLTestHelpers::toKURL("http://example.com/");

FrameTestHelpers::loadHTMLString(webViewImpl->mainFrame(),

"<!DOCTYPE html>"

@@ -2226,8 +2233,7 @@ TEST_P(VisualViewportTest, ResizeNonCompositedAndFixedBackground) {

webViewImpl->resizeWithBrowserControls(WebSize(pageWidth, pageHeight),

browserControlsHeight, false);

- URLTestHelpers::registerMockedURLLoad(toKURL("http://example.com/foo.png"),

- "white-1x1.png");

+ registerMockedHttpURLLoad("http://example.com/foo.png", "white-1x1.png");

WebURL baseURL = URLTestHelpers::toKURL("http://example.com/");

FrameTestHelpers::loadHTMLString(webViewImpl->mainFrame(),

"<!DOCTYPE html>"

@@ -2334,8 +2340,7 @@ TEST_P(VisualViewportTest, ResizeNonFixedBackgroundNoLayoutOrInvalidation) {

webViewImpl->resizeWithBrowserControls(WebSize(pageWidth, pageHeight),

browserControlsHeight, false);

- URLTestHelpers::registerMockedURLLoad(toKURL("http://example.com/foo.png"),

- "white-1x1.png");

+ registerMockedHttpURLLoad("http://example.com/foo.png", "white-1x1.png");

WebURL baseURL = URLTestHelpers::toKURL("http://example.com/");

// This time the background is the default attachment.

FrameTestHelpers::loadHTMLString(webViewImpl->mainFrame(),