Fix output string of pepper_hash_for_uma.

Fix output string of pepper_hash_for_uma.

So that the string can be directly copied into histograms.xml.

BUG=111542
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=251255

[xhwang, 2014-02-13 18:43:19]

PTAL

[teravest, 2014-02-13 18:44:22]

lgtm

Thanks!

On Thu, Feb 13, 2014 at 11:43 AM,  <xhwang@chromium.org> wrote:
> Reviewers: teravest,
>
> Message:
> PTAL
>
> Description:
> Fix output string of pepper_hash_for_uma.
>
> So that the string can be directly copied into histograms.xml.
>
> BUG=111542
>
> Please review this at https://codereview.chromium.org/164253002/
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/src
>
> Affected files (+1, -1 lines):
>   M ppapi/tools/pepper_hash_for_uma.cc
>
>
> Index: ppapi/tools/pepper_hash_for_uma.cc
> diff --git a/ppapi/tools/pepper_hash_for_uma.cc
> b/ppapi/tools/pepper_hash_for_uma.cc
> index
>
758d5ba3056b5033de93464d7dc8d2ff8d2a552d..6e38cfdd4b6483c7fe879cc50e17f4dd401a3037
> 100644
> --- a/ppapi/tools/pepper_hash_for_uma.cc
> +++ b/ppapi/tools/pepper_hash_for_uma.cc
> @@ -31,7 +31,7 @@ int main(int argc, char **argv) {
>      // Strip off the signed bit because UMA doesn't support negative
> values,
>      // but takes a signed int as input.
>      int hash = static_cast<int>(data & 0x7fffffff);
> -    printf("<int value=\"%d\" label=\"%s\">\n", hash, argv[i]);
> +    printf("<int value=\"%d\" label=\"%s\"/>\n", hash, argv[i]);
>    }
>    return 0;
>  }
>
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[xhwang, 2014-02-13 18:47:27]

The CQ bit was checked by xhwang@chromium.org

[commit-bot: I haz the power, 2014-02-13 18:49:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/xhwang@chromium.org/164253002/1

[commit-bot: I haz the power, 2014-02-13 20:00:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/xhwang@chromium.org/164253002/1

[commit-bot: I haz the power, 2014-02-13 23:45:15]

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

[commit-bot: I haz the power, 2014-02-13 23:45:16]

Retried try job too often on linux_chromeos for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...

[xhwang, 2014-02-14 01:45:26]

The CQ bit was checked by xhwang@chromium.org

[commit-bot: I haz the power, 2014-02-14 01:46:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/xhwang@chromium.org/164253002/1

[commit-bot: I haz the power, 2014-02-14 04:16:52]

Change committed as 251255