Constructor without callback added to ProfilerMetricsProvider.

Constructor without callback added to ProfilerMetricsProvider.

As ProfilerMetricsProvider is used in same cases where network connection type is irrelevant, a constructor without the connection type callback is added to support those cases.
Constructor with a callback was introduced in CL/937263002.

BUG=455847
Committed: https://crrev.com/5bece2596244816b3ce8e4e0f78598c8fa28d350
Cr-Commit-Position: refs/heads/master@{#318086}

[gayane -on leave until 09-2017, 2015-02-25 16:49:22]

gayane@chromium.org changed reviewers:
+ asvitkine@chromium.org

[gunsch, 2015-02-25 17:37:14]

On 2015/02/25 16:49:22, gayane wrote:
> mailto:gayane@chromium.org changed reviewers:
> + mailto:asvitkine@chromium.org

Great, thanks for the quick fix. lgtm for chromecast

[Alexei Svitkine (slow), 2015-02-25 17:39:46]

lgtm

https://codereview.chromium.org/953403002/diff/1/components/metrics/profiler/...
File components/metrics/profiler/profiler_metrics_provider.h (right):

https://codereview.chromium.org/953403002/diff/1/components/metrics/profiler/...
components/metrics/profiler/profiler_metrics_provider.h:23:
ProfilerMetricsProvider();
Nit: Add a comment that this creates the provider with a null callback. Also,
move it below the next one.

[gayane -on leave until 09-2017, 2015-02-25 17:52:27]

The CQ bit was checked by gayane@chromium.org

[gayane -on leave until 09-2017, 2015-02-25 17:52:33]

The CQ bit was unchecked by gayane@chromium.org

[gayane -on leave until 09-2017, 2015-02-25 18:21:06]

Constructor of provider is moved and comment added.

https://codereview.chromium.org/953403002/diff/1/components/metrics/profiler/...
File components/metrics/profiler/profiler_metrics_provider.h (right):

https://codereview.chromium.org/953403002/diff/1/components/metrics/profiler/...
components/metrics/profiler/profiler_metrics_provider.h:23:
ProfilerMetricsProvider();
On 2015/02/25 17:39:45, Alexei Svitkine wrote:
> Nit: Add a comment that this creates the provider with a null callback. Also,
> move it below the next one.

Done.

[gayane -on leave until 09-2017, 2015-02-25 18:21:28]

The CQ bit was checked by gayane@chromium.org

[gayane -on leave until 09-2017, 2015-02-25 18:21:28]

The patchset sent to the CQ was uploaded after l-g-t-m from
asvitkine@chromium.org, gunsch@chromium.org
Link to the patchset: https://codereview.chromium.org/953403002/#ps20001 (title:
"Constructor of provider is moved and comment added")

[commit-bot: I haz the power, 2015-02-25 18:22:57]

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

[commit-bot: I haz the power, 2015-02-25 19:17:48]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-25 19:21:04]

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