[reland] Add a default task runner argument to the constructor of JsonPrefStore.

[reland] Add a default task runner argument to the constructor of JsonPrefStore.

This was reverted because of a ChromeOS compile failure. The
compile failure was fixed by including "components/prefs/pref_filter.h"

Callers that don't need to run tasks on the same sequence as the
JsonPrefStore will be able to use the default argument.

BUG=667892
Review-Url: https://codereview.chromium.org/2876643003
Cr-Original-Commit-Position: refs/heads/master@{#471428}
Committed: https://chromium.googlesource.com/chromium/src/+/22f197b4c0399140f9cf118136a8c96b8ad8af9a
Review-Url: https://codereview.chromium.org/2876643003
Cr-Commit-Position: refs/heads/master@{#471781}
Committed: https://chromium.googlesource.com/chromium/src/+/5f4a7a0bb70d7d0ad0027bf44d0640a5675bba94

[fdoray, 2017-05-11 16:25:23]

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

[commit-bot: I haz the power, 2017-05-11 16:25:51]

Dry run: CQ is trying da patch.

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

[fdoray, 2017-05-11 16:46:58]

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

[commit-bot: I haz the power, 2017-05-11 16:47:26]

Dry run: CQ is trying da patch.

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

[fdoray, 2017-05-11 17:47:43]

fdoray@chromium.org changed reviewers:
+ gab@chromium.org

[fdoray, 2017-05-11 17:47:43]

PTAL

[commit-bot: I haz the power, 2017-05-11 17:52:46]

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

[commit-bot: I haz the power, 2017-05-11 17:52:46]

Dry run: This issue passed the CQ dry run.

[gab, 2017-05-12 18:18:42]

lgtm

[gab, 2017-05-12 18:18:47]

The CQ bit was checked by gab@chromium.org

[commit-bot: I haz the power, 2017-05-12 18:19:47]

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, 2017-05-12 20:21:23]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1494613127221540,
"parent_rev": "57b0df585ca73e3c920515ad895a7e68ee0ef8db", "commit_rev":
"22f197b4c0399140f9cf118136a8c96b8ad8af9a"}

[commit-bot: I haz the power, 2017-05-12 20:21:33]

Description was changed from

==========
Add a default task runner argument to the constructor of JsonPrefStore.

Callers that don't need to run tasks on the same sequence as the
JsonPrefStore will be able to use the default argument.

BUG=667892
==========

to

==========
Add a default task runner argument to the constructor of JsonPrefStore.

Callers that don't need to run tasks on the same sequence as the
JsonPrefStore will be able to use the default argument.

BUG=667892

