Minor changes as per Christians most-recent review of the docs.

Minor changes as per Christians most-recent review of the docs.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=122337

[noelallen_use_chromium, 2012-02-11 00:09:40]

LGTM

I don't remember seeing this.  Make sure you --send_mail when you upload, or
just ping me if you don't get a quick response.

[commit-bot: I haz the power, 2012-02-15 17:07:37]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[dmichael (off chromium), 2012-02-15 17:53:50]

http://codereview.chromium.org/9288006/diff/1011/ppapi/api/ppb_file_system.idl
File ppapi/api/ppb_file_system.idl (right):

http://codereview.chromium.org/9288006/diff/1011/ppapi/api/ppb_file_system.id...
ppapi/api/ppb_file_system.idl:57: // be opened in it. Clarify what it means to
be "completed."
Use C-style comments here ( /* ... */ )

http://codereview.chromium.org/9288006/diff/1011/ppapi/cpp/input_event.h
File ppapi/cpp/input_event.h (right):

http://codereview.chromium.org/9288006/diff/1011/ppapi/cpp/input_event.h#newc...
ppapi/cpp/input_event.h:164: /// movement information is provided only if the
mouse is locked. If the mouse
164 and 165 also should have "//" instead of "///", right?

[jond, 2012-02-15 22:33:53]

http://codereview.chromium.org/9288006/diff/1011/ppapi/api/ppb_file_system.idl
File ppapi/api/ppb_file_system.idl (right):

http://codereview.chromium.org/9288006/diff/1011/ppapi/api/ppb_file_system.id...
ppapi/api/ppb_file_system.idl:57: // be opened in it. Clarify what it means to
be "completed."
On 2012/02/15 17:53:50, dmichael wrote:
> Use C-style comments here ( /* ... */ )

Done.

http://codereview.chromium.org/9288006/diff/1011/ppapi/cpp/input_event.h
File ppapi/cpp/input_event.h (right):

http://codereview.chromium.org/9288006/diff/1011/ppapi/cpp/input_event.h#newc...
ppapi/cpp/input_event.h:164: /// movement information is provided only if the
mouse is locked. If the mouse
On 2012/02/15 17:53:50, dmichael wrote:
> 164 and 165 also should have "//" instead of "///", right?

Done.

[dmichael (off chromium), 2012-02-15 22:40:36]

lgtm

[dmichael (off chromium), 2012-02-15 22:42:25]

Oops, one more thing. Otherwise lgtm.

http://codereview.chromium.org/9288006/diff/11001/ppapi/cpp/var.h
File ppapi/cpp/var.h (right):

http://codereview.chromium.org/9288006/diff/11001/ppapi/cpp/var.h#newcode72
ppapi/cpp/var.h:72: ///             
http://code.google.com/p/chromium/issues/detail?id=52105
This bug should also have a // comment instead of ///

[jond, 2012-02-16 18:10:08]

http://codereview.chromium.org/9288006/diff/11001/ppapi/cpp/var.h
File ppapi/cpp/var.h (right):

http://codereview.chromium.org/9288006/diff/11001/ppapi/cpp/var.h#newcode72
ppapi/cpp/var.h:72: ///             
http://code.google.com/p/chromium/issues/detail?id=52105
On 2012/02/15 22:42:25, dmichael wrote:
> This bug should also have a // comment instead of ///

Done.

[commit-bot: I haz the power, 2012-02-16 18:10:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jond@google.com/9288006/15001

[commit-bot: I haz the power, 2012-02-16 19:33:11]

Change committed as 122337