[Metrics] Merge MetricsService and MetricsServiceBase classes.

[Metrics] Merge MetricsService and MetricsServiceBase classes.

BUG=374200
TEST=compiles
R=asvitkine@chromium.org

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

[Ilya Sherman, 2014-05-21 12:58:05]

Alexei, this leaves the //chrome/common/metrics/ directory empty except for
variations files.  WDYT of renaming the "metrics" directory to "variations" at
this point?

[Alexei Svitkine (slow), 2014-05-21 13:05:22]

LGTM % comment

https://chromiumcodereview.appspot.com/294033007/diff/1/chrome/common/metrics...
File chrome/common/metrics/OWNERS (left):

https://chromiumcodereview.appspot.com/294033007/diff/1/chrome/common/metrics...
chrome/common/metrics/OWNERS:5: mpearson@chromium.org
There are still sub-directories here, so let's keep the OWNERS file.

[Ilya Sherman, 2014-05-21 13:12:03]

The CQ bit was checked by isherman@chromium.org

[Ilya Sherman, 2014-05-21 13:14:08]

https://chromiumcodereview.appspot.com/294033007/diff/1/chrome/common/metrics...
File chrome/common/metrics/OWNERS (left):

https://chromiumcodereview.appspot.com/294033007/diff/1/chrome/common/metrics...
chrome/common/metrics/OWNERS:5: mpearson@chromium.org
On 2014/05/21 13:05:23, Alexei Svitkine wrote:
> There are still sub-directories here, so let's keep the OWNERS file.

Done.

[commit-bot: I haz the power, 2014-05-21 19:59:15]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/isherman@chromium.org/294033007/20001

[commit-bot: I haz the power, 2014-05-22 08:54:20]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  win_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)

[commit-bot: I haz the power, 2014-05-22 08:57:04]

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

[commit-bot: I haz the power, 2014-05-22 08:57:05]

Try jobs failed on following builders:
  win_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu_triggered...)

[Ilya Sherman, 2014-05-22 11:38:27]

The CQ bit was checked by isherman@chromium.org

[commit-bot: I haz the power, 2014-05-22 11:40:13]

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

[Ilya Sherman, 2014-05-22 14:05:56]

The CQ bit was checked by isherman@chromium.org

[commit-bot: I haz the power, 2014-05-22 14:07:56]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/isherman@chromium.org/294033007/30007

[commit-bot: I haz the power, 2014-05-22 18:29:15]

Change committed as 272247