Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(149)

Issue 2494063002: Move memory docs to docs/memory-infra (Closed)

Created:
4 years, 1 month ago by hjd
Modified:
4 years, 1 month ago
CC:
chromium-reviews, tracing+reviews_chromium.org, wfh+watch_chromium.org, cc-bugs_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Move memory docs to docs/memory-infra I replaced the old documents with links that redirect to the new locations so we don't break external links. Also fixes a couple of broken links I found. BUG=chromium:663353 Committed: https://crrev.com/0304f11a58c3a10b78a12c317d18f553d4cbb889 Cr-Commit-Position: refs/heads/master@{#431920}

Patch Set 1 #

Patch Set 2 : more #

Patch Set 3 : rename x to probe-x #

Patch Set 4 : rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -997 lines) Patch
M base/allocator/README.md View 1 2 1 chunk +2 lines, -2 lines 0 comments Download
M cc/memory.md View 1 2 1 chunk +1 line, -125 lines 0 comments Download
M components/tracing/docs/adding_memory_infra_tracing.md View 1 1 chunk +1 line, -177 lines 0 comments Download
M components/tracing/docs/heap_profiler.md View 1 1 chunk +1 line, -167 lines 0 comments Download
M components/tracing/docs/heap_profiler_internals.md View 1 1 chunk +1 line, -183 lines 0 comments Download
M components/tracing/docs/memory_infra.md View 1 2 3 1 chunk +1 line, -170 lines 0 comments Download
M components/tracing/docs/memory_infra_gpu.md View 1 2 1 chunk +1 line, -92 lines 0 comments Download
M components/tracing/docs/memory_infra_startup_tracing.md View 1 1 chunk +1 line, -73 lines 0 comments Download
A + docs/memory-infra/README.md View 1 2 3 1 chunk +3 lines, -3 lines 0 comments Download
A + docs/memory-infra/adding_memory_infra_tracing.md View 1 0 chunks +-1 lines, --1 lines 0 comments Download
A + docs/memory-infra/heap_profiler.md View 1 1 chunk +1 line, -1 line 0 comments Download
A + docs/memory-infra/heap_profiler_internals.md View 1 0 chunks +-1 lines, --1 lines 0 comments Download
A + docs/memory-infra/memory_infra_startup_tracing.md View 1 2 chunks +2 lines, -2 lines 0 comments Download
A + docs/memory-infra/probe-cc.md View 1 2 1 chunk +1 line, -1 line 0 comments Download
A + docs/memory-infra/probe-gpu.md View 1 2 3 chunks +3 lines, -3 lines 0 comments Download

Messages

Total messages: 28 (14 generated)
hjd
+Sami for cc/ OWNERS stamp +primiano for everything else ptal, thanks! :)
4 years, 1 month ago (2016-11-11 13:19:47 UTC) #3
Sami
cc/ lgtm assuming Primiano is happy with this direction.
4 years, 1 month ago (2016-11-11 14:10:40 UTC) #4
Primiano Tucci (use gerrit)
Yup LGTM thanks. Can you also update the pointers in the wiki https://www.chromium.org/developers/how-tos/trace-event-profiling-tool and the ...
4 years, 1 month ago (2016-11-11 14:25:40 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2494063002/20001
4 years, 1 month ago (2016-11-11 14:25:54 UTC) #7
Primiano Tucci (use gerrit)
Ah actually, can we move files like cc.md and memory_infra_gpu.md into /docs/memory-infra/probe-cc.md /docs/memory-infra/probe-gpu.md or something ...
4 years, 1 month ago (2016-11-11 14:27:53 UTC) #9
hjd
On 2016/11/11 14:27:53, Primiano Tucci wrote: > Ah actually, can we move files like cc.md ...
4 years, 1 month ago (2016-11-11 15:26:57 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2494063002/40001
4 years, 1 month ago (2016-11-11 23:26:35 UTC) #13
commit-bot: I haz the power
Try jobs failed on following builders: linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_chromeos_rel_ng/builds/313695) ios-simulator on master.tryserver.chromium.mac (JOB_FAILED, ...
4 years, 1 month ago (2016-11-11 23:29:51 UTC) #15
hjd
ptal, rebased
4 years, 1 month ago (2016-11-14 18:50:26 UTC) #16
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2494063002/60001
4 years, 1 month ago (2016-11-14 18:58:11 UTC) #19
commit-bot: I haz the power
Try jobs failed on following builders: linux_trusty_blink_rel on master.tryserver.blink (JOB_TIMED_OUT, no build URL)
4 years, 1 month ago (2016-11-14 21:04:56 UTC) #21
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2494063002/60001
4 years, 1 month ago (2016-11-14 21:22:08 UTC) #24
commit-bot: I haz the power
Committed patchset #4 (id:60001)
4 years, 1 month ago (2016-11-14 22:37:27 UTC) #26
commit-bot: I haz the power
4 years, 1 month ago (2016-11-14 22:39:49 UTC) #28
Message was sent while issue was closed.
Patchset 4 (id:??) landed as
https://crrev.com/0304f11a58c3a10b78a12c317d18f553d4cbb889
Cr-Commit-Position: refs/heads/master@{#431920}

Powered by Google App Engine
This is Rietveld 408576698