ServiceWorker: Factor out functions to collect client information from ServiceWorkerVersion

ServiceWorker: Factor out functions to collect client information from ServiceWorkerVersion

Functions for collecting client information (GetWindowClients etc) are not
related to Service Worker lifecycle and it is not neccessary to put them in
ServiceWorkerVersion that is the core class to manage the lifecycle.

For improving readability and maintainability, this patch factors out the
functions into a separate file. This should not change behavior of them.

In addition, this patch renames service_worker_client_navigation_utils namespace
to service_worker_client_utils because this namespace now contains functions not
related to navigation.

BUG=570223
Committed: https://crrev.com/910fc8840b56c94f2a172b3ae7b5c1b3b5108985
Cr-Commit-Position: refs/heads/master@{#367485}

[nhiroki, 2015-12-18 14:18:20]

Description was changed from

==========
(WIP) ServiceWorker: Factor out functions to collect client information from
SWVersion

BUG=570223
==========

to

==========
ServiceWorker: Factor out functions to collect client information from SWVersion

Functions for collecting client information (GetWindowClients etc) are not
related to Service Worker lifecycle and it is not neccessary to put them in
ServiceWorkerVersion that is the core class to manage the lifecycle.

For improving readability and maintainability, this patch factors out the
functions into a separate file. This should not change behavior of them.

In addition, this patch also renames service_worker_client_navigation_utils
namespace to service_worker_client_utils because this namespace now contains
functions not related to navigation.

BUG=570223
==========

[nhiroki, 2015-12-18 14:18:56]

Description was changed from

==========
ServiceWorker: Factor out functions to collect client information from SWVersion

Functions for collecting client information (GetWindowClients etc) are not
related to Service Worker lifecycle and it is not neccessary to put them in
ServiceWorkerVersion that is the core class to manage the lifecycle.

For improving readability and maintainability, this patch factors out the
functions into a separate file. This should not change behavior of them.

In addition, this patch also renames service_worker_client_navigation_utils
namespace to service_worker_client_utils because this namespace now contains
functions not related to navigation.

BUG=570223
==========

to

==========
ServiceWorker: Factor out functions to collect client information from
ServiceWorkerVersion

Functions for collecting client information (GetWindowClients etc) are not
related to Service Worker lifecycle and it is not neccessary to put them in
ServiceWorkerVersion that is the core class to manage the lifecycle.

For improving readability and maintainability, this patch factors out the
functions into a separate file. This should not change behavior of them.

In addition, this patch also renames service_worker_client_navigation_utils
namespace to service_worker_client_utils because this namespace now contains
functions not related to navigation.

BUG=570223
==========

[nhiroki, 2015-12-22 11:03:17]

Description was changed from

==========
ServiceWorker: Factor out functions to collect client information from
ServiceWorkerVersion

Functions for collecting client information (GetWindowClients etc) are not
related to Service Worker lifecycle and it is not neccessary to put them in
ServiceWorkerVersion that is the core class to manage the lifecycle.

For improving readability and maintainability, this patch factors out the
functions into a separate file. This should not change behavior of them.

In addition, this patch also renames service_worker_client_navigation_utils
namespace to service_worker_client_utils because this namespace now contains
functions not related to navigation.

BUG=570223
==========

to

==========
ServiceWorker: Factor out functions to collect client information from
ServiceWorkerVersion

Functions for collecting client information (GetWindowClients etc) are not
related to Service Worker lifecycle and it is not neccessary to put them in
ServiceWorkerVersion that is the core class to manage the lifecycle.

For improving readability and maintainability, this patch factors out the
functions into a separate file. This should not change behavior of them.

In addition, this patch renames service_worker_client_navigation_utils namespace
to service_worker_client_utils because this namespace now contains functions not
related to navigation.

BUG=570223
==========

[nhiroki, 2015-12-22 12:38:23]

Patchset #1 (id:1) has been deleted

[nhiroki, 2015-12-22 12:38:31]

Patchset #1 (id:20001) has been deleted

[nhiroki, 2015-12-22 12:38:39]

Patchset #1 (id:40001) has been deleted

[nhiroki, 2015-12-22 12:38:49]

Patchset #1 (id:60001) has been deleted

[nhiroki, 2015-12-22 12:39:01]

Patchset #1 (id:80001) has been deleted

[nhiroki, 2015-12-22 12:46:37]

Description was changed from

==========
ServiceWorker: Factor out functions to collect client information from
ServiceWorkerVersion

Functions for collecting client information (GetWindowClients etc) are not
related to Service Worker lifecycle and it is not neccessary to put them in
ServiceWorkerVersion that is the core class to manage the lifecycle.

For improving readability and maintainability, this patch factors out the
functions into a separate file. This should not change behavior of them.

In addition, this patch renames service_worker_client_navigation_utils namespace
to service_worker_client_utils because this namespace now contains functions not
related to navigation.

BUG=570223
==========

to

==========
ServiceWorker: Factor out functions to collect client information from
ServiceWorkerVersion

Functions for collecting client information (GetWindowClients etc) are not
related to Service Worker lifecycle and it is not neccessary to put them in
ServiceWorkerVersion that is the core class to manage the lifecycle.

For improving readability and maintainability, this patch factors out the
functions into a separate file. This should not change behavior of them.

In addition, this patch renames service_worker_client_navigation_utils namespace
to service_worker_client_utils because this namespace now contains functions not
related to navigation.

BUG=570223
==========

[nhiroki, 2015-12-22 12:46:38]

nhiroki@chromium.org changed reviewers:
+ falken@chromium.org

[nhiroki, 2015-12-22 12:46:49]

PTAL, thanks!

[falken, 2016-01-05 05:10:54]

Thanks for cleaning up SWVersion! lgtm

[nhiroki, 2016-01-05 05:11:54]

The CQ bit was checked by nhiroki@chromium.org

[commit-bot: I haz the power, 2016-01-05 05:12:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1535983002/120001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1535983002/120001

[commit-bot: I haz the power, 2016-01-05 05:16:35]

Description was changed from

==========
ServiceWorker: Factor out functions to collect client information from
ServiceWorkerVersion

Functions for collecting client information (GetWindowClients etc) are not
related to Service Worker lifecycle and it is not neccessary to put them in
ServiceWorkerVersion that is the core class to manage the lifecycle.

For improving readability and maintainability, this patch factors out the
functions into a separate file. This should not change behavior of them.

In addition, this patch renames service_worker_client_navigation_utils namespace
to service_worker_client_utils because this namespace now contains functions not
related to navigation.

BUG=570223
==========

to

==========
ServiceWorker: Factor out functions to collect client information from
ServiceWorkerVersion

Functions for collecting client information (GetWindowClients etc) are not
related to Service Worker lifecycle and it is not neccessary to put them in
ServiceWorkerVersion that is the core class to manage the lifecycle.

For improving readability and maintainability, this patch factors out the
functions into a separate file. This should not change behavior of them.

In addition, this patch renames service_worker_client_navigation_utils namespace
to service_worker_client_utils because this namespace now contains functions not
related to navigation.

BUG=570223
==========

[commit-bot: I haz the power, 2016-01-05 05:16:36]

Committed patchset #2 (id:120001)

[commit-bot: I haz the power, 2016-01-05 05:17:25]

Description was changed from

==========
ServiceWorker: Factor out functions to collect client information from
ServiceWorkerVersion

Functions for collecting client information (GetWindowClients etc) are not
related to Service Worker lifecycle and it is not neccessary to put them in
ServiceWorkerVersion that is the core class to manage the lifecycle.

For improving readability and maintainability, this patch factors out the
functions into a separate file. This should not change behavior of them.

In addition, this patch renames service_worker_client_navigation_utils namespace
to service_worker_client_utils because this namespace now contains functions not
related to navigation.

BUG=570223
==========

to

==========
ServiceWorker: Factor out functions to collect client information from
ServiceWorkerVersion

Functions for collecting client information (GetWindowClients etc) are not
related to Service Worker lifecycle and it is not neccessary to put them in
ServiceWorkerVersion that is the core class to manage the lifecycle.

For improving readability and maintainability, this patch factors out the
functions into a separate file. This should not change behavior of them.

In addition, this patch renames service_worker_client_navigation_utils namespace
to service_worker_client_utils because this namespace now contains functions not
related to navigation.

BUG=570223

Committed: https://crrev.com/910fc8840b56c94f2a172b3ae7b5c1b3b5108985
Cr-Commit-Position: refs/heads/master@{#367485}
==========

[commit-bot: I haz the power, 2016-01-05 05:17:26]

Patchset 2 (id:??) landed as
https://crrev.com/910fc8840b56c94f2a172b3ae7b5c1b3b5108985
Cr-Commit-Position: refs/heads/master@{#367485}