[Payments] Implement openWindow for service worker based payment handler

third_party/WebKit/Source/modules/serviceworkers/ServiceWorkerClients.h

 // Copyright 2014 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.
 
 #ifndef ServiceWorkerClients_h
 #define ServiceWorkerClients_h
 
 #include "modules/serviceworkers/ClientQueryOptions.h"
 #include "platform/bindings/ScriptWrappable.h"
 #include "platform/heap/Handle.h"
@@ skipping 11 matching lines @@
 
  public:
   static ServiceWorkerClients* Create();
 
   // Clients.idl
   ScriptPromise get(ScriptState*, const String& id);
   ScriptPromise matchAll(ScriptState*, const ClientQueryOptions&);
   ScriptPromise openWindow(ScriptState*, const String& url);
   ScriptPromise claim(ScriptState*);
 
+  ScriptPromise openWindow(ScriptState*,

[please use gerrit instead, 2017/05/25 14:53:27]

Perhaps a better name for this method would be a more explicit
"openWindowForPaymentHandler". That would clarify that this openWindow() is not
part of the Clients.idl.

[gogerald1, 2017/05/25 16:53:32]

This interface is also used by service worker, so I add a comment to explain it.

[zino, 2017/05/25 17:02:42]

+1 I agree with rouslan@'s opinion.

[gogerald1, 2017/05/29 22:13:16]

Done.

+                           const String& redirect_url,
+                           const String& url);
+
   DEFINE_INLINE_TRACE() {}
 
  private:
   ServiceWorkerClients();
 };
 
 }  // namespace blink
 
 #endif  // ServiceWorkerClients_h