Cleanup: removing unused descendants information from tracked objects

Cleanup: removing unused descendants information from tracked objects.

BUG=456354
Committed: https://crrev.com/cf8983ef6ccb5101181b2f854023b2124dd31882
Cr-Commit-Position: refs/heads/master@{#327956}

[vadimt, 2015-04-29 20:39:31]

vadimt@chromium.org changed reviewers:
+ asvitkine@chromium.org, isherman@chromium.org

[Ilya Sherman, 2015-04-30 05:30:28]

LGTM.  Thanks, Vadim.

https://codereview.chromium.org/1104053004/diff/1/base/tracked_objects.h
File base/tracked_objects.h (right):

https://codereview.chromium.org/1104053004/diff/1/base/tracked_objects.h#newc...
base/tracked_objects.h:425: UNINITIALIZED,         // PRistine, link-time state
before running.
nit: "Pristine"

https://codereview.chromium.org/1104053004/diff/1/base/tracked_objects.h#newc...
base/tracked_objects.h:506: // If tracking is not compiled in, this function
will return false.
Is the case of tracking not being compiled in still possible?

[Alexei Svitkine (slow), 2015-04-30 14:31:03]

lgtm

[vadimt, 2015-04-30 20:37:43]

https://codereview.chromium.org/1104053004/diff/1/base/tracked_objects.h
File base/tracked_objects.h (right):

https://codereview.chromium.org/1104053004/diff/1/base/tracked_objects.h#newc...
base/tracked_objects.h:425: UNINITIALIZED,         // PRistine, link-time state
before running.
On 2015/04/30 05:30:28, Ilya Sherman (OOO May 2-10) wrote:
> nit: "Pristine"

Done.

https://codereview.chromium.org/1104053004/diff/1/base/tracked_objects.h#newc...
base/tracked_objects.h:506: // If tracking is not compiled in, this function
will return false.
Impossible. Still can return false if fails to initialize TLS slot.
Perhaps, though, the TLS slot cannot fail to initialize, but this would be
beyond this CLs scope.

[vadimt, 2015-04-30 20:39:14]

vadimt@chromium.org changed reviewers:
+ cpu@chromium.org

[vadimt, 2015-04-30 20:39:15]

cpu@, please provide OWNERs approval. Note that we change the IPC protocol, but
only by removing a field.

[vadimt, 2015-04-30 20:40:36]

Ah, you aren't an owner of child_process_messages.h (per the cr tool)...
I'll add a reviewer.

[vadimt, 2015-04-30 20:42:06]

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

[vadimt, 2015-04-30 20:42:07]

dcheng@chromium.org: Please review changes in 

child_process_messages.h

[dcheng, 2015-04-30 20:43:04]

rs lgtm for removal-only IPC changes

[cpu_(ooo_6.6-7.5), 2015-05-01 02:13:14]

lgtm

[vadimt, 2015-05-01 14:10:41]

The CQ bit was checked by vadimt@chromium.org

[vadimt, 2015-05-01 14:10:42]

The patchset sent to the CQ was uploaded after l-g-t-m from
isherman@chromium.org, asvitkine@chromium.org
Link to the patchset: https://codereview.chromium.org/1104053004/#ps20001
(title: "isherman@ comments")

[commit-bot: I haz the power, 2015-05-01 14:10:56]

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

[commit-bot: I haz the power, 2015-05-01 16:47:05]

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

[commit-bot: I haz the power, 2015-05-01 16:47:07]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[vadimt, 2015-05-01 17:57:21]

The CQ bit was checked by vadimt@chromium.org

[commit-bot: I haz the power, 2015-05-01 17:58:12]

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

[commit-bot: I haz the power, 2015-05-01 19:13:14]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-01 19:15:10]

Patchset 2 (id:??) landed as
https://crrev.com/cf8983ef6ccb5101181b2f854023b2124dd31882
Cr-Commit-Position: refs/heads/master@{#327956}