[Sync] Move //sync to //components/sync.

[Sync] Move //sync to //components/sync.

There are a bunch of subfolder reorganizations going along with this:

- engine/ -> engine_impl/
- sessions/ -> sessions_impl/
- util/ -> base/
- internal_api/public/base/ -> base/
- internal_api/public/sessions/ -> sessions/
- internal_api/public/engine/ -> engine/
- internal_api/public/events/ -> engine/events/
- internal_api/public/util/ -> base/
- internal_api/public/ -> core/
- internal_api/ -> core_impl/

This change was generated using the following script:
http://pastebin.com/vVAkRCkh

Because this patch is so large, some review was done in split CLs:

Manual changes: http://crrev.com/2155763002
Sync internal changes: http://crrev.com/2156693002
Sync external changes: http://crrev.com/2148293004

This is a minimal change to support file movement; it doesn't do
anything like merging sync_unit_tests into component_unit_tests,
which will be done in a followup.

The gyp files are not sorted because they're all going to be removed
very soon anyways.

BUG=626114
R=jam@chromium.org, zea@chromium.org
TBR=jam

Committed: https://crrev.com/fef332d69faf129b64689e30304061322ca9bcad
Cr-Commit-Position: refs/heads/master@{#408497}

[maxbogue, 2016-07-14 22:35:52]

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

[commit-bot: I haz the power, 2016-07-14 22:36:40]

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-07-15 00:10:14]

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

[commit-bot: I haz the power, 2016-07-15 00:10:22]

Dry run: This issue passed the CQ dry run.

[maxbogue, 2016-07-15 18:02:18]

Description was changed from

==========
[Sync] Move //sync to //components/sync.

There are a bunch of subfolder reorganizations going along with this:

- engine/ -> engine_impl/
- sessions/ -> sessions_impl/
- util/ -> base/
- internal_api/public/base/ -> base/
- internal_api/public/sessions/ -> sessions/
- internal_api/public/engine/ -> engine/
- internal_api/public/events/protocol_event -> engine/events/protocol_event
- internal_api/public/events/ -> engine_impl/events/
- internal_api/public/util/ -> base/
- internal_api/public/ -> core/
- internal_api/ -> core_impl/

BUG=626114
==========

to

==========
[Sync] Move //sync to //components/sync.

There are a bunch of subfolder reorganizations going along with this:

- engine/ -> engine_impl/
- sessions/ -> sessions_impl/
- util/ -> base/
- internal_api/public/base/ -> base/
- internal_api/public/sessions/ -> sessions/
- internal_api/public/engine/ -> engine/
- internal_api/public/events/protocol_event -> engine/events/protocol_event
- internal_api/public/events/ -> engine_impl/events/
- internal_api/public/util/ -> base/
- internal_api/public/ -> core/
- internal_api/ -> core_impl/

This change was generated using the following script:
http://pastebin.com/vVAkRCkh

The manual changes that are cherry-picked in can be found in the
patch before the most recent patch.

This is a minimal change to support file movement; it doesn't do
anything like merging sync_unit_tests into component_unit tests.

The gyp files are not sorted because they're all going to be removed
very soon anyways.

BUG=626114
==========

[maxbogue, 2016-07-15 18:05:27]

Description was changed from

==========
[Sync] Move //sync to //components/sync.

There are a bunch of subfolder reorganizations going along with this:

- engine/ -> engine_impl/
- sessions/ -> sessions_impl/
- util/ -> base/
- internal_api/public/base/ -> base/
- internal_api/public/sessions/ -> sessions/
- internal_api/public/engine/ -> engine/
- internal_api/public/events/protocol_event -> engine/events/protocol_event
- internal_api/public/events/ -> engine_impl/events/
- internal_api/public/util/ -> base/
- internal_api/public/ -> core/
- internal_api/ -> core_impl/

This change was generated using the following script:
http://pastebin.com/vVAkRCkh

