Use individual functions to join browser threads.

Use individual functions to join browser threads.

This enables figuring out which thread is blocking shutdown just by looking
at the UI thread stack.

BUG=403610
R=sky@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=290015

[sky, 2014-08-15 04:48:01]

LGTM

https://codereview.chromium.org/478563002/diff/1/content/browser/browser_main...
File content/browser/browser_main_loop.cc (right):

https://codereview.chromium.org/478563002/diff/1/content/browser/browser_main...
content/browser/browser_main_loop.cc:849: TRACE_EVENT0("shutdown",
"BrowserMainLoop::Subsystem:DBThread");
nit: spacing is off in all these case statements, should be 2 space indented (as
line 888).

[rvargas (doing something else), 2014-08-15 18:01:26]

Thanks!

https://codereview.chromium.org/478563002/diff/1/content/browser/browser_main...
File content/browser/browser_main_loop.cc (right):

https://codereview.chromium.org/478563002/diff/1/content/browser/browser_main...
content/browser/browser_main_loop.cc:849: TRACE_EVENT0("shutdown",
"BrowserMainLoop::Subsystem:DBThread");
On 2014/08/15 04:48:01, sky wrote:
> nit: spacing is off in all these case statements, should be 2 space indented
(as
> line 888).

Done.

[rvargas (doing something else), 2014-08-15 18:01:32]

The CQ bit was checked by rvargas@chromium.org

[commit-bot: I haz the power, 2014-08-15 18:03:34]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rvargas@chromium.org/478563002/40001

[commit-bot: I haz the power, 2014-08-15 21:29:20]

Committed patchset #3 (40001) as 290015