Tom, I uploaded a new CL since the simpler FilterURL change blew up in unit
tests. I'm passing in the RenderProcessHost now instead of the ID, though I was
hoping to avoid that.
(It might be possible to skip the FilterURL check, but I felt like it's worth
having it in there.)

LGTM
Btw, all the stuff where you switch passing the render process id to passing the
RenderProcessHost object? Make sure you have a proper review for that. I didn't
understand it, so hopefully someone else did. If not, you should make sure
someone reviews it who understands what you're doing :)

On 2012/10/28 07:58:20, willchan wrote:
> Btw, all the stuff where you switch passing the render process id to passing
the
> RenderProcessHost object? Make sure you have a proper review for that. I
didn't
> understand it, so hopefully someone else did. If not, you should make sure
> someone reviews it who understands what you're doing :)
Yes, I'm waiting on Tom's review for that part. Thanks!
On 2012/10/29 00:48:09, Fady Samuel wrote:
> Please remove the TODO in BrowserPluginEmbedder::NavigateGuest. Thanks!
Good catch. Done.