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

Issue 345703002: Adding development environment tutorial. (Closed)

Created:
6 years, 6 months ago by bradn
Modified:
6 years, 6 months ago
Reviewers:
mackinlay, binji, Sam Clegg
CC:
chromium-reviews, binji+watch_chromium.org, native-client-reviews_googlegroups.com, Sam Clegg
Project:
chromium
Visibility:
Public.

Description

Adding development environment tutorial. First cut at the landing page. Missing debugger walk thru. BUG=None TEST=None R=binji@chromium.org,sbc@chromium.org

Patch Set 1 #

Total comments: 8

Patch Set 2 : fix #

Total comments: 6

Patch Set 3 : fix #

Patch Set 4 : drop extra #

Total comments: 1

Patch Set 5 : fix #

Unified diffs Side-by-side diffs Delta from patch set Stats (+363 lines, -2 lines) Patch
A native_client_sdk/doc_generated/images/voronoi1.png View 1 2 Binary file 0 comments Download
A native_client_sdk/doc_generated/images/voronoi2.png View 1 2 Binary file 0 comments Download
A native_client_sdk/doc_generated/io2014.html View 1 2 3 4 1 chunk +168 lines, -0 lines 0 comments Download
M native_client_sdk/doc_generated/sitemap.html View 1 chunk +4 lines, -0 lines 0 comments Download
M native_client_sdk/src/doc/README View 1 1 chunk +4 lines, -2 lines 0 comments Download
A native_client_sdk/src/doc/images/voronoi1.png View 1 2 Binary file 0 comments Download
A native_client_sdk/src/doc/images/voronoi2.png View 1 2 Binary file 0 comments Download
A native_client_sdk/src/doc/io2014.rst View 1 2 3 4 1 chunk +186 lines, -0 lines 0 comments Download
M native_client_sdk/src/doc/sitemap.rst View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
bradn
First draft, PTAL.
6 years, 6 months ago (2014-06-19 00:58:51 UTC) #1
Sam Clegg
https://codereview.chromium.org/345703002/diff/1/native_client_sdk/src/doc/README File native_client_sdk/src/doc/README (right): https://codereview.chromium.org/345703002/diff/1/native_client_sdk/src/doc/README#newcode27 native_client_sdk/src/doc/README:27: ``python-beautifulsoup`` Use rst list syntax like above? https://codereview.chromium.org/345703002/diff/1/native_client_sdk/src/doc/io2014.rst File ...
6 years, 6 months ago (2014-06-19 01:09:52 UTC) #2
bradn
https://codereview.chromium.org/345703002/diff/1/native_client_sdk/src/doc/README File native_client_sdk/src/doc/README (right): https://codereview.chromium.org/345703002/diff/1/native_client_sdk/src/doc/README#newcode27 native_client_sdk/src/doc/README:27: ``python-beautifulsoup`` On 2014/06/19 01:09:52, Sam Clegg wrote: > Use ...
6 years, 6 months ago (2014-06-19 17:22:56 UTC) #3
Sam Clegg
lgtm https://codereview.chromium.org/345703002/diff/20001/native_client_sdk/src/doc/io2014.rst File native_client_sdk/src/doc/io2014.rst (right): https://codereview.chromium.org/345703002/diff/20001/native_client_sdk/src/doc/io2014.rst#newcode149 native_client_sdk/src/doc/io2014.rst:149: </svg> Shame about the raw html.. maybe we ...
6 years, 6 months ago (2014-06-19 17:37:26 UTC) #4
binji
https://codereview.chromium.org/345703002/diff/20001/native_client_sdk/src/doc/io2014.rst File native_client_sdk/src/doc/io2014.rst (right): https://codereview.chromium.org/345703002/diff/20001/native_client_sdk/src/doc/io2014.rst#newcode149 native_client_sdk/src/doc/io2014.rst:149: </svg> On 2014/06/19 17:37:26, Sam Clegg wrote: > Shame ...
6 years, 6 months ago (2014-06-19 19:12:56 UTC) #5
bradn
https://codereview.chromium.org/345703002/diff/20001/native_client_sdk/src/doc/io2014.rst File native_client_sdk/src/doc/io2014.rst (right): https://codereview.chromium.org/345703002/diff/20001/native_client_sdk/src/doc/io2014.rst#newcode149 native_client_sdk/src/doc/io2014.rst:149: </svg> On 2014/06/19 17:37:26, Sam Clegg wrote: > Shame ...
6 years, 6 months ago (2014-06-19 21:52:36 UTC) #6
Sam Clegg
slgtm. https://codereview.chromium.org/345703002/diff/60001/native_client_sdk/src/doc/io2014.rst File native_client_sdk/src/doc/io2014.rst (right): https://codereview.chromium.org/345703002/diff/60001/native_client_sdk/src/doc/io2014.rst#newcode172 native_client_sdk/src/doc/io2014.rst:172: Remove extra newline.
6 years, 6 months ago (2014-06-19 22:11:17 UTC) #7
bradn
6 years, 6 months ago (2014-06-19 22:34:48 UTC) #8
Message was sent while issue was closed.
Commited as 278510

Powered by Google App Engine
This is Rietveld 408576698