Properly restart extensions with no visible UI.

Properly restart extensions with no visible UI.

http://crbug.com/16070

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

[Paweł Hajdan Jr., 2009-07-09 21:29:19]

I'm not sure what is the best way to handle the case when #ifdef TOOLKIT_VIEWS
evaluates to false and not duplicate too much code.

But on Windows, it works.

[Matt Perry, 2009-07-09 21:38:15]

http://codereview.chromium.org/149416/diff/1/2
File chrome/browser/extensions/extension_host.cc (right):

http://codereview.chromium.org/149416/diff/1/2#newcode141
Line 141: render_view_host_->CreateRenderView();
I think you want to call CreateRenderView(NULL) here (so it reloads the URL).

http://codereview.chromium.org/149416/diff/1/2#newcode143
Line 143: #endif
go ahead and add an #else CreateRenderView(NULL).  It's not much duplicate code,
and it's less ugly than any alternative I can think of.

[Paweł Hajdan Jr., 2009-07-09 22:02:46]

Done.

[Matt Perry, 2009-07-09 22:06:12]

LGTM