Take 2 at fixing anchor links opening in background tabs on linux.

Take 2 at fixing anchor links opening in background tabs on linux.
Also fix a code path that seems to be missing on Windows (maybe
this regressed?).

This time make sure RWHV is not NULL.

BUG=619
TEST=http://code.google.com/p/chromium/issues/detail?id=619#c6

[Nate Chapin, 2009-08-13 18:24:36]

http://codereview.chromium.org/165468/diff/1/3
File chrome/browser/tab_contents/tab_contents_view_gtk.cc (right):

http://codereview.chromium.org/165468/diff/1/3#newcode582
Line 582: if (tab_contents()->render_widget_host_view()) {
I don't have a great understanding of this layer of the code, so pardon my dumb
question:  Is there any reason we would want to initialize the RWHV here instead
of doing nothing if it's null?

[Evan Stade, 2009-08-13 18:30:52]

lgtm

http://codereview.chromium.org/165468/diff/1/3
File chrome/browser/tab_contents/tab_contents_view_gtk.cc (right):

http://codereview.chromium.org/165468/diff/1/3#newcode582
Line 582: if (tab_contents()->render_widget_host_view()) {
On 2009/08/13 18:24:36, Nate Chapin wrote:
> I don't have a great understanding of this layer of the code, so pardon my
dumb
> question:  Is there any reason we would want to initialize the RWHV here
instead
> of doing nothing if it's null?

The tab contents view doesn't own the RWHV, so no.