Get rid of RenderViewHostDelegate::BrowserIntegration since it was unnecessar...

Get rid of RenderViewHostDelegate::BrowserIntegration since it was unnecessary.  RenderViewHost used to dispatch messages only to call that interface, and TabContents would do the work.  Instead have TabContents dispatch the messages directly.

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

[Avi (use Gerrit), 2011-01-21 21:23:19]

Uh, thanks for giving me more stuff to refactor? :)

Code LGTM

[jam, 2011-01-21 21:26:23]

I may be not understanding your comment, so if I'm wildly off, please
ignore!  This change is just getting rid of unnecessary code in getting
these notifications to TabContents.  The refactoring so that TabContents
doesn't have to know about all these things should still be done.

On Fri, Jan 21, 2011 at 1:23 PM, <avi@chromium.org> wrote:

> Uh, thanks for giving me more stuff to refactor? :)
>
> Code LGTM
>
>
> http://codereview.chromium.org/6336012/
>

[Evan Martin, 2011-07-20 20:09:46]

http://codereview.chromium.org/6336012/diff/33001/chrome/browser/renderer_hos...
File chrome/browser/renderer_host/render_view_host_delegate.h (right):

http://codereview.chromium.org/6336012/diff/33001/chrome/browser/renderer_hos...
chrome/browser/renderer_host/render_view_host_delegate.h:679: void
WorkerCrashed() {}
Missing 'virtual'.  Dun dun dunnnnnnn.  Review carefully!