The manual changes that are cherry-picked in can be found in the
patch before the most recent patch.

This is a minimal change to support file movement; it doesn't do
anything like merging sync_unit_tests into component_unit tests.

The gyp files are not sorted because they're all going to be removed
very soon anyways.

BUG=626114
==========

to

==========
[Sync] Move //sync to //components/sync.

There are a bunch of subfolder reorganizations going along with this:

- engine/ -> engine_impl/
- sessions/ -> sessions_impl/
- util/ -> base/
- internal_api/public/base/ -> base/
- internal_api/public/sessions/ -> sessions/
- internal_api/public/engine/ -> engine/
- internal_api/public/events/ -> engine/events/
- internal_api/public/util/ -> base/
- internal_api/public/ -> core/
- internal_api/ -> core_impl/

This change was generated using the following script:
http://pastebin.com/vVAkRCkh

The manual changes that are cherry-picked in can be found in the
patch before the most recent patch.

This is a minimal change to support file movement; it doesn't do
anything like merging sync_unit_tests into component_unit tests.

The gyp files are not sorted because they're all going to be removed
very soon anyways.

BUG=626114
==========

[maxbogue, 2016-07-15 18:14:40]

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

[maxbogue, 2016-07-15 18:14:53]

Hey Nicolas, PTAL! I think this is ready to start reviewing. It's impossible to
look at every file, but please look at the script I used, the manual changes,
and any non-cc/h/gn/gyp files.

[maxbogue, 2016-07-18 22:29:55]

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

[commit-bot: I haz the power, 2016-07-18 22:30:37]

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-07-19 02:07:57]

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

[commit-bot: I haz the power, 2016-07-19 02:08:06]

Dry run: This issue passed the CQ dry run.

[maxbogue, 2016-07-19 17:37:33]

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

[maxbogue, 2016-07-19 17:37:40]

+Colin for //component owners.

[maxbogue, 2016-07-25 22:17:10]

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

[maxbogue, 2016-07-25 22:17:17]

-blundell +jam, here is the full CL (which is slow to load in Rietveld).

[maxbogue, 2016-07-25 22:24:32]

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

[commit-bot: I haz the power, 2016-07-25 22:25:13]

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-07-25 23:39:51]

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

[commit-bot: I haz the power, 2016-07-25 23:40:01]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[jam, 2016-07-26 17:31:04]

rs lgtm based on the changes I reviewed before

since you mentioned sync_unit_tests, in the future that'll be merged with
components_unit_tests right?

[maxbogue, 2016-07-26 18:01:16]

Description was changed from

==========
[Sync] Move //sync to //components/sync.

There are a bunch of subfolder reorganizations going along with this:

- engine/ -> engine_impl/
- sessions/ -> sessions_impl/
- util/ -> base/
- internal_api/public/base/ -> base/
- internal_api/public/sessions/ -> sessions/
- internal_api/public/engine/ -> engine/
- internal_api/public/events/ -> engine/events/
- internal_api/public/util/ -> base/
- internal_api/public/ -> core/
- internal_api/ -> core_impl/

This change was generated using the following script:
http://pastebin.com/vVAkRCkh

The manual changes that are cherry-picked in can be found in the
patch before the most recent patch.

This is a minimal change to support file movement; it doesn't do
anything like merging sync_unit_tests into component_unit tests.

The gyp files are not sorted because they're all going to be removed
very soon anyways.

BUG=626114
==========

to

==========
[Sync] Move //sync to //components/sync.

There are a bunch of subfolder reorganizations going along with this:

- engine/ -> engine_impl/
- sessions/ -> sessions_impl/
- util/ -> base/
- internal_api/public/base/ -> base/
- internal_api/public/sessions/ -> sessions/
- internal_api/public/engine/ -> engine/
- internal_api/public/events/ -> engine/events/
- internal_api/public/util/ -> base/
- internal_api/public/ -> core/
- internal_api/ -> core_impl/

This change was generated using the following script:
http://pastebin.com/vVAkRCkh

