service worker: Implement NavigationPreloadManager.setHeaderValue

third_party/WebKit/Source/modules/serviceworkers/NavigationPreloadManager.cpp

 // Copyright 2016 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 "modules/serviceworkers/NavigationPreloadManager.h"
 
+#include "bindings/core/v8/CallbackPromiseAdapter.h"
 #include "core/dom/DOMException.h"
 #include "modules/serviceworkers/NavigationPreloadCallbacks.h"
 #include "modules/serviceworkers/ServiceWorkerContainerClient.h"
 #include "modules/serviceworkers/ServiceWorkerRegistration.h"
 
 namespace blink {
 
 ScriptPromise NavigationPreloadManager::enable(ScriptState* scriptState) {
   return setEnabled(true, scriptState);
 }
 
 ScriptPromise NavigationPreloadManager::disable(ScriptState* scriptState) {
   return setEnabled(false, scriptState);
 }
 
-ScriptPromise NavigationPreloadManager::setHeaderValue(ScriptState*,
+ScriptPromise NavigationPreloadManager::setHeaderValue(ScriptState* scriptState,
                                                        const String& value) {

[horo, 2016/10/31 03:32:52]

It is not written in the spec yet, but I think we should block '\r', '\n' and
'\0'.

https://cs.chromium.org/chromium/src/third_party/WebKit/Source/platform/netwo...

[falken, 2016/10/31 08:32:08]

Done. I used IsValidHttpHeaderValue which looks like is what fetch() uses too.
Unfortunately the check on Chrome side is not exactly the same, but it shouldn't
be so bad. It's probably best to match fetch() at least.

-  NOTIMPLEMENTED();
-  return ScriptPromise();
+  ServiceWorkerContainerClient* client =
+      ServiceWorkerContainerClient::from(m_registration->getExecutionContext());
+  if (!client || !client->provider()) {
+    return ScriptPromise::rejectWithDOMException(
+        scriptState, DOMException::create(InvalidStateError, "No provider."));
+  }
+  ScriptPromiseResolver* resolver = ScriptPromiseResolver::create(scriptState);
+  ScriptPromise promise = resolver->promise();
+  m_registration->webRegistration()->setNavigationPreloadHeader(
+      value, client->provider(),
+      wrapUnique(new SetNavigationPreloadHeaderCallbacks(resolver)));
+  return promise;
 }
 
 ScriptPromise NavigationPreloadManager::getState(ScriptState* scriptState) {
   ServiceWorkerContainerClient* client =
       ServiceWorkerContainerClient::from(m_registration->getExecutionContext());
   if (!client || !client->provider()) {
     return ScriptPromise::rejectWithDOMException(
         scriptState, DOMException::create(InvalidStateError, "No provider."));
   }
   ScriptPromiseResolver* resolver = ScriptPromiseResolver::create(scriptState);
@@ skipping 22 matching lines @@
       enable, client->provider(),
       wrapUnique(new EnableNavigationPreloadCallbacks(resolver)));
   return promise;
 }
 
 DEFINE_TRACE(NavigationPreloadManager) {
   visitor->trace(m_registration);
 }
 
 }  // namespace blink