PaymentApp: Implement PaymentAppManager.setManifest().

third_party/WebKit/Source/modules/payments/PaymentAppManager.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/payments/PaymentAppManager.h"
 
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptState.h"
 #include "core/dom/DOMException.h"
 #include "modules/payments/PaymentAppManifest.h"
@@ skipping 72 matching lines @@
                                       wrapPersistent(resolver))));
 
   return promise;
 }
 
 void PaymentAppManager::onSetManifest(
     ScriptPromiseResolver* resolver,
     payments::mojom::blink::PaymentAppManifestError error) {
   DCHECK(resolver);
   switch (error) {
+    case payments::mojom::blink::PaymentAppManifestError::NONE:
+      resolver->resolve();
+      break;
     case payments::mojom::blink::PaymentAppManifestError::NOT_IMPLEMENTED:
       resolver->reject(
           DOMException::create(NotSupportedError, "Not implemented yet."));
       break;
+    case payments::mojom::blink::PaymentAppManifestError::NO_ACTIVE_WORKER:
+      resolver->reject(
+          DOMException::create(InvalidStateError, "No active service worker."));
+      break;
+    case payments::mojom::blink::PaymentAppManifestError::STORE_MANIFEST_FAILED:
+      resolver->reject(DOMException::create(
+          InvalidStateError, "Storing manifest data is failed."));
+      break;
     default:

[please use gerrit instead, 2016/11/15 20:04:27]

If PaymentAppManifestError is an enum, remove the default clause. This will make
compiler force you to handle all possible values of the enum, thus eliminating
hard-to-catch bugs.

[zino, 2016/11/16 18:12:59]

Done.

       NOTREACHED();
   }
 }
 
 DEFINE_TRACE(PaymentAppManager) {
   visitor->trace(m_registration);
   ContextLifecycleObserver::trace(visitor);
 }
 
 PaymentAppManager::PaymentAppManager(ScriptState* scriptState,
@@ skipping 14 matching lines @@
   }
 
   m_manager.reset();
 }
 
 void PaymentAppManager::contextDestroyed() {
   m_manager.reset();
 }
 
 }  // namespace blink