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

Issue 7314007: All new file i/o documentation. (Closed)

Created:
9 years, 5 months ago by jond
Modified:
9 years, 5 months ago
CC:
chromium-reviews, piman+watch_chromium.org
Visibility:
Public.

Description

All new file i/o documentation. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=91753

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Total comments: 2

Patch Set 4 : '' #

Patch Set 5 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+376 lines, -87 lines) Patch
M ppapi/c/ppb_file_io.h View 1 2 3 4 1 chunk +181 lines, -47 lines 0 comments Download
M ppapi/c/ppb_file_ref.h View 1 2 3 1 chunk +140 lines, -33 lines 0 comments Download
M ppapi/c/ppb_file_system.h View 2 chunks +55 lines, -7 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
jond
9 years, 5 months ago (2011-07-06 19:56:50 UTC) #1
sanga
http://codereview.chromium.org/7314007/diff/5001/ppapi/c/ppb_file_ref.h File ppapi/c/ppb_file_ref.h (right): http://codereview.chromium.org/7314007/diff/5001/ppapi/c/ppb_file_ref.h#newcode80 ppapi/c/ppb_file_ref.h:80: * file. Use GetPath() get the full file path. ...
9 years, 5 months ago (2011-07-06 23:20:59 UTC) #2
jond
http://codereview.chromium.org/7314007/diff/5001/ppapi/c/ppb_file_ref.h File ppapi/c/ppb_file_ref.h (right): http://codereview.chromium.org/7314007/diff/5001/ppapi/c/ppb_file_ref.h#newcode80 ppapi/c/ppb_file_ref.h:80: * file. Use GetPath() get the full file path. ...
9 years, 5 months ago (2011-07-07 15:11:15 UTC) #3
sanga
lgtm
9 years, 5 months ago (2011-07-07 16:28:40 UTC) #4
jond
9 years, 5 months ago (2011-07-07 21:00:43 UTC) #5
dmichael (off chromium)
9 years, 5 months ago (2011-07-07 21:07:52 UTC) #6
rubber-stamp LGTM

Powered by Google App Engine
This is Rietveld 408576698