Rename Heap to ThreadHeap

Rename Heap to ThreadHeap

BUG=591606
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq

Committed: https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e
Cr-Commit-Position: refs/heads/master@{#387233}

Committed: https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62
Cr-Commit-Position: refs/heads/master@{#387307}

[keishi, 2016-03-30 03:57:02]

Description was changed from

==========
Rename Heap to ThreadHeap

BUG=591606
==========

to

==========
Rename Heap to ThreadHeap

BUG=591606
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:android_s5_perf_cq;tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq
==========

[keishi, 2016-03-30 03:59:40]

keishi@chromium.org changed reviewers:
+ haraken@chromium.org, oilpan-reviews@chromium.org

[keishi, 2016-03-30 03:59:41]

crbug.com/1842803003 Introduce ProcessHeap
crbug.com/1840103004 Introduce ThreadHeapStats
crbug.com/1845543002 Rename Heap to ThreadHeap <- this

[haraken, 2016-03-30 04:30:24]

LGTM

[keishi, 2016-04-01 02:42:46]

The CQ bit was checked by keishi@chromium.org

[keishi, 2016-04-01 02:42:46]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/1845543002/#ps40001
(title: " ")

[commit-bot: I haz the power, 2016-04-01 02:43:14]

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

[commit-bot: I haz the power, 2016-04-01 02:54:58]

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

[commit-bot: I haz the power, 2016-04-01 02:54:59]

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

[keishi, 2016-04-01 03:41:27]

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

[commit-bot: I haz the power, 2016-04-01 03:41:49]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1845543002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1845543002/60001

[commit-bot: I haz the power, 2016-04-01 05:20:16]

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

[commit-bot: I haz the power, 2016-04-01 05:20:17]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[keishi, 2016-04-11 08:47:56]

keishi@chromium.org changed reviewers:
+ caseq@chromium.org, peria@chromium.org

[keishi, 2016-04-11 08:47:58]

+peria Could you review tools/perf/measurements/oilpan_gc_times*.py?
+caseq Could you review TimelineModel.js?

Thanks!

[peria, 2016-04-11 08:55:52]

lgtm

oilpan_gc_times*.py LGTM

[caseq, 2016-04-11 13:31:25]

On 2016/04/11 08:47:58, keishi wrote:
> +caseq Could you review TimelineModel.js?
> 

Can we just drop the class name there, so that we don't have to rename it next
time class is renamed or an event is moved?

[keishi, 2016-04-11 16:31:01]

On 2016/04/11 13:31:25, caseq wrote:
> On 2016/04/11 08:47:58, keishi wrote:
> > +caseq Could you review TimelineModel.js?
> > 
> 
> Can we just drop the class name there, so that we don't have to rename it next
> time class is renamed or an event is moved?

Done. Renamed to BlinkGCMarking.

[caseq, 2016-04-11 16:33:31]

On 2016/04/11 16:31:01, keishi wrote:
> On 2016/04/11 13:31:25, caseq wrote:
> > On 2016/04/11 08:47:58, keishi wrote:
> > > +caseq Could you review TimelineModel.js?
> > > 
> > 
> > Can we just drop the class name there, so that we don't have to rename it
next
> > time class is renamed or an event is moved?
> 
> Done. Renamed to BlinkGCMarking.

lgtm and thanks!

[keishi, 2016-04-12 01:19:35]

The CQ bit was checked by keishi@chromium.org

[keishi, 2016-04-12 01:19:35]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, peria@chromium.org
Link to the patchset: https://codereview.chromium.org/1845543002/#ps100001
(title: "changed trace event to BlinkGCMarking")

[commit-bot: I haz the power, 2016-04-12 01:20:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1845543002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1845543002/100001

[commit-bot: I haz the power, 2016-04-12 01:48:37]

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

[commit-bot: I haz the power, 2016-04-12 01:48:39]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[keishi, 2016-04-12 02:42:33]

The CQ bit was checked by keishi@chromium.org

[keishi, 2016-04-12 02:42:34]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, peria@chromium.org, caseq@chromium.org
Link to the patchset: https://codereview.chromium.org/1845543002/#ps120001
(title: " ")

[commit-bot: I haz the power, 2016-04-12 02:43:02]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1845543002/120001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1845543002/120001

[commit-bot: I haz the power, 2016-04-12 03:04:50]

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

[commit-bot: I haz the power, 2016-04-12 03:04:52]

Try jobs failed on following builders:
  android_chromium_gn_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[keishi, 2016-04-12 04:16:50]

The CQ bit was checked by keishi@chromium.org

[keishi, 2016-04-12 04:16:51]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, peria@chromium.org, caseq@chromium.org
Link to the patchset: https://codereview.chromium.org/1845543002/#ps140001
(title: " ")

[commit-bot: I haz the power, 2016-04-12 04:17:10]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1845543002/140001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1845543002/140001

[commit-bot: I haz the power, 2016-04-12 05:23:51]

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

[commit-bot: I haz the power, 2016-04-12 05:23:54]

Try jobs failed on following builders:
  android_s5_perf_cq on tryserver.chromium.perf (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.perf/builders/android_s5_perf_...)

[keishi, 2016-04-13 06:02:32]

The CQ bit was checked by keishi@chromium.org

[keishi, 2016-04-13 06:02:33]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, peria@chromium.org, caseq@chromium.org
Link to the patchset: https://codereview.chromium.org/1845543002/#ps160001
(title: "rebased")

[commit-bot: I haz the power, 2016-04-13 06:02:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1845543002/160001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1845543002/160001

[commit-bot: I haz the power, 2016-04-13 07:15:17]

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

[commit-bot: I haz the power, 2016-04-13 07:15:20]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[keishi, 2016-04-13 07:41:32]

The CQ bit was checked by keishi@chromium.org

[keishi, 2016-04-13 07:41:33]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, peria@chromium.org, caseq@chromium.org
Link to the patchset: https://codereview.chromium.org/1845543002/#ps180001
(title: " ")

[commit-bot: I haz the power, 2016-04-13 07:41:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1845543002/180001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1845543002/180001

[commit-bot: I haz the power, 2016-04-13 08:55:22]

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

[commit-bot: I haz the power, 2016-04-13 08:55:25]

Try jobs failed on following builders:
  android_s5_perf_cq on tryserver.chromium.perf (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.perf/builders/android_s5_perf_...)

[keishi, 2016-04-13 09:28:00]

The CQ bit was checked by keishi@chromium.org

[keishi, 2016-04-13 09:28:01]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, peria@chromium.org, caseq@chromium.org
Link to the patchset: https://codereview.chromium.org/1845543002/#ps200001
(title: " ")

[commit-bot: I haz the power, 2016-04-13 09:28:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1845543002/200001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1845543002/200001

[commit-bot: I haz the power, 2016-04-13 09:40:23]

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

[commit-bot: I haz the power, 2016-04-13 09:40:25]

Try jobs failed on following builders:
  cast_shell_android on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)

[keishi, 2016-04-13 11:09:05]

The CQ bit was checked by keishi@chromium.org

[keishi, 2016-04-13 11:09:06]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, peria@chromium.org, caseq@chromium.org
Link to the patchset: https://codereview.chromium.org/1845543002/#ps220001
(title: " ")

[commit-bot: I haz the power, 2016-04-13 11:09:14]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1845543002/220001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1845543002/220001

[commit-bot: I haz the power, 2016-04-13 12:17:25]

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

[commit-bot: I haz the power, 2016-04-13 12:17:27]

Try jobs failed on following builders:
  android_s5_perf_cq on tryserver.chromium.perf (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.perf/builders/android_s5_perf_...)

[keishi, 2016-04-13 13:36:06]

The CQ bit was checked by keishi@chromium.org

[commit-bot: I haz the power, 2016-04-13 13:36:21]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1845543002/220001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1845543002/220001

[commit-bot: I haz the power, 2016-04-13 15:00:02]

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

[commit-bot: I haz the power, 2016-04-13 15:00:04]

Try jobs failed on following builders:
  android_s5_perf_cq on tryserver.chromium.perf (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.perf/builders/android_s5_perf_...)

[keishi, 2016-04-14 02:11:03]

Description was changed from

==========
Rename Heap to ThreadHeap

BUG=591606
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:android_s5_perf_cq;tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq
==========

to

==========
Rename Heap to ThreadHeap

BUG=591606
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq
==========

[keishi, 2016-04-14 02:11:13]

The CQ bit was checked by keishi@chromium.org

[keishi, 2016-04-14 02:11:14]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, peria@chromium.org, caseq@chromium.org
Link to the patchset: https://codereview.chromium.org/1845543002/#ps240001
(title: " ")

[commit-bot: I haz the power, 2016-04-14 02:11:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1845543002/240001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1845543002/240001

[commit-bot: I haz the power, 2016-04-14 04:04:23]

Description was changed from

==========
Rename Heap to ThreadHeap

BUG=591606
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq
==========

to

==========
Rename Heap to ThreadHeap

BUG=591606
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq
==========

[commit-bot: I haz the power, 2016-04-14 04:04:25]

Committed patchset #13 (id:240001)

[commit-bot: I haz the power, 2016-04-14 04:05:32]

Description was changed from

==========
Rename Heap to ThreadHeap

BUG=591606
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq
==========

to

==========
Rename Heap to ThreadHeap

BUG=591606
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq

Committed: https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e
Cr-Commit-Position: refs/heads/master@{#387233}
==========

[commit-bot: I haz the power, 2016-04-14 04:05:34]

Patchset 13 (id:??) landed as
https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e
Cr-Commit-Position: refs/heads/master@{#387233}

[keishi, 2016-04-14 07:11:20]

A revert of this CL (patchset #13 id:240001) has been created in
https://codereview.chromium.org/1884083002/ by keishi@chromium.org.

The reason for reverting is: Failed memory_test: content_unittests.

[keishi, 2016-04-14 11:05:53]

Description was changed from

==========
Rename Heap to ThreadHeap

BUG=591606
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq

Committed: https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e
Cr-Commit-Position: refs/heads/master@{#387233}
==========

to

==========
Rename Heap to ThreadHeap

BUG=591606
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq

Committed: https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e
Cr-Commit-Position: refs/heads/master@{#387233}
==========

[keishi, 2016-04-14 12:36:28]

The CQ bit was checked by keishi@chromium.org

[keishi, 2016-04-14 12:36:28]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, peria@chromium.org, caseq@chromium.org
Link to the patchset: https://codereview.chromium.org/1845543002/#ps260001
(title: " ")

[commit-bot: I haz the power, 2016-04-14 12:36:54]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1845543002/260001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1845543002/260001

[commit-bot: I haz the power, 2016-04-14 14:21:50]

Description was changed from

==========
Rename Heap to ThreadHeap

BUG=591606
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq

Committed: https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e
Cr-Commit-Position: refs/heads/master@{#387233}
==========

to

==========
Rename Heap to ThreadHeap

BUG=591606
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq

Committed: https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e
Cr-Commit-Position: refs/heads/master@{#387233}
==========

[commit-bot: I haz the power, 2016-04-14 14:21:53]

Committed patchset #14 (id:260001)

[commit-bot: I haz the power, 2016-04-14 14:22:55]

Description was changed from

==========
Rename Heap to ThreadHeap

BUG=591606
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq

Committed: https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e
Cr-Commit-Position: refs/heads/master@{#387233}
==========

to

==========
Rename Heap to ThreadHeap

BUG=591606
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq

Committed: https://crrev.com/191c640cc25f76c0c0f8a0b448de33cd059f4a8e
Cr-Commit-Position: refs/heads/master@{#387233}

Committed: https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62
Cr-Commit-Position: refs/heads/master@{#387307}
==========

[commit-bot: I haz the power, 2016-04-14 14:22:58]

Patchset 14 (id:??) landed as
https://crrev.com/011b8046a07ac2cedde530a05f81e8fb41f80c62
Cr-Commit-Position: refs/heads/master@{#387307}