PaymentApp: Implement PaymentAppManager.setManifest().

third_party/WebKit/Source/modules/payments/PaymentAppManager.cpp

Index: third_party/WebKit/Source/modules/payments/PaymentAppManager.cpp
diff --git a/third_party/WebKit/Source/modules/payments/PaymentAppManager.cpp b/third_party/WebKit/Source/modules/payments/PaymentAppManager.cpp
index 2ca6830c43ec8116ce190fa351b85eaf7e76e27b..d032f50ac49399e039dd4f33aea6bb2eeb770229 100644
--- a/third_party/WebKit/Source/modules/payments/PaymentAppManager.cpp
+++ b/third_party/WebKit/Source/modules/payments/PaymentAppManager.cpp
@@ -90,10 +90,21 @@ void PaymentAppManager::onSetManifest(
     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();
   }