Implement ServiceWorkerClient attributes [1/3]

public/platform/WebServiceWorkerClientsInfo.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef WebServiceWorkerClientsInfo_h
 #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;
 };
 
 typedef WebCallbacks<WebServiceWorkerClientsInfo, WebServiceWorkerError> WebServiceWorkerClientsCallbacks;
 
 } // namespace blink
 
 #endif // WebServiceWorkerClientsInfo_h