Remove duplicated UMA recording when destructing SafeBrowsingErrorUI.

Remove duplicated UMA recording when destructing SafeBrowsingErrorUI.
Unlike SSL interstitial, when destructing safe browsing interstitial,
SafeBrowsingBlockingPage::OnDontProceed() function gets called. This
function takes cares of recording the DONT_PROCEED action. Therefore,
no need to recording DONT_PROCEED in SafeBrowsingErrorUI's destructor.

BUG=675786
Committed: https://crrev.com/19f322ce5c257661c75c178315db05f342dec57c
Cr-Commit-Position: refs/heads/master@{#439672}

[Jialiu Lin, 2016-12-20 00:39:47]

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

[commit-bot: I haz the power, 2016-12-20 00:40:14]

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

[Jialiu Lin, 2016-12-20 00:43:02]

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

[Jialiu Lin, 2016-12-20 00:43:03]

Hi nparker@,
This CL is for fixing a regression on
interstitial.malware/phishing/harmful.decision UMA metrics. This regression is
currently impacting CANARY only.

[Nathan Parker, 2016-12-20 01:05:42]

lgtm

[Jialiu Lin, 2016-12-20 01:51:45]

The CQ bit was unchecked by jialiul@chromium.org

[Jialiu Lin, 2016-12-20 01:51:48]

The CQ bit was checked by jialiul@chromium.org

[commit-bot: I haz the power, 2016-12-20 01:52:06]

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-20 02:22:29]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482198708280970, "parent_rev":
"34225ff86883ff0729f06154b48f7e44e1967f8a", "commit_rev":
"e0558bbf5725c3b50bff7c7513964d53b5a119e7"}

[commit-bot: I haz the power, 2016-12-20 02:22:57]

Description was changed from

==========
Remove duplicated UMA recording when destructing SafeBrowsingErrorUI.
Unlike SSL interstitial, when destructing safe browsing interstitial,
SafeBrowsingBlockingPage::OnDontProceed() function gets called. This
function takes cares of recording the DONT_PROCEED action. Therefore,
no need to recording DONT_PROCEED in SafeBrowsingErrorUI's destructor.

BUG=675786
==========

to

==========
Remove duplicated UMA recording when destructing SafeBrowsingErrorUI.
Unlike SSL interstitial, when destructing safe browsing interstitial,
SafeBrowsingBlockingPage::OnDontProceed() function gets called. This
function takes cares of recording the DONT_PROCEED action. Therefore,
no need to recording DONT_PROCEED in SafeBrowsingErrorUI's destructor.

BUG=675786

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

[commit-bot: I haz the power, 2016-12-20 02:22:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-20 02:25:48]

Description was changed from

==========
Remove duplicated UMA recording when destructing SafeBrowsingErrorUI.
Unlike SSL interstitial, when destructing safe browsing interstitial,
SafeBrowsingBlockingPage::OnDontProceed() function gets called. This
function takes cares of recording the DONT_PROCEED action. Therefore,
no need to recording DONT_PROCEED in SafeBrowsingErrorUI's destructor.

BUG=675786

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

to

==========
Remove duplicated UMA recording when destructing SafeBrowsingErrorUI.
Unlike SSL interstitial, when destructing safe browsing interstitial,
SafeBrowsingBlockingPage::OnDontProceed() function gets called. This
function takes cares of recording the DONT_PROCEED action. Therefore,
no need to recording DONT_PROCEED in SafeBrowsingErrorUI's destructor.

BUG=675786

Committed: https://crrev.com/19f322ce5c257661c75c178315db05f342dec57c
Cr-Commit-Position: refs/heads/master@{#439672}
==========

[commit-bot: I haz the power, 2016-12-20 02:25:49]

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