Because this patch is so large, some review was done in split CLs:

Manual changes: http://crrev.com/2155763002
Sync internal changes: http://crrev.com/2156693002
Sync external changes: http://crrev.com/2148293004

This is a minimal change to support file movement; it doesn't do
anything like merging sync_unit_tests into component_unit_tests,
which will be done in a followup.

The gyp files are not sorted because they're all going to be removed
very soon anyways.

BUG=626114
==========

[maxbogue, 2016-07-26 18:04:52]

On 2016/07/26 17:31:04, jam wrote:
> rs lgtm based on the changes I reviewed before
> 
> since you mentioned sync_unit_tests, in the future that'll be merged with
> components_unit_tests right?

Yep! That will be my first follow-up CL.

[Nicolas Zea, 2016-07-26 18:20:06]

rs lgtm based on other reviews

[maxbogue, 2016-07-28 03:43:43]

The CQ bit was checked by maxbogue@chromium.org

[maxbogue, 2016-07-28 03:43:48]

The patchset sent to the CQ was uploaded after l-g-t-m from zea@chromium.org,
jam@chromium.org
Link to the patchset: https://codereview.chromium.org/2130453004/#ps120001
(title: "Rebase.")

[commit-bot: I haz the power, 2016-07-28 03:44:07]

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-07-28 04:36:33]

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

[commit-bot: I haz the power, 2016-07-28 04:36:44]

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

[maxbogue, 2016-07-28 04:49:52]

Description was changed from

==========
[Sync] Move //sync to //components/sync.

There are a bunch of subfolder reorganizations going along with this:

- engine/ -> engine_impl/
- sessions/ -> sessions_impl/
- util/ -> base/
- internal_api/public/base/ -> base/
- internal_api/public/sessions/ -> sessions/
- internal_api/public/engine/ -> engine/
- internal_api/public/events/ -> engine/events/
- internal_api/public/util/ -> base/
- internal_api/public/ -> core/
- internal_api/ -> core_impl/

This change was generated using the following script:
http://pastebin.com/vVAkRCkh

Because this patch is so large, some review was done in split CLs:

Manual changes: http://crrev.com/2155763002
Sync internal changes: http://crrev.com/2156693002
Sync external changes: http://crrev.com/2148293004

This is a minimal change to support file movement; it doesn't do
anything like merging sync_unit_tests into component_unit_tests,
which will be done in a followup.

The gyp files are not sorted because they're all going to be removed
very soon anyways.

BUG=626114
==========

to

==========
[Sync] Move //sync to //components/sync.

There are a bunch of subfolder reorganizations going along with this:

- engine/ -> engine_impl/
- sessions/ -> sessions_impl/
- util/ -> base/
- internal_api/public/base/ -> base/
- internal_api/public/sessions/ -> sessions/
- internal_api/public/engine/ -> engine/
- internal_api/public/events/ -> engine/events/
- internal_api/public/util/ -> base/
- internal_api/public/ -> core/
- internal_api/ -> core_impl/

This change was generated using the following script:
http://pastebin.com/vVAkRCkh

Because this patch is so large, some review was done in split CLs:

Manual changes: http://crrev.com/2155763002
Sync internal changes: http://crrev.com/2156693002
Sync external changes: http://crrev.com/2148293004

This is a minimal change to support file movement; it doesn't do
anything like merging sync_unit_tests into component_unit_tests,
which will be done in a followup.

The gyp files are not sorted because they're all going to be removed
very soon anyways.

BUG=626114
TBR=jam
==========

[maxbogue, 2016-07-28 04:54:06]

The CQ bit was checked by maxbogue@chromium.org

[commit-bot: I haz the power, 2016-07-28 04:54:35]

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-07-28 06:00:11]

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

[commit-bot: I haz the power, 2016-07-28 06:00:22]

CQ experienced an internal error when committing your CL and the maintainers
were notified. Sorry for the inconvenience.

