[Sync] Move GenerateSyncableHash to base.

[Sync] Move GenerateSyncableHash to base.

It's not only used by directory, so it doesn't make sense to live in
syncable.

BUG=637950
Committed: https://crrev.com/2b0bb91030096c9241d50ec723f63d0f19906b06
Cr-Commit-Position: refs/heads/master@{#432937}

[maxbogue, 2016-11-16 17:51:16]

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

[commit-bot: I haz the power, 2016-11-16 17:51:40]

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

[maxbogue, 2016-11-16 19:53:39]

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

[maxbogue, 2016-11-16 19:53:40]

Sky, PTAL!

[commit-bot: I haz the power, 2016-11-16 19:53:53]

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

[commit-bot: I haz the power, 2016-11-16 19:53:54]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[maxbogue, 2016-11-16 23:31:22]

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

[maxbogue, 2016-11-16 23:31:24]

Actually, when the change processor classes get moved to model/ they'll bring a
ton of syncable/ DEPS so that will stay.

[commit-bot: I haz the power, 2016-11-16 23:34:23]

Dry run: 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-11-17 04:13:59]

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

[commit-bot: I haz the power, 2016-11-17 04:14:00]

Dry run: This issue passed the CQ dry run.

[maxbogue, 2016-11-17 18:12:46]

Description was changed from

==========
[Sync] Move GenerateSyncableHash to base.

It's not only used by directory, so it doesn't make sense to live in
syncable. Additionally, this lets us tighten the DEPS a bit more, although
model/ still depends on kInvalidId from syncable/ (seems to make sense).

BUG=637950
==========

to

==========
[Sync] Move GenerateSyncableHash to base.

It's not only used by directory, so it doesn't make sense to live in
syncable.

BUG=637950
==========

[maxbogue, 2016-11-17 18:15:33]

maxbogue@chromium.org changed reviewers:
+ pavely@chromium.org
- skym@chromium.org

[maxbogue, 2016-11-17 18:15:34]

Reassigning since Sky is out sick; Pavel PTAL!

[pavely, 2016-11-17 19:16:43]

lgtm

[maxbogue, 2016-11-17 19:32:48]

The CQ bit was checked by maxbogue@chromium.org

[commit-bot: I haz the power, 2016-11-17 19:34:24]

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-11-17 20:08:04]

Description was changed from

==========
[Sync] Move GenerateSyncableHash to base.

It's not only used by directory, so it doesn't make sense to live in
syncable.

BUG=637950
==========

to

==========
[Sync] Move GenerateSyncableHash to base.

It's not only used by directory, so it doesn't make sense to live in
syncable.

BUG=637950
==========

[commit-bot: I haz the power, 2016-11-17 20:08:05]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-17 20:28:53]

Description was changed from

==========
[Sync] Move GenerateSyncableHash to base.

It's not only used by directory, so it doesn't make sense to live in
syncable.

BUG=637950
==========

to

==========
[Sync] Move GenerateSyncableHash to base.

It's not only used by directory, so it doesn't make sense to live in
syncable.

BUG=637950

Committed: https://crrev.com/2b0bb91030096c9241d50ec723f63d0f19906b06
Cr-Commit-Position: refs/heads/master@{#432937}
==========

[commit-bot: I haz the power, 2016-11-17 20:28:55]

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