[Sync] Move sessions files from sync_driver to sync_sessions.

[Sync] Move sessions files from sync_driver to sync_sessions.

The sync team has decided that datatype specific code should no longer
live in sync_driver. //c/sync_driver will conceptually be at the same
level as //sync in the dependency tree, with datatype logic living in
special components such as sync_sessions or the datatype components
themselves.

BUG=543199
TBR=jam

Committed: https://crrev.com/8ef2508962ebddf75fe5ec3e7b8284293df733d4
Cr-Commit-Position: refs/heads/master@{#359886}

[maxbogue, 2015-11-13 00:28:17]

Description was changed from

==========
[Sync] Move sessions files from sync_driver to sync_sessions.

Datatype specific code should no longer live in sync_driver.

BUG=543199
==========

to

==========
[Sync] Move sessions files from sync_driver to sync_sessions.

The sync team has decided that datatype specific code should no longer
live in sync_driver. //c/sync_driver will conceptually be at the same
level as //sync in the dependency tree, with datatype logic living in
special components such as sync_sessions or the datatype components
themselves.

BUG=543199
==========

[maxbogue, 2015-11-13 00:30:58]

maxbogue@chromium.org changed reviewers:
+ zea@chromium.org

[maxbogue, 2015-11-13 00:30:59]

Hey Nicolas, PTAL! I think the only datatype dependencies left in sync_driver
after this are for the fake_sync_client; maybe that should be moved to
browser_sync?

[Nicolas Zea, 2015-11-13 23:34:55]

LGTM with one comment

https://codereview.chromium.org/1443453002/diff/40001/components/sync_driver....
File components/sync_driver.gypi (right):

https://codereview.chromium.org/1443453002/diff/40001/components/sync_driver....
components/sync_driver.gypi:17: '../ui/gfx/gfx.gyp:gfx',
Can this one be removed now?

[maxbogue, 2015-11-14 01:43:57]

The CQ bit was checked by maxbogue@chromium.org

[maxbogue, 2015-11-14 01:43:57]

The patchset sent to the CQ was uploaded after l-g-t-m from zea@chromium.org
Link to the patchset: https://codereview.chromium.org/1443453002/#ps60001
(title: "Remove ui/gfx deps.")

[maxbogue, 2015-11-14 01:44:04]

https://codereview.chromium.org/1443453002/diff/40001/components/sync_driver....
File components/sync_driver.gypi (right):

https://codereview.chromium.org/1443453002/diff/40001/components/sync_driver....
components/sync_driver.gypi:17: '../ui/gfx/gfx.gyp:gfx',
On 2015/11/13 23:34:55, Nicolas Zea wrote:
> Can this one be removed now?

Done. I think that was left over from an autofill file or something?

[commit-bot: I haz the power, 2015-11-14 01:44:44]

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

[commit-bot: I haz the power, 2015-11-14 01:58:25]

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

[commit-bot: I haz the power, 2015-11-14 01:58:26]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[maxbogue, 2015-11-16 18:06:40]

maxbogue@chromium.org changed reviewers:
+ jam@chromium.org

[maxbogue, 2015-11-16 18:06:41]

+TBR jam for include changes in:

chrome/browser/android/*
chrome/browser/extensions/*
chrome/browser/ui/*
ios/*

[maxbogue, 2015-11-16 18:06:53]

Description was changed from

==========
[Sync] Move sessions files from sync_driver to sync_sessions.

The sync team has decided that datatype specific code should no longer
live in sync_driver. //c/sync_driver will conceptually be at the same
level as //sync in the dependency tree, with datatype logic living in
special components such as sync_sessions or the datatype components
themselves.

BUG=543199
==========

to

==========
[Sync] Move sessions files from sync_driver to sync_sessions.

The sync team has decided that datatype specific code should no longer
live in sync_driver. //c/sync_driver will conceptually be at the same
level as //sync in the dependency tree, with datatype logic living in
special components such as sync_sessions or the datatype components
themselves.

BUG=543199
TBR=jam
==========

[maxbogue, 2015-11-16 18:06:59]

The CQ bit was checked by maxbogue@chromium.org

[commit-bot: I haz the power, 2015-11-16 18:07:48]

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

[commit-bot: I haz the power, 2015-11-16 19:10:07]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-11-16 19:10:59]

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