Fix BrowserTest.ThirtyFourTabs and possibly other ui tests in single-process ...

Fix BrowserTest.ThirtyFourTabs and possibly other ui tests in single-process mode.  The problem was that RenderProcess was being destructed on both threads at the same time because of the scoped_ptr.

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

[Matt Perry, 2009-02-26 20:34:30]

LGTM

http://codereview.chromium.org/27228/diff/1/2
File chrome/browser/renderer_host/browser_render_process_host.cc (right):

http://codereview.chromium.org/27228/diff/1/2#newcode74
Line 74: class RendererMainThread : public base::Thread {
please comment here explaining this is only used in single-process mode.

[jam, 2009-02-26 20:36:00]

http://codereview.chromium.org/27228/diff/1/2
File chrome/browser/renderer_host/browser_render_process_host.cc (right):

http://codereview.chromium.org/27228/diff/1/2#newcode74
Line 74: class RendererMainThread : public base::Thread {
On 2009/02/26 20:34:30, Matt Perry wrote:
> please comment here explaining this is only used in single-process mode.

Done.

[kuchhal, 2009-02-26 21:02:45]

http://codereview.chromium.org/27228/diff/1003/6
File chrome/browser/browser_uitest.cc (right):

http://codereview.chromium.org/27228/diff/1003/6#newcode93
Line 93: TEST_F(BrowserTest, ThirtyFourTabs) {
You can remove the comment above this line too.