[Payments] Implement openWindow for service worker based payment handler

third_party/WebKit/public/web/modules/serviceworker/WebServiceWorkerContextClient.h

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 254 matching lines @@
   // WebServiceWorkerClientsInfo and WebServiceWorkerError ownerships are
   // passed to the WebServiceWorkerClientsCallbacks implementation.
   virtual void GetClients(
       const WebServiceWorkerClientQueryOptions&,
       std::unique_ptr<WebServiceWorkerClientsCallbacks>) = 0;
 
   // Ownership of the passed callbacks is transferred to the callee, callee
   // should delete the callbacks after calling either onSuccess or onError.
   // WebServiceWorkerClientInfo and WebServiceWorkerError ownerships are
   // passed to the WebServiceWorkerClientsCallbacks implementation.
-  virtual void OpenWindow(const WebURL&,
-                          std::unique_ptr<WebServiceWorkerClientCallbacks>) = 0;
+  virtual void OpenWindowForClients(
+      const WebURL&,
+      std::unique_ptr<WebServiceWorkerClientCallbacks>) = 0;
+
+  // Similar like OpenWindowForClients above.
+  virtual void OpenWindowForPaymentHandler(

[dcheng, 2017/06/06 21:18:06]

Btw, my point was that we should use a consistent name for this throughout,
including in the Blink public API. Do you mind fixing this in a followup?

+      const WebURL&,
+      std::unique_ptr<WebServiceWorkerClientCallbacks>) = 0;
 
   // A suggestion to cache this metadata in association with this URL.
   virtual void SetCachedMetadata(const WebURL& url,
                                  const char* data,
                                  size_t size) {}
 
   // A suggestion to clear the cached metadata in association with this URL.
   virtual void ClearCachedMetadata(const WebURL& url) {}
 
   // Callee receives ownership of the passed vector.
@@ skipping 29 matching lines @@
   // fetch. Will only be called while an install event is in progress.
   virtual void RegisterForeignFetchScopes(
       int install_event_id,
       const WebVector<WebURL>& sub_scopes,
       const WebVector<WebSecurityOrigin>& origins) = 0;
 };
 
 }  // namespace blink
 
 #endif  // WebServiceWorkerContextClient_h