Make sure PluginDataRemoverImpl::Init is executed after the constructor has finished.

Make sure PluginDataRemoverImpl::Init is executed after the constructor has finished.

BUG=106709
TEST=none


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

[Bernhard Bauer, 2011-12-07 18:14:22]

Please review.

[jam, 2011-12-07 21:57:12]

lgtm

http://codereview.chromium.org/8848001/diff/1/content/browser/plugin_data_rem...
File content/browser/plugin_data_remover_impl.cc (right):

http://codereview.chromium.org/8848001/diff/1/content/browser/plugin_data_rem...
content/browser/plugin_data_remover_impl.cc:226: BrowserThread::PostTask(
nit: usually this is solved by having an Init function on the refcounted object,
and it can do the calls below, since they're an implementation detail of
Context. Here we would just call Context::Init() which does the code below.

[Bernhard Bauer, 2011-12-08 09:43:17]

http://codereview.chromium.org/8848001/diff/1/content/browser/plugin_data_rem...
File content/browser/plugin_data_remover_impl.cc (right):

http://codereview.chromium.org/8848001/diff/1/content/browser/plugin_data_rem...
content/browser/plugin_data_remover_impl.cc:226: BrowserThread::PostTask(
On 2011/12/07 21:57:12, John Abd-El-Malek wrote:
> nit: usually this is solved by having an Init function on the refcounted
object,
> and it can do the calls below, since they're an implementation detail of
> Context. Here we would just call Context::Init() which does the code below.

Done.

[commit-bot: I haz the power, 2011-12-08 09:43:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bauerb@chromium.org/8848001/5001

[commit-bot: I haz the power, 2011-12-08 11:14:35]

Change committed as 113593