Review-Url: https://codereview.chromium.org/2876643003
Cr-Commit-Position: refs/heads/master@{#471428}
Committed:
https://chromium.googlesource.com/chromium/src/+/22f197b4c0399140f9cf118136a8...
==========

[commit-bot: I haz the power, 2017-05-12 20:21:34]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/22f197b4c0399140f9cf118136a8...

[Justin Donnelly, 2017-05-12 21:40:04]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/2884473002/ by jdonnelly@chromium.org.

The reason for reverting is: Broke compile step on ChromiumOS x86-generic
Compile:

https://uberchromegw.corp.google.com/i/chromium.chromiumos/builders/ChromiumO....

[fdoray, 2017-05-15 12:08:23]

Description was changed from

==========
Add a default task runner argument to the constructor of JsonPrefStore.

Callers that don't need to run tasks on the same sequence as the
JsonPrefStore will be able to use the default argument.

BUG=667892

Review-Url: https://codereview.chromium.org/2876643003
Cr-Commit-Position: refs/heads/master@{#471428}
Committed:
https://chromium.googlesource.com/chromium/src/+/22f197b4c0399140f9cf118136a8...
==========

to

==========
Add a default task runner argument to the constructor of JsonPrefStore.

Callers that don't need to run tasks on the same sequence as the
JsonPrefStore will be able to use the default argument.

BUG=667892

Review-Url: https://codereview.chromium.org/2876643003
Cr-Commit-Position: refs/heads/master@{#471428}
Committed:
https://chromium.googlesource.com/chromium/src/+/22f197b4c0399140f9cf118136a8...
==========

[fdoray, 2017-05-15 12:11:09]

Description was changed from

==========
Add a default task runner argument to the constructor of JsonPrefStore.

Callers that don't need to run tasks on the same sequence as the
JsonPrefStore will be able to use the default argument.

BUG=667892

Review-Url: https://codereview.chromium.org/2876643003
Cr-Commit-Position: refs/heads/master@{#471428}
Committed:
https://chromium.googlesource.com/chromium/src/+/22f197b4c0399140f9cf118136a8...
==========

to

==========
Add a default task runner argument to the constructor of JsonPrefStore.

This was reverted because of a ChromeOS compile failure. The
compile failure was fixed by including "components/prefs/pref_filter.h"

Callers that don't need to run tasks on the same sequence as the
JsonPrefStore will be able to use the default argument.

BUG=667892

Review-Url: https://codereview.chromium.org/2876643003
Cr-Commit-Position: refs/heads/master@{#471428}
Committed:
https://chromium.googlesource.com/chromium/src/+/22f197b4c0399140f9cf118136a8...
==========

[fdoray, 2017-05-15 12:11:16]

Description was changed from

==========
Add a default task runner argument to the constructor of JsonPrefStore.

This was reverted because of a ChromeOS compile failure. The
compile failure was fixed by including "components/prefs/pref_filter.h"

Callers that don't need to run tasks on the same sequence as the
JsonPrefStore will be able to use the default argument.

BUG=667892

Review-Url: https://codereview.chromium.org/2876643003
Cr-Commit-Position: refs/heads/master@{#471428}
Committed:
https://chromium.googlesource.com/chromium/src/+/22f197b4c0399140f9cf118136a8...
==========

to

==========
[reland] Add a default task runner argument to the constructor of JsonPrefStore.

This was reverted because of a ChromeOS compile failure. The
compile failure was fixed by including "components/prefs/pref_filter.h"

Callers that don't need to run tasks on the same sequence as the
JsonPrefStore will be able to use the default argument.

BUG=667892

Review-Url: https://codereview.chromium.org/2876643003
Cr-Commit-Position: refs/heads/master@{#471428}
Committed:
https://chromium.googlesource.com/chromium/src/+/22f197b4c0399140f9cf118136a8...
==========

[fdoray, 2017-05-15 12:11:19]

The CQ bit was checked by fdoray@chromium.org

[fdoray, 2017-05-15 12:11:20]

The patchset sent to the CQ was uploaded after l-g-t-m from gab@chromium.org
Link to the patchset: https://codereview.chromium.org/2876643003/#ps60001
(title: "include preffilter")

[commit-bot: I haz the power, 2017-05-15 12:11:31]

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, 2017-05-15 12:46:57]

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

[commit-bot: I haz the power, 2017-05-15 12:46:58]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[fdoray, 2017-05-15 12:55:29]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2017-05-15 12:55:38]

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, 2017-05-15 14:43:50]

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

[commit-bot: I haz the power, 2017-05-15 14:43:51]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[fdoray, 2017-05-15 14:56:36]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2017-05-15 14:57:05]

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, 2017-05-15 16:16:01]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1494860196041110,
"parent_rev": "db69d6219c2103e298ab8262bf4a9cd05e870305", "commit_rev":
"5f4a7a0bb70d7d0ad0027bf44d0640a5675bba94"}

[commit-bot: I haz the power, 2017-05-15 16:16:14]

Description was changed from

==========
[reland] Add a default task runner argument to the constructor of JsonPrefStore.

This was reverted because of a ChromeOS compile failure. The
compile failure was fixed by including "components/prefs/pref_filter.h"

Callers that don't need to run tasks on the same sequence as the
JsonPrefStore will be able to use the default argument.

BUG=667892

Review-Url: https://codereview.chromium.org/2876643003
Cr-Commit-Position: refs/heads/master@{#471428}
Committed:
https://chromium.googlesource.com/chromium/src/+/22f197b4c0399140f9cf118136a8...
==========

to

==========
[reland] Add a default task runner argument to the constructor of JsonPrefStore.

This was reverted because of a ChromeOS compile failure. The
compile failure was fixed by including "components/prefs/pref_filter.h"

Callers that don't need to run tasks on the same sequence as the
JsonPrefStore will be able to use the default argument.

BUG=667892

Review-Url: https://codereview.chromium.org/2876643003
Cr-Original-Commit-Position: refs/heads/master@{#471428}
Committed:
https://chromium.googlesource.com/chromium/src/+/22f197b4c0399140f9cf118136a8...
Review-Url: https://codereview.chromium.org/2876643003
Cr-Commit-Position: refs/heads/master@{#471781}
Committed:
https://chromium.googlesource.com/chromium/src/+/5f4a7a0bb70d7d0ad0027bf44d06...
==========

[commit-bot: I haz the power, 2017-05-15 16:16:15]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/5f4a7a0bb70d7d0ad0027bf44d06...

[gab, 2017-05-15 16:33:20]

Post-commit nit after PS3

https://codereview.chromium.org/2876643003/diff/60001/components/prefs/json_p...
File components/prefs/json_pref_store.cc (right):

https://codereview.chromium.org/2876643003/diff/60001/components/prefs/json_p...
components/prefs/json_pref_store.cc:29: #include
"components/prefs/pref_filter.h"
rm