Renamed /services/memory_infrastructure to /services/resource_coordinator

Renamed /services/memory_infrastructure to /services/resource_coordinator

The memory_infra bindings are now in a subfolder of this service, in preparation
for more functionality being added.

Also changed the OWNERS file from pointing to /base/trace_event/OWNERS to a reasonable set of people involved who are also owners elsewhere.

R=chiniforooshan@chromium.org,jam@chromium.org
BUG=691886
Review-Url: https://codereview.chromium.org/2693063002
Cr-Commit-Position: refs/heads/master@{#450482}
Committed: https://chromium.googlesource.com/chromium/src/+/f4729b0eccefc8bd9f46d4e36144b9a53cd02e33

[oystein (OOO til 10th of July), 2017-02-14 03:59:09]

oysteine@chromium.org changed reviewers:
+ rockot@chromium.org
- jam@chromium.org

[oystein (OOO til 10th of July), 2017-02-14 04:00:07]

ptal!

chiniforooshan: I rebased your WIP CL for the continued work in
https://codereview.chromium.org/2690143004 so you should hopefully be able to
just patch in that and continue without a hitch.

[oystein (OOO til 10th of July), 2017-02-14 04:00:17]

The CQ bit was checked by oysteine@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-14 04:01:12]

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

[oystein (OOO til 10th of July), 2017-02-14 04:04:40]

Description was changed from

==========
Renamed /services/memory_infrastructure to /services/resource_coordinator

The memory_infra bindings are now in a subfolder of this service, in preparation
for more functionality being added.

R=chiniforooshan@chromium.org,jam@chromium.org
BUG=691886
==========

to

==========
Renamed /services/memory_infrastructure to /services/resource_coordinator

The memory_infra bindings are now in a subfolder of this service, in preparation
for more functionality being added.

Also changed the OWNERS file from pointing to /base/trace_event/OWNERS to a
reasonable set of people involved who are also owners elsewhere.

R=chiniforooshan@chromium.org,jam@chromium.org
BUG=691886
==========

[commit-bot: I haz the power, 2017-02-14 04:04:57]

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

[commit-bot: I haz the power, 2017-02-14 04:04:58]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[oystein (OOO til 10th of July), 2017-02-14 04:07:36]

The CQ bit was checked by oysteine@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-14 04:08:29]

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

[commit-bot: I haz the power, 2017-02-14 05:17:24]

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

[commit-bot: I haz the power, 2017-02-14 05:17:24]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Primiano Tucci (use gerrit), 2017-02-14 12:33:13]

I am a bit backlogged for doing an actual review, but the plan here sgtm.
RS-LGTM in case you need it for base/trace_event/memory_dump_request_args.h and
the move of memory files.

[haraken, 2017-02-14 12:52:17]

LGTM 2

[chiniforooshan, 2017-02-14 16:08:27]

lgtm

[Ken Rockot(use gerrit already), 2017-02-14 16:46:15]

rockot@chromium.org changed reviewers:
+ haraken@chromium.org, primiano@chromium.org

[Ken Rockot(use gerrit already), 2017-02-14 16:46:16]

lgtm

[oystein (OOO til 10th of July), 2017-02-14 17:38:16]

oysteine@chromium.org changed reviewers:
+ dcheng@chromium.org

[oystein (OOO til 10th of July), 2017-02-14 17:38:39]

On 2017/02/14 at 16:46:16, rockot wrote:
> lgtm

Thanks!

+dcheng for the .mojom move

[Will Harris, 2017-02-14 18:10:09]

rs security lgtm if you need one in a hurry.

[oystein (OOO til 10th of July), 2017-02-14 18:57:39]

The CQ bit was checked by oysteine@chromium.org to run a CQ dry run

[oystein (OOO til 10th of July), 2017-02-14 18:57:51]

On 2017/02/14 at 18:10:09, wfh wrote:
> rs security lgtm if you need one in a hurry.

Awesome, thanks!

[commit-bot: I haz the power, 2017-02-14 18:58:02]

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

[oystein (OOO til 10th of July), 2017-02-14 18:58:08]

The CQ bit was unchecked by oysteine@chromium.org

[oystein (OOO til 10th of July), 2017-02-14 18:58:11]

