[ServiceWorker] Add type attribute into Client object

third_party/WebKit/Source/modules/serviceworkers/ServiceWorkerClient.cpp

Index: third_party/WebKit/Source/modules/serviceworkers/ServiceWorkerClient.cpp
diff --git a/third_party/WebKit/Source/modules/serviceworkers/ServiceWorkerClient.cpp b/third_party/WebKit/Source/modules/serviceworkers/ServiceWorkerClient.cpp
index e54315d65277fed9fa01869b3dd762bb52a1f38c..b10caa573457bc18f052b0f65b5d622f08d5409b 100644
--- a/third_party/WebKit/Source/modules/serviceworkers/ServiceWorkerClient.cpp
+++ b/third_party/WebKit/Source/modules/serviceworkers/ServiceWorkerClient.cpp
@@ -45,10 +45,28 @@ ServiceWorkerClient* ServiceWorkerClient::Create(
 ServiceWorkerClient::ServiceWorkerClient(const WebServiceWorkerClientInfo& info)
     : uuid_(info.uuid),
       url_(info.url.GetString()),
+      type_(info.client_type),
       frame_type_(info.frame_type) {}
 
 ServiceWorkerClient::~ServiceWorkerClient() {}
 
+String ServiceWorkerClient::type() const {
+  switch (type_) {
+    case kWebServiceWorkerClientTypeWindow:
+      return "window";
+    case kWebServiceWorkerClientTypeWorker:
+      return "worker";
+    case kWebServiceWorkerClientTypeSharedWorker:
+      return "sharedworker";
+    case kWebServiceWorkerClientTypeLast:

[shimazu, 2017/05/01 06:40:35]

I prefer kWebServiceWorkerClientTypeAll to Last to clarify what is prohibited.
Just removing this case looks also good.

[leonhsl(Using Gerrit), 2017/05/02 03:14:35]

Done.

+      NOTREACHED();
+      return String();
+  }
+
+  NOTREACHED();
+  return String();
+}
+
 String ServiceWorkerClient::frameType() const {
   switch (frame_type_) {
     case WebURLRequest::kFrameTypeAuxiliary: