Remove redundant condition in policy setup

Remove redundant condition in policy setup

!initial_policy_fetch_timeout.is_max() ==> allow_failed_policy_fetches_
thus allow_failed_policy_fetches_ can be dropped.

At that occasion, also improve the constructor comment and move one
member initialization into the class declaration.

BUG=none

Review-Url: https://codereview.chromium.org/2940883002
Cr-Commit-Position: refs/heads/master@{#482232}
Committed: https://chromium.googlesource.com/chromium/src/+/1ade9dfba70113fb2e9898bb187248ed18d07a05

[Thiemo Nagel, 2017-06-14 15:02:17]

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

[commit-bot: I haz the power, 2017-06-14 15:02:46]

Dry run: CQ is trying da patch.

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

[Thiemo Nagel, 2017-06-14 15:14:23]

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

[commit-bot: I haz the power, 2017-06-14 15:14:48]

Dry run: CQ is trying da patch.

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

[Thiemo Nagel, 2017-06-14 16:38:10]

Description was changed from

==========
Remove redundant condition in policy setup

!initial_policy_fetch_timeout.is_max() ==> allow_failed_policy_fetches_
thus allow_failed_policy_fetches_ can be dropped.

BUG=none
==========

to

==========
Remove redundant condition in policy setup

!initial_policy_fetch_timeout.is_max() ==> allow_failed_policy_fetches_
thus allow_failed_policy_fetches_ can be dropped.

At that occasion, also improve the constructor comment and move one
member initialization into the class declaration.

BUG=none
==========

[Thiemo Nagel, 2017-06-14 16:38:23]

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

[commit-bot: I haz the power, 2017-06-14 16:38:42]

Dry run: CQ is trying da patch.

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

[Thiemo Nagel, 2017-06-14 16:38:47]

tnagel@chromium.org changed reviewers:
+ atwilson@chromium.org

[Thiemo Nagel, 2017-06-14 16:38:48]

Hey Drew,

could you please take a look?

Thank you!
Thiemo

[commit-bot: I haz the power, 2017-06-14 17:11:50]

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

[commit-bot: I haz the power, 2017-06-14 17:11:52]

Dry run: This issue passed the CQ dry run.

[Thiemo Nagel, 2017-06-26 09:34:09]

Drew, friendly ping!

[Andrew T Wilson (Slow), 2017-06-26 10:13:03]

lgtm

[Thiemo Nagel, 2017-06-26 10:14:20]

On 2017/06/26 10:13:03, Andrew T Wilson (Slow) wrote:
> lgtm

Thank you!

[Thiemo Nagel, 2017-06-26 10:14:25]

The CQ bit was checked by tnagel@chromium.org

[commit-bot: I haz the power, 2017-06-26 10:14:39]

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-26 10:47:40]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1498472065760420,
"parent_rev": "5fb8ce021baeea825436206fafe8eb93d84ecc7c", "commit_rev":
"1ade9dfba70113fb2e9898bb187248ed18d07a05"}

[commit-bot: I haz the power, 2017-06-26 10:47:51]

Description was changed from

==========
Remove redundant condition in policy setup

!initial_policy_fetch_timeout.is_max() ==> allow_failed_policy_fetches_
thus allow_failed_policy_fetches_ can be dropped.

At that occasion, also improve the constructor comment and move one
member initialization into the class declaration.

BUG=none
==========

to

==========
Remove redundant condition in policy setup

!initial_policy_fetch_timeout.is_max() ==> allow_failed_policy_fetches_
thus allow_failed_policy_fetches_ can be dropped.

At that occasion, also improve the constructor comment and move one
member initialization into the class declaration.

BUG=none

Review-Url: https://codereview.chromium.org/2940883002
Cr-Commit-Position: refs/heads/master@{#482232}
Committed:
https://chromium.googlesource.com/chromium/src/+/1ade9dfba70113fb2e9898bb1872...
==========

[commit-bot: I haz the power, 2017-06-26 10:47:55]

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