Record one more reason that download is canceled

Record one more reason that download is canceled

Download was cancelled when a dangerous download infobar is dismissed.
We missed it in the previous CL.

BUG=618864
Committed: https://crrev.com/513a888b2e3fb9b99fb623d32d22b6059c93951f
Cr-Commit-Position: refs/heads/master@{#403274}

[qinmin, 2016-06-30 17:23:03]

qinmin@chromium.org changed reviewers:
+ isherman@chromium.org, tedchoc@chromium.org

[qinmin, 2016-06-30 17:23:04]

PTAL

[Ilya Sherman, 2016-06-30 17:26:54]

histograms.xml lgtm

[Ilya Sherman, 2016-06-30 17:27:35]

Actually, LGTM with one nit:

https://codereview.chromium.org/2111983002/diff/1/tools/metrics/histograms/hi...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/2111983002/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:82372: +  <int value="5"
label="Dangerous infobar dismissed"/>
nit: Maybe phrase this as "Dangerous download infobar dismissed"?  Otherwise, it
sounds like the infobar itself is dangerous ;-)

[Ted C, 2016-06-30 17:55:21]

lgtm

[qinmin, 2016-06-30 19:37:49]

https://codereview.chromium.org/2111983002/diff/1/tools/metrics/histograms/hi...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/2111983002/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:82372: +  <int value="5"
label="Dangerous infobar dismissed"/>
On 2016/06/30 17:27:35, Ilya Sherman wrote:
> nit: Maybe phrase this as "Dangerous download infobar dismissed"?  Otherwise,
it
> sounds like the infobar itself is dangerous ;-)

Done.

[qinmin, 2016-06-30 19:37:55]

The CQ bit was checked by qinmin@chromium.org

[qinmin, 2016-06-30 19:37:55]

The patchset sent to the CQ was uploaded after l-g-t-m from
tedchoc@chromium.org, isherman@chromium.org
Link to the patchset: https://codereview.chromium.org/2111983002/#ps20001
(title: "nits")

[commit-bot: I haz the power, 2016-06-30 19:38: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-06-30 20:43:54]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-30 20:44:11]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-06-30 20:45:07]

Description was changed from

==========
Record one more reason that download is canceled

Download was cancelled when a dangerous download infobar is dismissed.
We missed it in the previous CL.

BUG=618864
==========

to

==========
Record one more reason that download is canceled

Download was cancelled when a dangerous download infobar is dismissed.
We missed it in the previous CL.

BUG=618864

Committed: https://crrev.com/513a888b2e3fb9b99fb623d32d22b6059c93951f
Cr-Commit-Position: refs/heads/master@{#403274}
==========

[commit-bot: I haz the power, 2016-06-30 20:45:09]

Patchset 2 (id:??) landed as
https://crrev.com/513a888b2e3fb9b99fb623d32d22b6059c93951f
Cr-Commit-Position: refs/heads/master@{#403274}