Moves ProfilerMetricsProvider to //components/metrics.

Moves ProfilerMetricsProvider to //components/metrics.

R=asvitkine@chromium.org,isherman@chromium.org,
BUG=404791
Committed: https://crrev.com/840bc4163159ba58413ffa9658e888b1336851fe
Cr-Commit-Position: refs/heads/master@{#295527}

[gunsch, 2014-09-17 00:01:02]

gunsch@chromium.org changed reviewers:
+ mseaborn@chromium.org, thestig@chromium.org

[gunsch, 2014-09-17 00:04:01]

asvitkine/isherman: PTAL!

+mseaborn for components/nacl/common (added DEPS moving code to
components/metrics/profiler/DEPS, no functionality change). Include is
profiler_metrics_provider.cc, referencing NaClProcessType.

+thestig for chrome/browser changes.

[Lei Zhang, 2014-09-17 00:33:37]

non-metrics chrome/ changes lgtm

[Alexei Svitkine (slow), 2014-09-17 15:07:02]

https://codereview.chromium.org/577823002/diff/1/components/metrics.gypi
File components/metrics.gypi (right):

https://codereview.chromium.org/577823002/diff/1/components/metrics.gypi#newc...
components/metrics.gypi:99: 'metrics/profiler/profiler_metrics_provider.cc',
Seems like there's a mistake with this diff? From the diff, it looks like you're
adding all the profiler sources to the metrics_net target and then moving all
the net sources to the new metrics_provider target...

[gunsch, 2014-09-17 17:28:28]

https://codereview.chromium.org/577823002/diff/1/components/metrics.gypi
File components/metrics.gypi (right):

https://codereview.chromium.org/577823002/diff/1/components/metrics.gypi#newc...
components/metrics.gypi:99: 'metrics/profiler/profiler_metrics_provider.cc',
On 2014/09/17 15:07:02, Alexei Svitkine wrote:
> Seems like there's a mistake with this diff? From the diff, it looks like
you're
> adding all the profiler sources to the metrics_net target and then moving all
> the net sources to the new metrics_provider target...

Thanks for catching, fixed.

[Alexei Svitkine (slow), 2014-09-17 17:34:27]

lgtm

[gunsch, 2014-09-17 18:41:48]

gunsch@chromium.org changed reviewers:
+ jochen@chromium.org

[gunsch, 2014-09-17 18:42:50]

+jochen for a content/ OWNER: TrackingSynchronizer and ProfilerMetricsProvider
depend on content/public/browser and content/public/common. Moved to
components/metrics/profiler and declared dependency in new DEPS file.

[Mark Seaborn, 2014-09-17 20:30:23]

On 2014/09/17 00:04:01, gunsch wrote:
> +mseaborn for components/nacl/common (added DEPS moving code to
> components/metrics/profiler/DEPS, no functionality change). Include is
> profiler_metrics_provider.cc, referencing NaClProcessType.

LGTM for that.

[jochen (gone - plz use gerrit), 2014-09-18 18:26:28]

On 2014/09/17 at 18:42:50, gunsch wrote:
> +jochen for a content/ OWNER: TrackingSynchronizer and ProfilerMetricsProvider
depend on content/public/browser and content/public/common. Moved to
components/metrics/profiler and declared dependency in new DEPS file.

lgtm for that

[gunsch, 2014-09-18 18:29:15]

The CQ bit was checked by gunsch@chromium.org

[commit-bot: I haz the power, 2014-09-18 18:30:25]

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

[commit-bot: I haz the power, 2014-09-18 18:35:47]

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

[commit-bot: I haz the power, 2014-09-18 18:35:48]

Try jobs failed on following builders:
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/57792)
  win_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu/builds/62587)
  android_aosp on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_aosp/bu...)
  android_arm64_dbg_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_arm64_d...)
  android_dbg_tests_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg_tes...)
  win8_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_rel...)
  win_chromium_compile_dbg on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)
  win_chromium_x64_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[gunsch, 2014-09-18 18:38:34]

The CQ bit was checked by gunsch@chromium.org

[commit-bot: I haz the power, 2014-09-18 18:40:03]

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

[commit-bot: I haz the power, 2014-09-18 19:38:15]

Committed patchset #3 (id:40001) as e1637c990568588a5e8eff726bce37cc4013e49c

[commit-bot: I haz the power, 2014-09-18 19:38:59]

Patchset 3 (id:??) landed as
https://crrev.com/840bc4163159ba58413ffa9658e888b1336851fe
Cr-Commit-Position: refs/heads/master@{#295527}