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

Unified Diff: third_party/WebKit/Source/web/LinkHighlightImplTest.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
Download patch
Index: third_party/WebKit/Source/web/LinkHighlightImplTest.cpp
diff --git a/third_party/WebKit/Source/web/LinkHighlightImplTest.cpp b/third_party/WebKit/Source/web/LinkHighlightImplTest.cpp
index 799eeecee4d808d196d215e15b9996d707680d22..512dca620266f64a0c73e9c5eceb2eda2417eeee 100644
--- a/third_party/WebKit/Source/web/LinkHighlightImplTest.cpp
+++ b/third_party/WebKit/Source/web/LinkHighlightImplTest.cpp
@@ -25,6 +25,7 @@
#include "web/LinkHighlightImpl.h"
+#include <memory>
#include "bindings/core/v8/ExceptionState.h"
#include "core/dom/Node.h"
#include "core/frame/FrameView.h"
@@ -33,6 +34,7 @@
#include "core/page/TouchDisambiguation.h"
#include "platform/geometry/IntRect.h"
#include "platform/testing/URLTestHelpers.h"
+#include "platform/testing/UnitTestHelpers.h"
#include "public/platform/Platform.h"
#include "public/platform/WebContentLayer.h"
#include "public/platform/WebFloatPoint.h"
@@ -49,10 +51,11 @@
#include "web/WebViewImpl.h"
#include "web/tests/FrameTestHelpers.h"
#include "wtf/PtrUtil.h"
-#include <memory>
namespace blink {
+namespace {
+
GestureEventWithHitTestResults getTargetedEvent(WebViewImpl* webViewImpl,
WebGestureEvent& touchEvent) {
WebGestureEvent scaledEvent = TransformWebGestureEvent(
@@ -63,16 +66,19 @@ GestureEventWithHitTestResults getTargetedEvent(WebViewImpl* webViewImpl,
.targetGestureEvent(scaledEvent, true);
}
-TEST(LinkHighlightImplTest, verifyWebViewImplIntegration) {
- const std::string baseURL("http://www.test.com/");
- const std::string fileName("test_touch_link_highlight.html");
-
- URLTestHelpers::registerMockedURLFromBaseURL(
- WebString::fromUTF8(baseURL.c_str()),
+std::string registerMockedURLLoad() {
+ WebURL url = URLTestHelpers::registerMockedURLLoadFromBase(
+ WebString::fromUTF8("http://www.test.com/"), testing::webTestDataPath(),
WebString::fromUTF8("test_touch_link_highlight.html"));
+ return url.string().utf8();
+}
+
+} // namespace
+
+TEST(LinkHighlightImplTest, verifyWebViewImplIntegration) {
+ const std::string url = registerMockedURLLoad();
FrameTestHelpers::WebViewHelper webViewHelper;
- WebViewImpl* webViewImpl =
- webViewHelper.initializeAndLoad(baseURL + fileName, true);
+ WebViewImpl* webViewImpl = webViewHelper.initializeAndLoad(url, true);
int pageWidth = 640;
int pageHeight = 480;
webViewImpl->resize(WebSize(pageWidth, pageHeight));
@@ -141,15 +147,10 @@ FakeCompositingWebViewClient* compositingWebViewClient() {
} // anonymous namespace
TEST(LinkHighlightImplTest, resetDuringNodeRemoval) {
- const std::string baseURL("http://www.test.com/");
- const std::string fileName("test_touch_link_highlight.html");
-
- URLTestHelpers::registerMockedURLFromBaseURL(
- WebString::fromUTF8(baseURL.c_str()),
- WebString::fromUTF8("test_touch_link_highlight.html"));
+ const std::string url = registerMockedURLLoad();
FrameTestHelpers::WebViewHelper webViewHelper;
- WebViewImpl* webViewImpl = webViewHelper.initializeAndLoad(
- baseURL + fileName, true, 0, compositingWebViewClient());
+ WebViewImpl* webViewImpl =
+ webViewHelper.initializeAndLoad(url, true, 0, compositingWebViewClient());
int pageWidth = 640;
int pageHeight = 480;
@@ -189,15 +190,10 @@ TEST(LinkHighlightImplTest, resetLayerTreeView) {
std::unique_ptr<FakeCompositingWebViewClient> webViewClient =
WTF::makeUnique<FakeCompositingWebViewClient>();
- const std::string baseURL("http://www.test.com/");
- const std::string fileName("test_touch_link_highlight.html");
-
- URLTestHelpers::registerMockedURLFromBaseURL(
- WebString::fromUTF8(baseURL.c_str()),
- WebString::fromUTF8("test_touch_link_highlight.html"));
+ const std::string url = registerMockedURLLoad();
FrameTestHelpers::WebViewHelper webViewHelper;
- WebViewImpl* webViewImpl = webViewHelper.initializeAndLoad(
- baseURL + fileName, true, 0, webViewClient.get());
+ WebViewImpl* webViewImpl =
+ webViewHelper.initializeAndLoad(url, true, 0, webViewClient.get());
int pageWidth = 640;
int pageHeight = 480;
@@ -233,15 +229,10 @@ TEST(LinkHighlightImplTest, resetLayerTreeView) {
}
TEST(LinkHighlightImplTest, multipleHighlights) {
- const std::string baseURL("http://www.test.com/");
- const std::string fileName("test_touch_link_highlight.html");
-
- URLTestHelpers::registerMockedURLFromBaseURL(
- WebString::fromUTF8(baseURL.c_str()),
- WebString::fromUTF8("test_touch_link_highlight.html"));
+ const std::string url = registerMockedURLLoad();
FrameTestHelpers::WebViewHelper webViewHelper;
- WebViewImpl* webViewImpl = webViewHelper.initializeAndLoad(
- baseURL + fileName, true, 0, compositingWebViewClient());
+ WebViewImpl* webViewImpl =
+ webViewHelper.initializeAndLoad(url, true, 0, compositingWebViewClient());
int pageWidth = 640;
int pageHeight = 480;

Powered by Google App Engine
This is Rietveld 408576698