Move base/prefs to components/prefs

Move base/prefs to components/prefs

This change is the minimal move change. It does not update namespaces or includes. Forwarding headers and targets exist to map to the new location. The namespaces, include guards, and references to these files will be updated in follow-ups.

The GYP build has been updated to point to the new location because the forwarding headers generated a .gyp file cycle. Avoiding a cycle with libaddressinput is why prefs is a separate .gyp from most of the rest of the components. The GN build uses forwarding targets to keep the change smaller.

The json pref store unit tests had to be updated because of the way that they used histograms conflicted with the hsitogram setup of the components unittests. Fortunately, there's a HistogramTester helper for this case, so this patch uses it in for the moved test in place of manually checking histogram counts.

json_pref_store_unittests.cc also depended on the files il base/test/data/prefs. Rather than worry about updating all the isolates and such, this just adds the data inline and avoids having separate test files.

Reland of http://crrev.com/1648403002 with iOS fix
TBR=jam@chromium.org

Committed: https://crrev.com/58cd1f1293144d72e2d4b1ec8f7e09da81e55654
Cr-Commit-Position: refs/heads/master@{#372536}

[brettw, 2016-01-30 04:27:43]

Description was changed from

==========
Move base/prefs to components/prefs

This change is the minimal move change. It does not update namespaces or
includes. Forwarding headers and targets exist to map to the new location. The
namespaces, include guards, and references to these files will be updated in
follow-ups.

The GYP build has been updated to point to the new location because the
forwarding headers generated a .gyp file cycle. Avoiding a cycle with
libaddressinput is why prefs is a separate .gyp from most of the rest of the
components. The GN build uses forwarding targets to keep the change smaller.

The json pref store unit tests had to be updated because of the way that they
used histograms conflicted with the hsitogram setup of the components unittests.
Fortunately, there's a HistogramTester helper for this case, so this patch uses
it in for the moved test in place of manually checking histogram counts.

json_pref_store_unittests.cc also depended on the files il base/test/data/prefs.
Rather than worry about updating all the isolates and such, this just adds the
data inline and avoids having separate test files.

Reland of http://crrev.com/1648403002
==========

to

==========
Move base/prefs to components/prefs

This change is the minimal move change. It does not update namespaces or
includes. Forwarding headers and targets exist to map to the new location. The
namespaces, include guards, and references to these files will be updated in
follow-ups.

The GYP build has been updated to point to the new location because the
forwarding headers generated a .gyp file cycle. Avoiding a cycle with
libaddressinput is why prefs is a separate .gyp from most of the rest of the
components. The GN build uses forwarding targets to keep the change smaller.

The json pref store unit tests had to be updated because of the way that they
used histograms conflicted with the hsitogram setup of the components unittests.
Fortunately, there's a HistogramTester helper for this case, so this patch uses
it in for the moved test in place of manually checking histogram counts.

json_pref_store_unittests.cc also depended on the files il base/test/data/prefs.
Rather than worry about updating all the isolates and such, this just adds the
data inline and avoids having separate test files.

Reland of http://crrev.com/1648403002 with iOS fix
TBR=jam@chromium.org
==========

[brettw, 2016-01-30 04:27:43]

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

[brettw, 2016-01-30 04:29:12]

iOS fix

[brettw, 2016-01-30 04:29:50]

Original is PS1, fix is in PS2.

[brettw, 2016-01-30 04:29:57]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2016-01-30 04:30:08]

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

[commit-bot: I haz the power, 2016-01-30 05:56:13]

Description was changed from

==========
Move base/prefs to components/prefs

This change is the minimal move change. It does not update namespaces or
includes. Forwarding headers and targets exist to map to the new location. The
namespaces, include guards, and references to these files will be updated in
follow-ups.

The GYP build has been updated to point to the new location because the
forwarding headers generated a .gyp file cycle. Avoiding a cycle with
libaddressinput is why prefs is a separate .gyp from most of the rest of the
components. The GN build uses forwarding targets to keep the change smaller.

