Pepper: Clean up PpapiDispatcher parent classes.

Pepper: Clean up PpapiDispatcher parent classes.

This change modifies PpapiDispatcher to inherit IPC::Listener and IPC::Sender
instead of ProxyChannel directly. This cleans up the class hierarchy a bit.

BUG=
R=dmichael@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=252108

[teravest, 2014-02-19 19:20:40]

I think this is easier to read, and the class wasn't using many features of
ProxyChannel, so I think this is more readable.

[dmichael (off chromium), 2014-02-19 19:23:51]

Thanks, that does seem better. lgtm

https://codereview.chromium.org/172103008/diff/1/ppapi/proxy/plugin_main_nacl.cc
File ppapi/proxy/plugin_main_nacl.cc (right):

https://codereview.chromium.org/172103008/diff/1/ppapi/proxy/plugin_main_nacl...
ppapi/proxy/plugin_main_nacl.cc:207: channel_.reset();
I wonder if we should exit() here...

[teravest, 2014-02-19 20:31:08]

The CQ bit was checked by teravest@chromium.org

[teravest, 2014-02-19 20:31:14]

On Wed, Feb 19, 2014 at 12:23 PM,  <dmichael@chromium.org> wrote:
> Thanks, that does seem better. lgtm
>
>
>
https://codereview.chromium.org/172103008/diff/1/ppapi/proxy/plugin_main_nacl.cc
> File ppapi/proxy/plugin_main_nacl.cc (right):
>
>
https://codereview.chromium.org/172103008/diff/1/ppapi/proxy/plugin_main_nacl...
> ppapi/proxy/plugin_main_nacl.cc:207: channel_.reset();
> I wonder if we should exit() here...

Done.

>
> https://codereview.chromium.org/172103008/

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2014-02-19 20:49:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/teravest@chromium.org/172103008/60001

[teravest, 2014-02-19 21:59:23]

Committed patchset #2 manually as r252108 (presubmit successful).