Add a base::Feature to change progress bar completion policy.

Add a base::Feature to change progress bar completion policy.

A command line flag will take priority. If none is present, the Feature will be
used to determine the policy.

BUG=651646
Committed: https://crrev.com/76a28a8d25f87f73dc2aa94f721e80cbf0666cc3
Cr-Commit-Position: refs/heads/master@{#426961}

[Nate Chapin, 2016-10-21 21:49:07]

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

[Nate Chapin, 2016-10-21 21:55:58]

japhet@chromium.org changed reviewers:
+ creis@chromium.org

[Charlie Reis, 2016-10-21 22:07:21]

LGTM

https://codereview.chromium.org/2445453002/diff/1/content/common/content_swit...
File content/common/content_switches_internal.cc (right):

https://codereview.chromium.org/2445453002/diff/1/content/common/content_swit...
content/common/content_switches_internal.cc:40: const base::Feature
kProgressBarCompletionResourcesBeforeDOMContentLoaded{
nit: Is there room for a space before the brace, as above?

[Nate Chapin, 2016-10-21 22:09:22]

https://codereview.chromium.org/2445453002/diff/1/content/common/content_swit...
File content/common/content_switches_internal.cc (right):

https://codereview.chromium.org/2445453002/diff/1/content/common/content_swit...
content/common/content_switches_internal.cc:40: const base::Feature
kProgressBarCompletionResourcesBeforeDOMContentLoaded{
On 2016/10/21 22:07:20, Charlie Reis wrote:
> nit: Is there room for a space before the brace, as above?

Yep, will fix before landing.

[Nate Chapin, 2016-10-21 22:11:01]

The CQ bit was checked by japhet@chromium.org

[commit-bot: I haz the power, 2016-10-21 22:11:49]

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-10-22 03:08:28]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-22 03:11:19]

Description was changed from

==========
Add a base::Feature to change progress bar completion policy.

A command line flag will take priority. If none is present, the Feature will be
used to determine the policy.

BUG=651646
==========

to

==========
Add a base::Feature to change progress bar completion policy.

A command line flag will take priority. If none is present, the Feature will be
used to determine the policy.

BUG=651646

Committed: https://crrev.com/76a28a8d25f87f73dc2aa94f721e80cbf0666cc3
Cr-Commit-Position: refs/heads/master@{#426961}
==========

[commit-bot: I haz the power, 2016-10-22 03:11:20]

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