The CQ bit was checked by oysteine@chromium.org

[commit-bot: I haz the power, 2017-02-14 18:58:31]

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

[dcheng, 2017-02-14 19:05:54]

LGTM with a nit.

https://codereview.chromium.org/2693063002/diff/20001/services/resource_coord...
File
services/resource_coordinator/public/cpp/memory/memory_instrumentation_traits.cc
(right):

https://codereview.chromium.org/2693063002/diff/20001/services/resource_coord...
services/resource_coordinator/public/cpp/memory/memory_instrumentation_traits.cc:1:
// Copyright 2017 The Chromium Authors. All rights reserved.
Please rename so the name ends with _struct_traits to make sure it goes through
review for future changes (I know it doesn't contain any StructTraits atm, but I
think that's OK: _traits is a bit too generic to have a PRESUBMIT rule on).

[dcheng, 2017-02-14 19:06:29]

https://codereview.chromium.org/2693063002/diff/20001/services/resource_coord...
File
services/resource_coordinator/public/cpp/memory/memory_instrumentation_traits.cc
(right):

https://codereview.chromium.org/2693063002/diff/20001/services/resource_coord...
services/resource_coordinator/public/cpp/memory/memory_instrumentation_traits.cc:1:
// Copyright 2017 The Chromium Authors. All rights reserved.
On 2017/02/14 19:05:53, dcheng wrote:
> Please rename so the name ends with _struct_traits to make sure it goes
through
> review for future changes (I know it doesn't contain any StructTraits atm, but
I
> think that's OK: _traits is a bit too generic to have a PRESUBMIT rule on).

(A followup CL for this is OK)

[commit-bot: I haz the power, 2017-02-14 19:09:35]

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

[commit-bot: I haz the power, 2017-02-14 19:09:36]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[oystein (OOO til 10th of July), 2017-02-14 19:50:11]

The CQ bit was checked by oysteine@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-14 19:51:07]

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

[oystein (OOO til 10th of July), 2017-02-14 21:22:56]

The CQ bit was unchecked by oysteine@chromium.org

[oystein (OOO til 10th of July), 2017-02-14 21:22:59]

The CQ bit was checked by oysteine@chromium.org

[oystein (OOO til 10th of July), 2017-02-14 21:23:00]

The patchset sent to the CQ was uploaded after l-g-t-m from
primiano@chromium.org, rockot@chromium.org, dcheng@chromium.org,
chiniforooshan@chromium.org, haraken@chromium.org, wfh@chromium.org
Link to the patchset: https://codereview.chromium.org/2693063002/#ps40001
(title: "Review fix")

[commit-bot: I haz the power, 2017-02-14 21:23:28]

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

[commit-bot: I haz the power, 2017-02-14 21:35:50]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1487107379445230,
"parent_rev": "7832d67481ad51c31126552301df007efb57fa8a", "commit_rev":
"f4729b0eccefc8bd9f46d4e36144b9a53cd02e33"}

[commit-bot: I haz the power, 2017-02-14 21:36:26]

Description was changed from

==========
Renamed /services/memory_infrastructure to /services/resource_coordinator

The memory_infra bindings are now in a subfolder of this service, in preparation
for more functionality being added.

Also changed the OWNERS file from pointing to /base/trace_event/OWNERS to a
reasonable set of people involved who are also owners elsewhere.

R=chiniforooshan@chromium.org,jam@chromium.org
BUG=691886
==========

to

==========
Renamed /services/memory_infrastructure to /services/resource_coordinator

The memory_infra bindings are now in a subfolder of this service, in preparation
for more functionality being added.

Also changed the OWNERS file from pointing to /base/trace_event/OWNERS to a
reasonable set of people involved who are also owners elsewhere.

R=chiniforooshan@chromium.org,jam@chromium.org
BUG=691886

Review-Url: https://codereview.chromium.org/2693063002
Cr-Commit-Position: refs/heads/master@{#450482}
Committed:
https://chromium.googlesource.com/chromium/src/+/f4729b0eccefc8bd9f46d4e36144...
==========

[commit-bot: I haz the power, 2017-02-14 21:36:27]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/f4729b0eccefc8bd9f46d4e36144...