PPAPI: Remove out-dated or unnecessary TODOs from public files.

PPAPI: Remove out-dated or unnecessary TODOs from public files.

BUG=
TEST=


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

[dmichael (off chromium), 2012-03-19 17:41:52]

http://codereview.chromium.org/9731003/diff/1/ppapi/api/ppb_file_system.idl
File ppapi/api/ppb_file_system.idl (left):

http://codereview.chromium.org/9731003/diff/1/ppapi/api/ppb_file_system.idl#o...
ppapi/api/ppb_file_system.idl:59: */
The first sentence in the documentation covers this, so this TODO is
unnecessary.

http://codereview.chromium.org/9731003/diff/1/ppapi/api/ppb_graphics_2d.idl
File ppapi/api/ppb_graphics_2d.idl (left):

http://codereview.chromium.org/9731003/diff/1/ppapi/api/ppb_graphics_2d.idl#o...
ppapi/api/ppb_graphics_2d.idl:244: * that it is easier for consumers to manage
memory referenced by a callback.
polina's work for "required" callbacks fixed this a while ago (though developers
can still use "optional" callbacks if they want; it's just not the default).

http://codereview.chromium.org/9731003/diff/1/ppapi/api/ppb_graphics_2d.idl#o...
ppapi/api/ppb_graphics_2d.idl:247: * for it.
That's just part of planned work.

http://codereview.chromium.org/9731003/diff/1/ppapi/cpp/var.cc
File ppapi/cpp/var.cc (left):

http://codereview.chromium.org/9731003/diff/1/ppapi/cpp/var.cc#oldcode224
ppapi/cpp/var.cc:224: // TODO(sehr,polina): remove this after newlib is changed.
this static_cast is correct; there's no need to remove it. You should never
assume int32_t is some particular type anyway.

http://codereview.chromium.org/9731003/diff/1/ppapi/cpp/var.cc#oldcode240
ppapi/cpp/var.cc:240: // virtual?
Just a nice-to-have.

[cstefansen, 2012-03-20 00:50:59]

lgtm

Thanks for taking care of this.

[commit-bot: I haz the power, 2012-03-20 15:08:48]

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-03-21 02:42:40]

+bbudge to have a committer review (or just rubber stamp if you like).

[bbudge-google, 2012-03-21 05:39:43]

lgtm

[commit-bot: I haz the power, 2012-03-21 05:53:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dmichael@chromium.org/9731003/1

[commit-bot: I haz the power, 2012-03-21 07:01:11]

Change committed as 127918