Standardize persisted log loading

Standardize persisted log loading

BUG=328417
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=283329

[Alexei Svitkine (slow), 2014-07-14 18:46:54]

LGTM, thanks

https://codereview.chromium.org/384063004/diff/20001/components/metrics/metri...
File components/metrics/metrics_service.cc (right):

https://codereview.chromium.org/384063004/diff/20001/components/metrics/metri...
components/metrics/metrics_service.cc:965:
static_cast<MetricsLog*>(log_manager_.current_log());
Nit: While you're here, you may as well remove this unnecessary cast now that
we've unified things.

[Steven Holte, 2014-07-15 01:22:01]

https://codereview.chromium.org/384063004/diff/20001/components/metrics/metri...
File components/metrics/metrics_service.cc (right):

https://codereview.chromium.org/384063004/diff/20001/components/metrics/metri...
components/metrics/metrics_service.cc:965:
static_cast<MetricsLog*>(log_manager_.current_log());
On 2014/07/14 18:46:54, Alexei Svitkine wrote:
> Nit: While you're here, you may as well remove this unnecessary cast now that
> we've unified things.

Done. x3

[Steven Holte, 2014-07-15 01:24:34]

The CQ bit was checked by holte@chromium.org

[commit-bot: I haz the power, 2014-07-15 01:26:08]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/holte@chromium.org/384063004/40001

[commit-bot: I haz the power, 2014-07-15 19:32:24]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_chromium_chromeos_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)
  win_chromium_x64_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_x64_rel/...)

[commit-bot: I haz the power, 2014-07-16 04:20:31]

Change committed as 283329