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

Issue 6588085: Minor doc changes. Mostly commented out TODO and {PENDING notes that engineer... (Closed)

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

Description

Minor doc changes. Mostly commented out TODO and {PENDING notes that engineers had made. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=76795

Patch Set 1 #

Total comments: 19

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+90 lines, -43 lines) Patch
M ppapi/c/pp_input_event.h View 1 2 chunks +3 lines, -3 lines 0 comments Download
M ppapi/c/ppb_class.h View 1 1 chunk +2 lines, -1 line 0 comments Download
M ppapi/c/ppb_core.h View 1 2 chunks +62 lines, -22 lines 0 comments Download
M ppapi/c/ppb_var.h View 1 10 chunks +20 lines, -15 lines 0 comments Download
M ppapi/c/ppp.h View 1 1 chunk +3 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
jond
Just commented out the TODO and {PENDING notes that engineers had left for themselves. Maybe ...
9 years, 9 months ago (2011-03-01 18:15:36 UTC) #1
dmichael(do not use this one)
http://codereview.chromium.org/6588085/diff/1/ppapi/c/ppb_class.h File ppapi/c/ppb_class.h (right): http://codereview.chromium.org/6588085/diff/1/ppapi/c/ppb_class.h#newcode18 ppapi/c/ppb_class.h:18: * This file defines the PPB_Class struct which is ...
9 years, 9 months ago (2011-03-02 15:15:26 UTC) #2
jond
http://codereview.chromium.org/6588085/diff/1/ppapi/c/ppb_class.h File ppapi/c/ppb_class.h (right): http://codereview.chromium.org/6588085/diff/1/ppapi/c/ppb_class.h#newcode18 ppapi/c/ppb_class.h:18: * This file defines the PPB_Class struct which is ...
9 years, 9 months ago (2011-03-02 17:11:12 UTC) #3
dmichael(do not use this one)
9 years, 9 months ago (2011-03-02 17:15:18 UTC) #4
LGTM, assuming trybots pass

Powered by Google App Engine
This is Rietveld 408576698