Nuke SamplerRegistry

Nuke SamplerRegistry

The registry is a simple list of active Samplers but uses additional Mutex. Useful parts were merged into SamplerThread, others removed completely.

BUG=None

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

[yurys, 2013-04-16 14:50:21]

Signal handling logic was extracted into SignalHandler all-static class to
reduce number of #ifdef USE_SIGNALS in the code. Now I'm thinking about doing
signal installation in SignalHandler constructor and removing it in its
destructor, it should simplify the code.

[yurys, 2013-04-16 15:01:51]

On 2013/04/16 14:50:21, Yury Semikhatsky wrote:
> Signal handling logic was extracted into SignalHandler all-static class to
> reduce number of #ifdef USE_SIGNALS in the code. Now I'm thinking about doing
> signal installation in SignalHandler constructor and removing it in its
> destructor, it should simplify the code.

This wouldn't work as because of current RemoveActiveSampler implementation
(line 432) we can in theory have two instances of SamplerThread simultaneously
(one of them would be terminating though).

[loislo, 2013-04-16 15:52:25]

lgtm

[Sven Panne, 2013-04-17 07:04:24]

LGTM, too.

[yurys, 2013-04-17 07:20:34]

Committed patchset #1 manually as r14297.