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

Issue 1074943003: Rewrite and move Cluster Telemetry documentation to skia.org (Closed)

Created:
5 years, 8 months ago by rmistry
Modified:
5 years, 8 months ago
Reviewers:
jcgregorio, hcm
CC:
reviews_skia.org
Base URL:
https://skia.googlesource.com/skia@master
Target Ref:
refs/heads/master
Project:
skia
Visibility:
Public.

Description

Rewrite and move Cluster Telemetry documentation to skia.org BUG=skia:3754 NOTRY=true DOCS_PREVIEW= https://skia.org/?cl=1074943003 Committed: https://skia.googlesource.com/skia/+/8892277d0b31c51d21e009759c7f543c7efb02d8

Patch Set 1 : Initial upload #

Patch Set 2 : Update #

Patch Set 3 : Add system diagram #

Patch Set 4 : More content #

Patch Set 5 : Rearrange #

Patch Set 6 : Fix link #

Patch Set 7 : Cleanup #

Patch Set 8 : Rename section #

Patch Set 9 : List more usage examples #

Patch Set 10 : Improve links #

Patch Set 11 : More cleanup #

Total comments: 2

Patch Set 12 : Address comment #

Unified diffs Side-by-side diffs Delta from patch set Stats (+102 lines, -0 lines) Patch
A site/dev/testing/ct.md View 1 2 3 4 5 6 7 8 9 10 11 1 chunk +102 lines, -0 lines 0 comments Download
A site/dev/testing/ct-system.png View 1 2 Binary file 0 comments Download

Messages

Total messages: 15 (9 generated)
rmistry
5 years, 8 months ago (2015-04-22 12:37:08 UTC) #8
jcgregorio
lgtm
5 years, 8 months ago (2015-04-22 16:52:56 UTC) #9
hcm
lgtm with one minor grammatical edit https://codereview.chromium.org/1074943003/diff/310001/site/dev/testing/ct.md File site/dev/testing/ct.md (right): https://codereview.chromium.org/1074943003/diff/310001/site/dev/testing/ct.md#newcode79 site/dev/testing/ct.md:79: 1. User submits ...
5 years, 8 months ago (2015-04-22 17:00:43 UTC) #10
rmistry
https://codereview.chromium.org/1074943003/diff/310001/site/dev/testing/ct.md File site/dev/testing/ct.md (right): https://codereview.chromium.org/1074943003/diff/310001/site/dev/testing/ct.md#newcode79 site/dev/testing/ct.md:79: 1. User submits a Admin task (rebuild chrome, recreate ...
5 years, 8 months ago (2015-04-22 18:02:28 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1074943003/330001
5 years, 8 months ago (2015-04-22 18:03:02 UTC) #14
commit-bot: I haz the power
5 years, 8 months ago (2015-04-22 18:03:18 UTC) #15
Message was sent while issue was closed.
Committed patchset #12 (id:330001) as
https://skia.googlesource.com/skia/+/8892277d0b31c51d21e009759c7f543c7efb02d8

Powered by Google App Engine
This is Rietveld 408576698