The json pref store unit tests had to be updated because of the way that they
used histograms conflicted with the hsitogram setup of the components unittests.
Fortunately, there's a HistogramTester helper for this case, so this patch uses
it in for the moved test in place of manually checking histogram counts.

json_pref_store_unittests.cc also depended on the files il base/test/data/prefs.
Rather than worry about updating all the isolates and such, this just adds the
data inline and avoids having separate test files.

Reland of http://crrev.com/1648403002 with iOS fix
TBR=jam@chromium.org
==========

to

==========
Move base/prefs to components/prefs

This change is the minimal move change. It does not update namespaces or
includes. Forwarding headers and targets exist to map to the new location. The
namespaces, include guards, and references to these files will be updated in
follow-ups.

The GYP build has been updated to point to the new location because the
forwarding headers generated a .gyp file cycle. Avoiding a cycle with
libaddressinput is why prefs is a separate .gyp from most of the rest of the
components. The GN build uses forwarding targets to keep the change smaller.

The json pref store unit tests had to be updated because of the way that they
used histograms conflicted with the hsitogram setup of the components unittests.
Fortunately, there's a HistogramTester helper for this case, so this patch uses
it in for the moved test in place of manually checking histogram counts.

json_pref_store_unittests.cc also depended on the files il base/test/data/prefs.
Rather than worry about updating all the isolates and such, this just adds the
data inline and avoids having separate test files.

Reland of http://crrev.com/1648403002 with iOS fix
TBR=jam@chromium.org
==========

[commit-bot: I haz the power, 2016-01-30 05:56:14]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-30 05:57:14]

Description was changed from

==========
Move base/prefs to components/prefs

This change is the minimal move change. It does not update namespaces or
includes. Forwarding headers and targets exist to map to the new location. The
namespaces, include guards, and references to these files will be updated in
follow-ups.

The GYP build has been updated to point to the new location because the
forwarding headers generated a .gyp file cycle. Avoiding a cycle with
libaddressinput is why prefs is a separate .gyp from most of the rest of the
components. The GN build uses forwarding targets to keep the change smaller.

The json pref store unit tests had to be updated because of the way that they
used histograms conflicted with the hsitogram setup of the components unittests.
Fortunately, there's a HistogramTester helper for this case, so this patch uses
it in for the moved test in place of manually checking histogram counts.

json_pref_store_unittests.cc also depended on the files il base/test/data/prefs.
Rather than worry about updating all the isolates and such, this just adds the
data inline and avoids having separate test files.

Reland of http://crrev.com/1648403002 with iOS fix
TBR=jam@chromium.org
==========

to

==========
Move base/prefs to components/prefs

This change is the minimal move change. It does not update namespaces or
includes. Forwarding headers and targets exist to map to the new location. The
namespaces, include guards, and references to these files will be updated in
follow-ups.

The GYP build has been updated to point to the new location because the
forwarding headers generated a .gyp file cycle. Avoiding a cycle with
libaddressinput is why prefs is a separate .gyp from most of the rest of the
components. The GN build uses forwarding targets to keep the change smaller.

The json pref store unit tests had to be updated because of the way that they
used histograms conflicted with the hsitogram setup of the components unittests.
Fortunately, there's a HistogramTester helper for this case, so this patch uses
it in for the moved test in place of manually checking histogram counts.

json_pref_store_unittests.cc also depended on the files il base/test/data/prefs.
Rather than worry about updating all the isolates and such, this just adds the
data inline and avoids having separate test files.

Reland of http://crrev.com/1648403002 with iOS fix
TBR=jam@chromium.org

Committed: https://crrev.com/58cd1f1293144d72e2d4b1ec8f7e09da81e55654
Cr-Commit-Position: refs/heads/master@{#372536}
==========

[commit-bot: I haz the power, 2016-01-30 05:57:15]

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