[Payments] Implement openWindow for service worker based payment handler

content/browser/service_worker/service_worker_version.h

Index: content/browser/service_worker/service_worker_version.h
diff --git a/content/browser/service_worker/service_worker_version.h b/content/browser/service_worker/service_worker_version.h
index 917fe64bf6a7e3eb4836136bace13309dd57bb6c..67d9f70c15c1868879671723ccabe67c0f0e0a18 100644
--- a/content/browser/service_worker/service_worker_version.h
+++ b/content/browser/service_worker/service_worker_version.h
@@ -584,7 +584,7 @@ class CONTENT_EXPORT ServiceWorkerVersion
   void OnGetClients(int request_id,
                     const ServiceWorkerClientQueryOptions& options);
 
-  void OnOpenWindow(int request_id, GURL url);
+  void OnOpenWindow(int request_id, GURL redirect_url, GURL url_to_open);

[falken, 2017/05/31 01:24:50]

redirect_url sounds like an HTTP redirect which doesn't seem to be the case
here. Is redirect_url a term in the spec being implemented?

[gogerald1, 2017/05/31 14:16:21]

It's not a term in the spec, but named according to the parameter
'redirect_chain' in LoadURLParams
(https://cs.chromium.org/chromium/src/content/public/browser/navigation_contro...).

It is used to identify the web page triggering the payment request so as to
caused the openWindow operation, the opened window be shown on top of the web
page.

   void OnOpenWindowFinished(int request_id,
                             ServiceWorkerStatusCode status,
                             const ServiceWorkerClientInfo& client_info);