[maxbogue, 2016-07-28 06:22:24]

The CQ bit was checked by maxbogue@chromium.org

[commit-bot: I haz the power, 2016-07-28 06:22:45]

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

[maxbogue, 2016-07-28 06:54:04]

The CQ bit was checked by maxbogue@chromium.org

[commit-bot: I haz the power, 2016-07-28 06:59:15]

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

[commit-bot: I haz the power, 2016-07-28 06:59:26]

CQ experienced an internal error when committing your CL and the maintainers
were notified. Sorry for the inconvenience.

[maxbogue, 2016-07-28 07:13:19]

Tried committing a few times. First time I failed the OWNERS check, which is
interesting given jam is in src/OWNERS. Second and third times I got "Committing
failed: CQ experienced an internal error when committing your CL and the
maintainers were notified. Sorry for the inconvenience.."

So maybe this CL is *literally* too big to land? tbd tomorrow...

[maxbogue, 2016-07-28 19:04:31]

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

[commit-bot: I haz the power, 2016-07-28 19:05:11]

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-07-28 19:20:42]

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

[commit-bot: I haz the power, 2016-07-28 19:20:51]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  chromeos_x86-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)
  linux_chromium_chromeos_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_clobber_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[maxbogue, 2016-07-28 19:47:53]

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

[commit-bot: I haz the power, 2016-07-28 19:48:33]

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-07-28 21:48:24]

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

[commit-bot: I haz the power, 2016-07-28 21:48:33]

Dry run: This issue passed the CQ dry run.

[commit-bot: I haz the power, 2016-07-28 22:14:56]

Description was changed from

==========
[Sync] Move //sync to //components/sync.

There are a bunch of subfolder reorganizations going along with this:

- engine/ -> engine_impl/
- sessions/ -> sessions_impl/
- util/ -> base/
- internal_api/public/base/ -> base/
- internal_api/public/sessions/ -> sessions/
- internal_api/public/engine/ -> engine/
- internal_api/public/events/ -> engine/events/
- internal_api/public/util/ -> base/
- internal_api/public/ -> core/
- internal_api/ -> core_impl/

This change was generated using the following script:
http://pastebin.com/vVAkRCkh

Because this patch is so large, some review was done in split CLs:

Manual changes: http://crrev.com/2155763002
Sync internal changes: http://crrev.com/2156693002
Sync external changes: http://crrev.com/2148293004

This is a minimal change to support file movement; it doesn't do
anything like merging sync_unit_tests into component_unit_tests,
which will be done in a followup.

The gyp files are not sorted because they're all going to be removed
very soon anyways.

BUG=626114
TBR=jam
==========

to

==========
[Sync] Move //sync to //components/sync.

There are a bunch of subfolder reorganizations going along with this:

- engine/ -> engine_impl/
- sessions/ -> sessions_impl/
- util/ -> base/
- internal_api/public/base/ -> base/
- internal_api/public/sessions/ -> sessions/
- internal_api/public/engine/ -> engine/
- internal_api/public/events/ -> engine/events/
- internal_api/public/util/ -> base/
- internal_api/public/ -> core/
- internal_api/ -> core_impl/

This change was generated using the following script:
http://pastebin.com/vVAkRCkh

Because this patch is so large, some review was done in split CLs:

Manual changes: http://crrev.com/2155763002
Sync internal changes: http://crrev.com/2156693002
Sync external changes: http://crrev.com/2148293004

This is a minimal change to support file movement; it doesn't do
anything like merging sync_unit_tests into component_unit_tests,
which will be done in a followup.

The gyp files are not sorted because they're all going to be removed
very soon anyways.

BUG=626114
R=jam@chromium.org, zea@chromium.org
TBR=jam

Committed: https://crrev.com/fef332d69faf129b64689e30304061322ca9bcad
Cr-Commit-Position: refs/heads/master@{#408497}
==========

[commit-bot: I haz the power, 2016-07-28 22:15:08]

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