Some small refactoring and renaming of sel_universel pepper emulation components.

Some small refactoring and renaming of sel_universel pepper emulation components.
This is in anticipation of making some pepper emulation (e.g. file access) independent
of the availability of SDL which is only used for video and audio.
Committed: http://src.chromium.org/viewvc/native_client?view=rev&revision=5613

[robertm, 2011-06-08 22:54:29]

PTAL

[robertm, 2011-06-09 14:26:24]

+ eugenis

[Sang Ahn, 2011-06-09 20:34:48]

http://codereview.chromium.org/7046064/diff/1/src/trusted/sel_universal/non_s...
File src/trusted/sel_universal/non_standard_pepper_events.cc (right):

http://codereview.chromium.org/7046064/diff/1/src/trusted/sel_universal/non_s...
src/trusted/sel_universal/non_standard_pepper_events.cc:7: // This file provives
a utilty function to translate and SDL event structure
s/provives/provides
s/and/an

http://codereview.chromium.org/7046064/diff/1/src/trusted/sel_universal/non_s...
src/trusted/sel_universal/non_standard_pepper_events.cc:8: // into a PPAPI event
structure.
s/into/to

http://codereview.chromium.org/7046064/diff/1/src/trusted/sel_universal/non_s...
src/trusted/sel_universal/non_standard_pepper_events.cc:26: } 
PP_InputEvent_User;
Could this be wrapped to a namespace?  It might be confusing, or possible
collision w/ pepper in the future.

http://codereview.chromium.org/7046064/diff/1/src/trusted/sel_universal/peppe...
File src/trusted/sel_universal/pepper_emu_handler.cc (right):

http://codereview.chromium.org/7046064/diff/1/src/trusted/sel_universal/peppe...
src/trusted/sel_universal/pepper_emu_handler.cc:580: const vector<string>& args)
{
Emu is short for Emulator?

[robertm, 2011-06-09 21:02:21]

PTAL

http://codereview.chromium.org/7046064/diff/1/src/trusted/sel_universal/non_s...
File src/trusted/sel_universal/non_standard_pepper_events.cc (right):

http://codereview.chromium.org/7046064/diff/1/src/trusted/sel_universal/non_s...
src/trusted/sel_universal/non_standard_pepper_events.cc:7: // This file provives
a utilty function to translate and SDL event structure
On 2011/06/09 20:34:48, Sang Ahn wrote:
> s/provives/provides
> s/and/an

Done.

http://codereview.chromium.org/7046064/diff/1/src/trusted/sel_universal/non_s...
src/trusted/sel_universal/non_standard_pepper_events.cc:8: // into a PPAPI event
structure.
On 2011/06/09 20:34:48, Sang Ahn wrote:
> s/into/to

Done.

http://codereview.chromium.org/7046064/diff/1/src/trusted/sel_universal/non_s...
src/trusted/sel_universal/non_standard_pepper_events.cc:26: } 
PP_InputEvent_User;
excellent idea - done
On 2011/06/09 20:34:48, Sang Ahn wrote:
> Could this be wrapped to a namespace?  It might be confusing, or possible
> collision w/ pepper in the future.

http://codereview.chromium.org/7046064/diff/1/src/trusted/sel_universal/peppe...
File src/trusted/sel_universal/pepper_emu_handler.cc (right):

http://codereview.chromium.org/7046064/diff/1/src/trusted/sel_universal/peppe...
src/trusted/sel_universal/pepper_emu_handler.cc:580: const vector<string>& args)
{
yes
On 2011/06/09 20:34:48, Sang Ahn wrote:
> Emu is short for Emulator?

[Sang Ahn, 2011-06-09 21:53:03]

lgtm