service worker: Fix the type of a register promise reject value

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

Index: third_party/WebKit/Source/modules/serviceworkers/ServiceWorkerContainer.cpp
diff --git a/third_party/WebKit/Source/modules/serviceworkers/ServiceWorkerContainer.cpp b/third_party/WebKit/Source/modules/serviceworkers/ServiceWorkerContainer.cpp
index efd57b705ecb0132efe521ca599154baee0085e0..ff4c793c14f092ff6b3103f00adef70c3a461a53 100644
--- a/third_party/WebKit/Source/modules/serviceworkers/ServiceWorkerContainer.cpp
+++ b/third_party/WebKit/Source/modules/serviceworkers/ServiceWorkerContainer.cpp
@@ -76,9 +76,16 @@ public:
     {
         if (!m_resolver->getExecutionContext() || m_resolver->getExecutionContext()->activeDOMObjectsAreStopped())
             return;
-        if (error.errorType == WebServiceWorkerError::ErrorTypeType) {
-            m_resolver->reject(V8ThrowException::createTypeError(m_resolver->getScriptState()->isolate(), error.message));
-        } else {
+        switch (error.errorType) {
+        case WebServiceWorkerError::ErrorTypeNetwork:
+        case WebServiceWorkerError::ErrorTypeScriptEvaluateFailed:
+        case WebServiceWorkerError::ErrorTypeType: {
+            ScriptState* scriptState = m_resolver->getScriptState();
+            ScriptState::Scope scope(scriptState);
+            m_resolver->reject(V8ThrowException::createTypeError(scriptState->isolate(), error.message));
+            break;

[falken, 2016/06/14 02:29:18]

Why are these types the ones in the switch?

This is also the same code as in the update patch. It'd be better if it's in one
place.

[e_hakkinen, 2016/07/08 12:21:13]

Done.

+        }
+        default:
             m_resolver->reject(ServiceWorkerError::take(m_resolver.get(), error));
         }
     }