Pepper: Add a C++ wrapper for the trusted file chooser.

Pepper: Add a C++ wrapper for the trusted file chooser.

This makes it easy (for trusted Pepper plugins) to opt for showing a file
chooser while avoiding the user-gesture check.

BUG=none
TEST=none

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

[bbudge, 2011-10-13 21:39:50]

LGTM.

[bbudge, 2011-10-13 21:42:23]

Whoops, too quick. A comment.

http://codereview.chromium.org/8275013/diff/1005/ppapi/cpp/dev/file_chooser_d...
File ppapi/cpp/dev/file_chooser_dev.h (right):

http://codereview.chromium.org/8275013/diff/1005/ppapi/cpp/dev/file_chooser_d...
ppapi/cpp/dev/file_chooser_dev.h:68: virtual FileRef GetNextChosenFile() const;
Does this need to be virtual?

[viettrungluu, 2011-10-13 21:47:56]

http://codereview.chromium.org/8275013/diff/1005/ppapi/cpp/dev/file_chooser_d...
File ppapi/cpp/dev/file_chooser_dev.h (right):

http://codereview.chromium.org/8275013/diff/1005/ppapi/cpp/dev/file_chooser_d...
ppapi/cpp/dev/file_chooser_dev.h:68: virtual FileRef GetNextChosenFile() const;
On 2011/10/13 21:42:23, bbudge1 wrote:
> Does this need to be virtual?

No it doesn't, but I figure it should be overrideable too, if Show() is.

[bbudge, 2011-10-13 22:08:11]

On 2011/10/13 21:47:56, viettrungluu wrote:
>
http://codereview.chromium.org/8275013/diff/1005/ppapi/cpp/dev/file_chooser_d...
> File ppapi/cpp/dev/file_chooser_dev.h (right):
> 
>
http://codereview.chromium.org/8275013/diff/1005/ppapi/cpp/dev/file_chooser_d...
> ppapi/cpp/dev/file_chooser_dev.h:68: virtual FileRef GetNextChosenFile()
const;
> On 2011/10/13 21:42:23, bbudge1 wrote:
> > Does this need to be virtual?
> 
> No it doesn't, but I figure it should be overrideable too, if Show() is.

OK then.

[piman, 2011-10-13 22:26:59]

lgtm