Change DataPersistent constructor to allow one line initialisation.

Change DataPersistent constructor to allow one line initialisation.

Currently, DataPersistent objects must be initialised twice: once via
the constructor that sets it to nullptr and once more via the Init()
method that sets it to a nonnull value.

This patch changes the default constructor to do Init() instead and
adds an extra constructor that sets it to nullptr. This way, we can
initialise DataPersistent objects in one line (useful for constructor
initializer lists).

BUG=628043
Review-Url: https://codereview.chromium.org/2935893002
Cr-Commit-Position: refs/heads/master@{#478883}
Committed: https://chromium.googlesource.com/chromium/src/+/3adbcbaa43c1d5069da92d154a7382943c5a2881

[shend, 2017-06-13 00:54:41]

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

[commit-bot: I haz the power, 2017-06-13 00:55:02]

Dry run: CQ is trying da patch.

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

[shend, 2017-06-13 00:55:08]

shend@chromium.org changed reviewers:
+ nainar@chromium.org

[shend, 2017-06-13 00:55:08]

Hi Naina, PTAL

[nainar, 2017-06-13 00:56:39]

lgtm

[commit-bot: I haz the power, 2017-06-13 03:14:25]

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

[commit-bot: I haz the power, 2017-06-13 03:14:26]

Dry run: This issue passed the CQ dry run.

[shend, 2017-06-13 03:15:49]

The CQ bit was checked by shend@chromium.org

[commit-bot: I haz the power, 2017-06-13 03:16:12]

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-06-13 03:19:36]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1497323749719060, "parent_rev":
"a113781033a747eaf60e8ad7799e9b95a58ceed4", "commit_rev":
"3adbcbaa43c1d5069da92d154a7382943c5a2881"}

[commit-bot: I haz the power, 2017-06-13 03:19:47]

Description was changed from

==========
Change DataPersistent constructor to allow one line initialisation.

Currently, DataPersistent objects must be initialised twice: once via
the constructor that sets it to nullptr and once more via the Init()
method that sets it to a nonnull value.

This patch changes the default constructor to do Init() instead and
adds an extra constructor that sets it to nullptr. This way, we can
initialise DataPersistent objects in one line (useful for constructor
initializer lists).

BUG=628043
==========

to

==========
Change DataPersistent constructor to allow one line initialisation.

Currently, DataPersistent objects must be initialised twice: once via
the constructor that sets it to nullptr and once more via the Init()
method that sets it to a nonnull value.

This patch changes the default constructor to do Init() instead and
adds an extra constructor that sets it to nullptr. This way, we can
initialise DataPersistent objects in one line (useful for constructor
initializer lists).

BUG=628043

Review-Url: https://codereview.chromium.org/2935893002
Cr-Commit-Position: refs/heads/master@{#478883}
Committed:
https://chromium.googlesource.com/chromium/src/+/3adbcbaa43c1d5069da92d154a73...
==========

[commit-bot: I haz the power, 2017-06-13 03:19:49]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/3adbcbaa43c1d5069da92d154a73...