[Patch 5 of 6] Split out the site_isolation_policy files

This CL moves classes (refactored in situ, in previous patches) into
appropriately named files.

New classes are CrossSiteDocumentClassifier and SiteIsolationStatsGatherer.
CrossSiteDocumentClassifier is moved to content/common so we can use it from
the browser process.

No files should be left now with the name site_isolation_policy*.*

BUG=481066
Committed: https://crrev.com/e7cd12af5b21f71f1e8dc8a2f5dfb476d44afa0b
Cr-Commit-Position: refs/heads/master@{#334779}

[ncarter (slow), 2015-06-11 06:31:34]

nick@chromium.org changed reviewers:
+ alexmos@chromium.org

[ncarter (slow), 2015-06-11 06:31:34]

alexmos: this is the last refactor step, just renaming/splitting files.

[alexmos, 2015-06-13 00:27:04]

LGTM

https://codereview.chromium.org/1174323002/diff/40001/content/child/site_isol...
File content/child/site_isolation_stats_gatherer_browsertest.cc (right):

https://codereview.chromium.org/1174323002/diff/40001/content/child/site_isol...
content/child/site_isolation_stats_gatherer_browsertest.cc:1: // Copyright 2013
The Chromium Authors. All rights reserved.
nit: 2015

https://codereview.chromium.org/1174323002/diff/40001/content/common/cross_si...
File content/common/cross_site_document_classifier.cc (right):

https://codereview.chromium.org/1174323002/diff/40001/content/common/cross_si...
content/common/cross_site_document_classifier.cc:1: // Copyright 2013 The
Chromium Authors. All rights reserved.
2015

[ncarter (slow), 2015-06-16 22:43:22]

nick@chromium.org changed reviewers:
+ creis@chromium.org

[ncarter (slow), 2015-06-16 22:43:23]

creis@chromium.org: Please review changes in content/

https://codereview.chromium.org/1174323002/diff/40001/content/child/site_isol...
File content/child/site_isolation_stats_gatherer_browsertest.cc (right):

https://codereview.chromium.org/1174323002/diff/40001/content/child/site_isol...
content/child/site_isolation_stats_gatherer_browsertest.cc:1: // Copyright 2013
The Chromium Authors. All rights reserved.
On 2015/06/13 00:27:03, alexmos wrote:
> nit: 2015

Done.

https://codereview.chromium.org/1174323002/diff/40001/content/common/cross_si...
File content/common/cross_site_document_classifier.cc (right):

https://codereview.chromium.org/1174323002/diff/40001/content/common/cross_si...
content/common/cross_site_document_classifier.cc:1: // Copyright 2013 The
Chromium Authors. All rights reserved.
On 2015/06/13 00:27:03, alexmos wrote:
> 2015

Done.

[Charlie Reis, 2015-06-16 23:22:07]

LGTM.  Do any GN files need updating?  (I haven't had to deal with that yet
myself, but I've seen CLs reverted for it.)

[ncarter (slow), 2015-06-17 03:59:22]

On 2015/06/16 23:22:07, Charlie Reis wrote:
> LGTM.  Do any GN files need updating?  (I haven't had to deal with that yet
> myself, but I've seen CLs reverted for it.)

I think the rule of thumb is: gypi files are used by both gyp and gn, so we're
good here.

[ncarter (slow), 2015-06-17 05:43:16]

The CQ bit was checked by nick@chromium.org

[ncarter (slow), 2015-06-17 05:43:19]

The patchset sent to the CQ was uploaded after l-g-t-m from alexmos@chromium.org
Link to the patchset: https://codereview.chromium.org/1174323002/#ps140001
(title: "Re-upload.")

[commit-bot: I haz the power, 2015-06-17 05:44:18]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1174323002/140001

[commit-bot: I haz the power, 2015-06-17 06:48:48]

Committed patchset #8 (id:140001)

[commit-bot: I haz the power, 2015-06-17 06:49:37]

Patchset 8 (id:??) landed as
https://crrev.com/e7cd12af5b21f71f1e8dc8a2f5dfb476d44afa0b
Cr-Commit-Position: refs/heads/master@{#334779}