Bug fix: in process router should not expect msg->is_sync

Bug fix: in process router should not expect a sync message from host to plugin.  Replies to sync messages are not sync message.

BUG=


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

[victorhsieh, 2012-10-11 23:06:39]

Hey Raymes, this is what we discussed yesterday.  Thanks for finding this with
me.

[raymes, 2012-10-12 18:41:45]

lgtm

[raymes, 2012-10-12 18:45:51]

On 2012/10/12 18:41:45, raymes wrote:
> lgtm

Please update the description. The reason this is needed is because replies to
sync messages are not sync messages. This means that a sync message should never
be sent from the host to the plugin in-process (because this functionality is
currently unsupported).

[victorhsieh, 2012-10-13 00:22:13]

On 2012/10/12 18:45:51, raymes wrote:
> On 2012/10/12 18:41:45, raymes wrote:
> > lgtm
> 
> Please update the description. The reason this is needed is because replies to
> sync messages are not sync messages. This means that a sync message should
never
> be sent from the host to the plugin in-process (because this functionality is
> currently unsupported).

See if that's clear.  If so, would you please submit for me?  I don't have
commit bit.

[raymes, 2012-10-15 23:02:50]

+brettw for OWNERS

[brettw, 2012-10-16 17:37:46]

lgtm

[commit-bot: I haz the power, 2012-10-16 17:45:49]

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

[commit-bot: I haz the power, 2012-10-16 20:23:51]

Change committed as 162229