Fix crash during CertificateReportingService shutdown

Fix crash during CertificateReportingService shutdown

CertificateReportingService::ShutDown posts a task to the IO thread to
delete reporter_. If the service's destructor is already run, this will
cause a crash during shutdown. Instead, delete reporter_ using a static
method and without referencing CertificateReportingService's |this|.

BUG=677329
Committed: https://crrev.com/9fd701ecdc48f80651ff8a6824f55875730ea1b3
Cr-Commit-Position: refs/heads/master@{#440933}

[meacer, 2016-12-29 00:35:07]

The CQ bit was checked by meacer@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-29 00:35:19]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[meacer, 2016-12-29 00:47:53]

meacer@chromium.org changed reviewers:
+ jialiul@chromium.org

[meacer, 2016-12-29 00:47:54]

jialiul: Can you please take a look? Thanks!

[commit-bot: I haz the power, 2016-12-29 01:18:02]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-29 01:18:02]

Dry run: This issue passed the CQ dry run.

[Jialiu Lin, 2016-12-29 01:57:20]

lgtm

[meacer, 2016-12-29 03:08:26]

The CQ bit was checked by meacer@chromium.org

[commit-bot: I haz the power, 2016-12-29 03:08:33]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-29 03:11:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482980906578920, "parent_rev":
"d661247bc862f9380559fd37d93920800058edb0", "commit_rev":
"14bfd9a999b9559ea18a81cf5e2bfafdf4ddaa49"}

[commit-bot: I haz the power, 2016-12-29 03:12:24]

Description was changed from

==========
Fix crash during CertificateReportingService shutdown

CertificateReportingService::ShutDown posts a task to the IO thread to
delete reporter_. If the service's destructor is already run, this will
cause a crash during shutdown. Instead, delete reporter_ using a static
method and without referencing CertificateReportingService's |this|.

BUG=677329
==========

to

==========
Fix crash during CertificateReportingService shutdown

CertificateReportingService::ShutDown posts a task to the IO thread to
delete reporter_. If the service's destructor is already run, this will
cause a crash during shutdown. Instead, delete reporter_ using a static
method and without referencing CertificateReportingService's |this|.

BUG=677329

Review-Url: https://codereview.chromium.org/2601203002
==========

[commit-bot: I haz the power, 2016-12-29 03:12:25]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2017-01-02 15:51:38]

Description was changed from

==========
Fix crash during CertificateReportingService shutdown

CertificateReportingService::ShutDown posts a task to the IO thread to
delete reporter_. If the service's destructor is already run, this will
cause a crash during shutdown. Instead, delete reporter_ using a static
method and without referencing CertificateReportingService's |this|.

BUG=677329

Review-Url: https://codereview.chromium.org/2601203002
==========

to

==========
Fix crash during CertificateReportingService shutdown

CertificateReportingService::ShutDown posts a task to the IO thread to
delete reporter_. If the service's destructor is already run, this will
cause a crash during shutdown. Instead, delete reporter_ using a static
method and without referencing CertificateReportingService's |this|.

BUG=677329

Committed: https://crrev.com/9fd701ecdc48f80651ff8a6824f55875730ea1b3
Cr-Commit-Position: refs/heads/master@{#440933}
==========

[commit-bot: I haz the power, 2017-01-02 15:51:39]

Patchset 1 (id:??) landed as
https://crrev.com/9fd701ecdc48f80651ff8a6824f55875730ea1b3
Cr-Commit-Position: refs/heads/master@{#440933}