[Extensions Histograms] Update ExtensionBubbleAction enum

[Extensions Histograms] Update ExtensionBubbleAction enum

We added a new value for extension bubble actions for dismissing by
deactivation in crrev.com/486b20e71bb481cf6ae6491a998aa252081a8833 in
November, but never updated the histogram entry. As a result, the data,
while recording correctly, shows "3" for dismissing by deactivation.
Update the histogram file to correct this.

BUG=None

Committed: https://crrev.com/c1144734513b4607d9b21ef21040a5ebc1bb1381
Cr-Commit-Position: refs/heads/master@{#407567}

[Devlin, 2016-07-25 16:21:30]

rdevlin.cronin@chromium.org changed reviewers:
+ isherman@chromium.org

[Devlin, 2016-07-25 16:21:31]

Ilya, mind taking a look?

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

https://codereview.chromium.org/2174403002/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:73077: +  <int value="2" label="Dismiss
by user action"/>
These are just labels, and changing the text won't have an impact, correct?

[Ilya Sherman, 2016-07-25 18:43:50]

LGTM

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

https://codereview.chromium.org/2174403002/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:73077: +  <int value="2" label="Dismiss
by user action"/>
On 2016/07/25 16:21:31, Devlin wrote:
> These are just labels, and changing the text won't have an impact, correct?

Correct, as long as "Dismiss" previously meant "Dismiss by user action", rather
than via either mechanism.

[Devlin, 2016-07-25 18:45:42]

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

https://codereview.chromium.org/2174403002/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:73077: +  <int value="2" label="Dismiss
by user action"/>
On 2016/07/25 18:43:50, Ilya Sherman wrote:
> On 2016/07/25 16:21:31, Devlin wrote:
> > These are just labels, and changing the text won't have an impact, correct?
> 
> Correct, as long as "Dismiss" previously meant "Dismiss by user action",
rather
> than via either mechanism.

It did.  We were recording all values; "3" just didn't have a label.

[Devlin, 2016-07-25 18:45:49]

The CQ bit was checked by rdevlin.cronin@chromium.org

[commit-bot: I haz the power, 2016-07-25 18:46:30]

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-07-25 20:40:28]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-25 20:42:32]

Description was changed from

==========
[Extensions Histograms] Update ExtensionBubbleAction enum

We added a new value for extension bubble actions for dismissing by
deactivation in crrev.com/486b20e71bb481cf6ae6491a998aa252081a8833 in
November, but never updated the histogram entry. As a result, the data,
while recording correctly, shows "3" for dismissing by deactivation.
Update the histogram file to correct this.

BUG=None
==========

to

==========
[Extensions Histograms] Update ExtensionBubbleAction enum

We added a new value for extension bubble actions for dismissing by
deactivation in crrev.com/486b20e71bb481cf6ae6491a998aa252081a8833 in
November, but never updated the histogram entry. As a result, the data,
while recording correctly, shows "3" for dismissing by deactivation.
Update the histogram file to correct this.

BUG=None

Committed: https://crrev.com/c1144734513b4607d9b21ef21040a5ebc1bb1381
Cr-Commit-Position: refs/heads/master@{#407567}
==========

[commit-bot: I haz the power, 2016-07-25 20:42:33]

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