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

Unified Diff: Source/modules/serviceworkers/ServiceWorkerContainerTest.cpp

Issue 1260003003: Move Service Worker %2f validation logic from browser into Blink (2) (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: More validation back into Blink. Created 5 years, 5 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: Source/modules/serviceworkers/ServiceWorkerContainerTest.cpp
diff --git a/Source/modules/serviceworkers/ServiceWorkerContainerTest.cpp b/Source/modules/serviceworkers/ServiceWorkerContainerTest.cpp
index cc061069440b6368821adc09885c3f589c86f0a3..3924dafe8d6b8185d269addd3618d43f3906989b 100644
--- a/Source/modules/serviceworkers/ServiceWorkerContainerTest.cpp
+++ b/Source/modules/serviceworkers/ServiceWorkerContainerTest.cpp
@@ -139,6 +139,11 @@ public:
ADD_FAILURE() << "the provider should not be called to register a Service Worker";
delete callbacks;
}
+
+ bool validateScopeAndScriptURL(const WebURL& scope, const WebURL& scriptURL, WebString* errorMessage)
+ {
+ return true;
+ }
};
class ServiceWorkerContainerTest : public ::testing::Test {
@@ -295,6 +300,11 @@ private:
m_getRegistrationCallbacksToDelete.append(adoptPtr(callbacks));
}
+ bool validateScopeAndScriptURL(const WebURL& scope, const WebURL& scriptURL, WebString* errorMessage)
+ {
+ return true;
+ }
+
private:
StubWebServiceWorkerProvider& m_owner;
Vector<OwnPtr<WebServiceWorkerRegistrationCallbacks>> m_registrationCallbacksToDelete;

Powered by Google App Engine
This is Rietveld 408576698