Service Worker: Add Clients.get(id)

content/common/service_worker/service_worker_client_info.cc

Index: content/common/service_worker/service_worker_client_info.cc
diff --git a/content/common/service_worker/service_worker_client_info.cc b/content/common/service_worker/service_worker_client_info.cc
index c864369c0390b67f978afd8652fd16109103c88a..50195750127cce4b14deeb7e13050665307bd673 100644
--- a/content/common/service_worker/service_worker_client_info.cc
+++ b/content/common/service_worker/service_worker_client_info.cc
@@ -10,25 +10,29 @@
 namespace content {
 
 ServiceWorkerClientInfo::ServiceWorkerClientInfo()
-    : page_visibility_state(blink::WebPageVisibilityStateLast),
-      is_focused(false),
-      frame_type(REQUEST_CONTEXT_FRAME_TYPE_LAST),
-      client_type(blink::WebServiceWorkerClientTypeLast),
-      last_focus_time(base::TimeTicks()) {}
+    : ServiceWorkerClientInfo(std::string(),
+                              blink::WebPageVisibilityStateLast,
+                              false,
+                              GURL(),
+                              REQUEST_CONTEXT_FRAME_TYPE_LAST,
+                              base::TimeTicks(),
+                              blink::WebServiceWorkerClientTypeLast) {}
 
 ServiceWorkerClientInfo::ServiceWorkerClientInfo(
+    const std::string& client_uuid,

[zino, 2016/02/17 18:38:13]

After this change, I think we can remove setting client_uuid in some callbacks.
It's duplicated unnecessarily.
 - ServiceWorkerVersion::OnFocusClientFinished
 - ServiceWorkerVersion::OnNavigateClientFinished
 - ServiceWorkerVersion::OnOpenWindowFinished

[jungkees, 2016/02/18 15:02:20]

Good point. I refactored those functions a bit according to the suggestion.
Thanks.

     blink::WebPageVisibilityState page_visibility_state,
     bool is_focused,
     const GURL& url,
     RequestContextFrameType frame_type,
     base::TimeTicks last_focus_time,
     blink::WebServiceWorkerClientType client_type)
-    : page_visibility_state(page_visibility_state),
+    : client_uuid(client_uuid),
+      page_visibility_state(page_visibility_state),
       is_focused(is_focused),
       url(url),
       frame_type(frame_type),
-      client_type(client_type),
-      last_focus_time(last_focus_time) {}
+      last_focus_time(last_focus_time),
+      client_type(client_type) {}
 
 bool ServiceWorkerClientInfo::IsEmpty() const {
   return page_visibility_state == blink::WebPageVisibilityStateLast &&