Move ImportantFile.TempFileFailures to histograms.xml.

Move ImportantFile.TempFileFailures to histograms.xml.

At that occasion, make it owned by rvargas@.

BUG=none

Committed: https://crrev.com/5dc312a3e602859f169ca1c7c26d12a8fd9107e2
Cr-Commit-Position: refs/heads/master@{#300320}

[Thiemo Nagel, 2014-10-20 07:16:28]

tnagel@chromium.org changed reviewers:
+ rvargas@chromium.org

[Thiemo Nagel, 2014-10-20 07:16:28]

tnagel@chromium.org changed required reviewers:
+ rvargas@chromium.org

[Thiemo Nagel, 2014-10-20 07:17:13]

tnagel@chromium.org changed reviewers:
+ asvitkine@chromium.org

[Thiemo Nagel, 2014-10-20 07:17:14]

tnagel@chromium.org changed required reviewers:
+ asvitkine@chromium.org

[Thiemo Nagel, 2014-10-20 07:17:14]

Hi Alexei,

could you please take a look?

Thank you!
Thiemo

[Alexei Svitkine (slow), 2014-10-20 15:25:35]

LGTM % comment (i.e. remove the new enum value from this CL and it's good go,
then send me another one that adds that enum value)

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

https://codereview.chromium.org/662273003/diff/1/tools/metrics/histograms/his...
tools/metrics/histograms/histograms.xml:53065: +  <int value="5"
label="Flushing"/>
Could you split this in two CLs? First CL to do the mechanical move and a
follow-up CL to introduce the new enum value?

[Thiemo Nagel, 2014-10-20 16:22:39]

Thank you, Alexei!

Ricardo, are you ok with the CL?

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

https://codereview.chromium.org/662273003/diff/1/tools/metrics/histograms/his...
tools/metrics/histograms/histograms.xml:53065: +  <int value="5"
label="Flushing"/>
On 2014/10/20 15:25:35, Alexei Svitkine wrote:
> Could you split this in two CLs? First CL to do the mechanical move and a
> follow-up CL to introduce the new enum value?

Done.

[rvargas (doing something else), 2014-10-20 18:24:36]

lgtm

[Thiemo Nagel, 2014-10-20 18:35:31]

The CQ bit was checked by tnagel@chromium.org

[commit-bot: I haz the power, 2014-10-20 18:37:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/662273003/20001

[commit-bot: I haz the power, 2014-10-20 20:18:47]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-20 20:19:28]

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