Revert of memory-infra: Finish moving memory_infra from TracingController

Revert of memory-infra: Finish moving memory_infra from TracingController (patchset #25 id:470001 of https://codereview.chromium.org/2694083005/ )

Reason for revert:
Unfortunately this broke lot of memory-infra metrics and caused flakiness in various bot.
Have to revert because this is causing problems to the sheriff rotation. More details:
crbug.com/697384 : metrics broken
crbug.com/697062 : memory dumps failing on webview

Original issue's description:
> memory-infra: Finish moving memory_infra from TracingController
> 
> This finishes moving memory instrumentation code from TracingController
> to its own location in //services/resource_coordinator.
>     
> BUG=679830
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Review-Url: https://codereview.chromium.org/2694083005
> Cr-Commit-Position: refs/heads/master@{#453243}
> Committed: https://chromium.googlesource.com/chromium/src/+/e7bee2f02da7fb7afbdb387197ad9461d8d2eba2

TBR=hjd@chromium.org,oysteine@chromium.org,ssid@chromium.org,jam@chromium.org,kenrb@chromium.org,chiniforooshan@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=679830, 697384
Committed: https://chromium.googlesource.com/chromium/src/+/74b034f002cf4f4f5456ed349d57672635e8c46b

[Primiano Tucci (use gerrit), 2017-03-01 11:33:38]

The CQ bit was checked by primiano@chromium.org

[Primiano Tucci (use gerrit), 2017-03-01 11:33:39]

Created Revert of memory-infra: Finish moving memory_infra from
TracingController

[commit-bot: I haz the power, 2017-03-01 11:33:54]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Primiano Tucci (use gerrit), 2017-03-01 11:35:00]

Description was changed from

==========
Revert of memory-infra: Finish moving memory_infra from TracingController
(patchset #25 id:470001 of https://codereview.chromium.org/2694083005/ )

Reason for revert:
Unfortunately this broke lot of memory-infra metrics and caused flakiness in
various bot.
Have to revert because this is causing problems to the sheriff rotation. More
details:
crbug.com/697384 : metrics broken
crbug.com/697062 : memory dumps failing on webview

Original issue's description:
> memory-infra: Finish moving memory_infra from TracingController
> 
> This finishes moving memory instrumentation code from TracingController
> to its own location in //services/resource_coordinator.
>     
> BUG=679830
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Review-Url: https://codereview.chromium.org/2694083005
> Cr-Commit-Position: refs/heads/master@{#453243}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e7bee2f02da7fb7afbdb387197ad...

TBR=hjd@chromium.org,oysteine@chromium.org,ssid@chromium.org,jam@chromium.org...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=679830
==========

to

==========
Revert of memory-infra: Finish moving memory_infra from TracingController
(patchset #25 id:470001 of https://codereview.chromium.org/2694083005/ )

Reason for revert:
Unfortunately this broke lot of memory-infra metrics and caused flakiness in
various bot.
Have to revert because this is causing problems to the sheriff rotation. More
details:
crbug.com/697384 : metrics broken
crbug.com/697062 : memory dumps failing on webview

Original issue's description:
> memory-infra: Finish moving memory_infra from TracingController
> 
> This finishes moving memory instrumentation code from TracingController
> to its own location in //services/resource_coordinator.
>     
> BUG=679830
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Review-Url: https://codereview.chromium.org/2694083005
> Cr-Commit-Position: refs/heads/master@{#453243}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e7bee2f02da7fb7afbdb387197ad...

TBR=hjd@chromium.org,oysteine@chromium.org,ssid@chromium.org,jam@chromium.org...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=679830,697384
==========

[commit-bot: I haz the power, 2017-03-01 12:31:51]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488368018641090, "parent_rev":
"4db6eb187710a109f98606588f4fa53b501e15ba", "commit_rev":
"74b034f002cf4f4f5456ed349d57672635e8c46b"}

[commit-bot: I haz the power, 2017-03-01 12:35:56]

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

[commit-bot: I haz the power, 2017-03-01 12:35:58]

Prior attempt to commit was detected, but we were not able to check whether the
issue was successfully committed. Please check Git history manually and re-check
CQ or close this issue as needed.

[ssid, 2017-03-01 18:24:44]

Description was changed from

==========
Revert of memory-infra: Finish moving memory_infra from TracingController
(patchset #25 id:470001 of https://codereview.chromium.org/2694083005/ )

Reason for revert:
Unfortunately this broke lot of memory-infra metrics and caused flakiness in
various bot.
Have to revert because this is causing problems to the sheriff rotation. More
details:
crbug.com/697384 : metrics broken
crbug.com/697062 : memory dumps failing on webview

Original issue's description:
> memory-infra: Finish moving memory_infra from TracingController
> 
> This finishes moving memory instrumentation code from TracingController
> to its own location in //services/resource_coordinator.
>     
> BUG=679830
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Review-Url: https://codereview.chromium.org/2694083005
> Cr-Commit-Position: refs/heads/master@{#453243}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e7bee2f02da7fb7afbdb387197ad...

TBR=hjd@chromium.org,oysteine@chromium.org,ssid@chromium.org,jam@chromium.org...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=679830,697384
==========

to

==========
Revert of memory-infra: Finish moving memory_infra from TracingController
(patchset #25 id:470001 of https://codereview.chromium.org/2694083005/ )

Reason for revert:
Unfortunately this broke lot of memory-infra metrics and caused flakiness in
various bot.
Have to revert because this is causing problems to the sheriff rotation. More
details:
crbug.com/697384 : metrics broken
crbug.com/697062 : memory dumps failing on webview

Original issue's description:
> memory-infra: Finish moving memory_infra from TracingController
> 
> This finishes moving memory instrumentation code from TracingController
> to its own location in //services/resource_coordinator.
>     
> BUG=679830
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Review-Url: https://codereview.chromium.org/2694083005
> Cr-Commit-Position: refs/heads/master@{#453243}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e7bee2f02da7fb7afbdb387197ad...

TBR=hjd@chromium.org,oysteine@chromium.org,ssid@chromium.org,jam@chromium.org...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=679830,697384

Committed:
https://chromium.googlesource.com/chromium/src/+/74b034f002cf4f4f5456ed349d57...
==========

[ssid, 2017-03-01 18:25:28]

On 2017/03/01 12:35:58, commit-bot: I haz the power wrote:
> Prior attempt to commit was detected, but we were not able to check whether
the
> issue was successfully committed. Please check Git history manually and
re-check
> CQ or close this issue as needed.

This seems to be committed at
https://chromium.googlesource.com/chromium/src/+/74b034f002cf4f4f5456ed349d57...
Issue is not closed because of some cq error. Marking this closed