PaymentHandler: Initial implementation for PaymentInstruments.

third_party/WebKit/Source/modules/payments/PaymentManager.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/PaymentManager.h"
 
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptState.h"
 #include "core/dom/DOMException.h"
 #include "modules/payments/PaymentAppManifest.h"
 #include "modules/payments/PaymentAppOption.h"
+#include "modules/payments/PaymentInstruments.h"
 #include "modules/serviceworkers/ServiceWorkerRegistration.h"
 #include "platform/mojo/MojoHelper.h"
 #include "public/platform/InterfaceProvider.h"
 #include "public/platform/Platform.h"
 
 namespace mojo {
 
 using payments::mojom::blink::PaymentAppManifest;
 using payments::mojom::blink::PaymentAppManifestPtr;
 using payments::mojom::blink::PaymentAppOption;
@@ skipping 99 matching lines @@
   ScriptPromiseResolver* resolver = ScriptPromiseResolver::create(scriptState);
   ScriptPromise promise = resolver->promise();
 
   m_manager->GetManifest(convertToBaseCallback(
       WTF::bind(&PaymentManager::onGetManifest, wrapPersistent(this),
                 wrapPersistent(resolver))));
 
   return promise;
 }
 
+PaymentInstruments* PaymentManager::instruments() {
+  if (!m_instruments) {

[please use gerrit instead, 2017/03/29 21:30:16]

No need for {} in a single-line if statement body.

[zino, 2017/03/30 00:12:53]

Done.

+    m_instruments = new PaymentInstruments();
+  }
+  return m_instruments;
+}
+
 DEFINE_TRACE(PaymentManager) {
   visitor->trace(m_registration);
+  visitor->trace(m_instruments);
 }
 
 PaymentManager::PaymentManager(ServiceWorkerRegistration* registration)
-    : m_registration(registration) {
+    : m_registration(registration), m_instruments(nullptr) {
   DCHECK(registration);
   Platform::current()->interfaceProvider()->getInterface(
       mojo::MakeRequest(&m_manager));
 
   m_manager.set_connection_error_handler(convertToBaseCallback(WTF::bind(
       &PaymentManager::onServiceConnectionError, wrapWeakPersistent(this))));
 
   m_manager->Init(m_registration->scope());
 }
 
@@ skipping 43 matching lines @@
           "No payment app manifest associated with the service worker."));
       break;
   }
 }
 
 void PaymentManager::onServiceConnectionError() {
   m_manager.reset();
 }
 
 }  // namespace blink