Implement ServiceWorkerClient attributes [1/3]

public/platform/WebServiceWorkerClientsInfo.h

Index: public/platform/WebServiceWorkerClientsInfo.h
diff --git a/public/platform/WebServiceWorkerClientsInfo.h b/public/platform/WebServiceWorkerClientsInfo.h
index 5fde9f43627691355b7fd224115046dd2f639986..5ba9ddc0d6df830cb36733ee2f2b5b0e426abeef 100644
--- a/public/platform/WebServiceWorkerClientsInfo.h
+++ b/public/platform/WebServiceWorkerClientsInfo.h
@@ -6,13 +6,27 @@
 #define WebServiceWorkerClientsInfo_h
 
 #include "WebCallbacks.h"
+#include "WebURL.h"

[tkent, 2014/12/08 04:54:25]

This should be "public/platform/WebURL.h"

It's inconsistent with the existing #includes.  You may ignore this comment, or
you may update the existing #includes.

[Kunihiko Sakamoto, 2014/12/08 05:12:04]

Fixed.

+#include "WebURLRequest.h"

[tkent, 2014/12/08 04:54:25]

should be "public/platform/WebURLRequest.h"

[Kunihiko Sakamoto, 2014/12/08 05:12:04]

Done.

 #include "WebVector.h"
 
 namespace blink {
 
 struct WebServiceWorkerError;
 
+struct WebServiceWorkerClientInfo {
+    int clientID;
+    // FIXME: Use WebPageVisibilityState? That will require moving
+    // WebPageVisibilityState from public/web to public/platform.
+    WebString visibilityState;
+    bool focused;

[tkent, 2014/12/08 04:54:25]

Probably, this should be |isFocused|.

[Kunihiko Sakamoto, 2014/12/08 05:12:04]

Wanted to match with the IDL attribute name, but isFocused seems better for code
readability. Done.

+    WebURL url;
+    WebURLRequest::FrameType frameType;
+};
+
 struct WebServiceWorkerClientsInfo {
+    WebVector<WebServiceWorkerClientInfo> clients;
+    // FIXME: Remove this once the embedder stops using it.
     WebVector<int> clientIDs;
 };