Move Service Worker %2f path validation logic from browser into Blink (1)

content/child/service_worker/web_service_worker_provider_impl.cc

Index: content/child/service_worker/web_service_worker_provider_impl.cc
diff --git a/content/child/service_worker/web_service_worker_provider_impl.cc b/content/child/service_worker/web_service_worker_provider_impl.cc
index 154dfe0281aae8a717387ef8bd2dd79bc7e73ee0..0c9dd435c1f2f79b2013b06ffcf07dd3d28fddbd 100644
--- a/content/child/service_worker/web_service_worker_provider_impl.cc
+++ b/content/child/service_worker/web_service_worker_provider_impl.cc
@@ -76,6 +76,14 @@ void WebServiceWorkerProviderImpl::getRegistrationForReady(
   GetDispatcher()->GetRegistrationForReady(context_->provider_id(), callbacks);
 }
 
+bool WebServiceWorkerProviderImpl::areScopeAndScriptURLValid(
+    const blink::WebURL& scope,
+    const blink::WebURL& script_url,
+    WTF::String* error_message) {
+  return !ServiceWorkerUtils::ContainsDisallowedCharacter(

[nhiroki, 2015/07/28 09:16:03]

This is a module dependency violation. We cannot include content/browser/* from
content/child/*. Can you move ServiceWorkerUtils into content/common/ which can
be accessed from content/{browser,child,renderer}/?

+      pattern->GURL(), script_url->GURL(), error_message);
+}
+
 int WebServiceWorkerProviderImpl::provider_id() const {
   return context_->provider_id();
 }