Add first party sites (PLUS, INBOX, DOCS) in ServiceWorkerSite UMA.

Add first party sites (PLUS, INBOX, DOCS) in ServiceWorkerSite UMA.

This CL depends on https://codereview.chromium.org/2250623003/.

https://codereview.chromium.org/2218943002/ depends on this CL.

BUG=561209
Committed: https://crrev.com/9b4b3efc524314409d7fe235f77947d0e6a6b7b1
Cr-Commit-Position: refs/heads/master@{#413638}

[horo, 2016-08-17 06:45:46]

Description was changed from

==========
Add first party sites for ServiceWorkerSite.

BUG=
==========

to

==========
Add first party sites for ServiceWorkerSite.

This CL depends on https://codereview.chromium.org/2250623003/.

BUG=561209
==========

[horo, 2016-08-17 06:47:06]

The CQ bit was checked by horo@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-17 06:47:29]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[horo, 2016-08-17 06:49:08]

Description was changed from

==========
Add first party sites for ServiceWorkerSite.

This CL depends on https://codereview.chromium.org/2250623003/.

BUG=561209
==========

to

==========
Add first party sites (PLUS, INBOX, DOCS) in ServiceWorkerSite UMA.

This CL depends on https://codereview.chromium.org/2250623003/.

BUG=561209
==========

[horo, 2016-08-17 06:49:30]

horo@chromium.org changed reviewers:
+ shimazu@chromium.org

[horo, 2016-08-17 06:49:30]

shimazu@
Could you please review this?

[horo, 2016-08-17 06:58:05]

Patchset #1 (id:1) has been deleted

[horo, 2016-08-17 07:00:32]

Description was changed from

==========
Add first party sites (PLUS, INBOX, DOCS) in ServiceWorkerSite UMA.

This CL depends on https://codereview.chromium.org/2250623003/.

BUG=561209
==========

to

==========
Add first party sites (PLUS, INBOX, DOCS) in ServiceWorkerSite UMA.

This CL depends on https://codereview.chromium.org/2250623003/.

https://codereview.chromium.org/2218943002/ depends on this CL.

BUG=561209
==========

[shimazu, 2016-08-18 04:49:06]

lgtm

[horo, 2016-08-22 03:19:19]

The CQ bit was checked by horo@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-22 03:19:37]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[horo, 2016-08-22 06:12:02]

horo@chromium.org changed reviewers:
+ mpearson@chromium.org

[horo, 2016-08-22 06:12:03]

mpearson@

Could you please review histograms.xml?

[commit-bot: I haz the power, 2016-08-22 07:47:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-22 07:48:01]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Mark P, 2016-08-22 17:00:53]

histograms.xml lgtm

https://codereview.chromium.org/2249063004/diff/60001/content/browser/service...
File content/browser/service_worker/service_worker_metrics.cc (right):

https://codereview.chromium.org/2249063004/diff/60001/content/browser/service...
content/browser/service_worker/service_worker_metrics.cc:218: if (host ==
"docs.google.com" || host == "drive.google.com")
nit: parens around binary operators (==)

[horo, 2016-08-22 23:14:33]

The CQ bit was checked by horo@chromium.org

[horo, 2016-08-22 23:14:34]

The patchset sent to the CQ was uploaded after l-g-t-m from
shimazu@chromium.org, mpearson@chromium.org
Link to the patchset: https://codereview.chromium.org/2249063004/#ps80001
(title: "incorporated mpearson@'s comment")

[horo, 2016-08-22 23:14:43]

Thank you!

[commit-bot: I haz the power, 2016-08-22 23:15:05]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-23 02:34:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-23 02:34:17]

Failed to apply the patch.
On branch working_branch
Your branch is up-to-date with 'origin/refs/pending/heads/master'.
nothing to commit, working tree clean

[commit-bot: I haz the power, 2016-08-23 02:35:10]

Description was changed from

==========
Add first party sites (PLUS, INBOX, DOCS) in ServiceWorkerSite UMA.

This CL depends on https://codereview.chromium.org/2250623003/.

https://codereview.chromium.org/2218943002/ depends on this CL.

BUG=561209
==========

to

==========
Add first party sites (PLUS, INBOX, DOCS) in ServiceWorkerSite UMA.

This CL depends on https://codereview.chromium.org/2250623003/.

https://codereview.chromium.org/2218943002/ depends on this CL.

BUG=561209

Committed: https://crrev.com/9b4b3efc524314409d7fe235f77947d0e6a6b7b1
Cr-Commit-Position: refs/heads/master@{#413638}
==========

[commit-bot: I haz the power, 2016-08-23 02:35:12]

Patchset 4 (id:??) landed as
https://crrev.com/9b4b3efc524314409d7fe235f77947d0e6a6b7b1
Cr-Commit-Position: refs/heads/master@{#413638}