Add a simple UMA for counting web intents dispatches.

Add a simple UMA for counting web intents dispatches.

R=jhawkins@chromium.org
BUG=111978
TEST=None

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

[James Hawkins, 2012-05-02 20:03:00]

https://chromiumcodereview.appspot.com/10347002/diff/1/chrome/browser/ui/brow...
File chrome/browser/ui/browser.cc (right):

https://chromiumcodereview.appspot.com/10347002/diff/1/chrome/browser/ui/brow...
chrome/browser/ui/browser.cc:4004: UMA_HISTOGRAM_COUNTS("WebIntents.Dispatch",
1);
Are you familiar with all the other requirements you need to fulfill to add a
histogram?

[Greg Billock, 2012-05-02 20:11:54]

https://chromiumcodereview.appspot.com/10347002/diff/1/chrome/browser/ui/brow...
File chrome/browser/ui/browser.cc (right):

https://chromiumcodereview.appspot.com/10347002/diff/1/chrome/browser/ui/brow...
chrome/browser/ui/browser.cc:4004: UMA_HISTOGRAM_COUNTS("WebIntents.Dispatch",
1);
No. I looked for details, but didn't find them, so I figured this way I'd learn.
:-)

On 2012/05/02 20:03:00, James Hawkins wrote:
> Are you familiar with all the other requirements you need to fulfill to add a
> histogram?

[Greg Billock, 2012-05-07 18:50:32]

On 2012/05/02 20:11:54, Greg Billock wrote:
>
https://chromiumcodereview.appspot.com/10347002/diff/1/chrome/browser/ui/brow...
> File chrome/browser/ui/browser.cc (right):
> 
>
https://chromiumcodereview.appspot.com/10347002/diff/1/chrome/browser/ui/brow...
> chrome/browser/ui/browser.cc:4004: UMA_HISTOGRAM_COUNTS("WebIntents.Dispatch",
> 1);
> No. I looked for details, but didn't find them, so I figured this way I'd
learn.
> :-)
> 
> On 2012/05/02 20:03:00, James Hawkins wrote:
> > Are you familiar with all the other requirements you need to fulfill to add
a
> > histogram?

Server-side change submitted.

[James Hawkins, 2012-05-07 20:23:56]

On 2012/05/07 18:50:32, Greg Billock wrote:
> On 2012/05/02 20:11:54, Greg Billock wrote:
> >
>
https://chromiumcodereview.appspot.com/10347002/diff/1/chrome/browser/ui/brow...
> > File chrome/browser/ui/browser.cc (right):
> > 
> >
>
https://chromiumcodereview.appspot.com/10347002/diff/1/chrome/browser/ui/brow...
> > chrome/browser/ui/browser.cc:4004:
UMA_HISTOGRAM_COUNTS("WebIntents.Dispatch",
> > 1);
> > No. I looked for details, but didn't find them, so I figured this way I'd
> learn.
> > :-)
> > 
> > On 2012/05/02 20:03:00, James Hawkins wrote:
> > > Are you familiar with all the other requirements you need to fulfill to
add
> a
> > > histogram?
> 
> Server-side change submitted.

Link?

[Greg Billock, 2012-05-08 02:29:53]

On 2012/05/07 20:23:56, James Hawkins wrote:
> On 2012/05/07 18:50:32, Greg Billock wrote:
> > On 2012/05/02 20:11:54, Greg Billock wrote:
> > >
> >
>
https://chromiumcodereview.appspot.com/10347002/diff/1/chrome/browser/ui/brow...
> > > File chrome/browser/ui/browser.cc (right):
> > > 
> > >
> >
>
https://chromiumcodereview.appspot.com/10347002/diff/1/chrome/browser/ui/brow...
> > > chrome/browser/ui/browser.cc:4004:
> UMA_HISTOGRAM_COUNTS("WebIntents.Dispatch",
> > > 1);
> > > No. I looked for details, but didn't find them, so I figured this way I'd
> > learn.
> > > :-)
> > > 
> > > On 2012/05/02 20:03:00, James Hawkins wrote:
> > > > Are you familiar with all the other requirements you need to fulfill to
> add
> > a
> > > > histogram?
> > 
> > Server-side change submitted.
> 
> Link?

https://chromereviews.googleplex.com/4453013/

[James Hawkins, 2012-05-10 00:51:26]

I'm not sure this is valuable data.  What do we have to compare it against?

[Greg Billock, 2012-05-14 15:07:27]

On 2012/05/10 00:51:26, James Hawkins wrote:
> I'm not sure this is valuable data.  What do we have to compare it against?

It's a volumetric measure, like "hits on google.com" or "number of Chrome
downloads." Obviously this isn't the complete set of UMAs we need, but it's
always nice to have volumetric scales going back to the beginning of time.

[James Hawkins, 2012-05-14 17:26:49]

On 2012/05/14 15:07:27, Greg Billock wrote:
> On 2012/05/10 00:51:26, James Hawkins wrote:
> > I'm not sure this is valuable data.  What do we have to compare it against?
> 
> It's a volumetric measure, like "hits on google.com" or "number of Chrome
> downloads." Obviously this isn't the complete set of UMAs we need, but it's
> always nice to have volumetric scales going back to the beginning of time.

OK.  I'm still suspicious we'll get any value from it, but at least we can point
to it as evidence of increased usage (though we won't have any data to say
why/how usage increased).

LGTM

[Greg Billock, 2012-05-15 20:11:35]

On 2012/05/14 17:26:49, James Hawkins wrote:
> On 2012/05/14 15:07:27, Greg Billock wrote:
> > On 2012/05/10 00:51:26, James Hawkins wrote:
> > > I'm not sure this is valuable data.  What do we have to compare it
against?
> > 
> > It's a volumetric measure, like "hits on google.com" or "number of Chrome
> > downloads." Obviously this isn't the complete set of UMAs we need, but it's
> > always nice to have volumetric scales going back to the beginning of time.
> 
> OK.  I'm still suspicious we'll get any value from it, but at least we can
point
> to it as evidence of increased usage (though we won't have any data to say
> why/how usage increased).

Yeah, this is definitely just the beginning of the UMAs we'll need. I want to
get a basic metric for M20, though, so we can compare aggregate usage as more
use cases are enabled.

Peter, can I get an owners LGTM?

> LGTM

[Peter Kasting, 2012-05-21 19:05:53]

Rubber-stamp LGTM

[commit-bot: I haz the power, 2012-05-21 19:07:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gbillock@chromium.org/10347002/14001

[commit-bot: I haz the power, 2012-05-21 23:06:42]

Try job failure for 10347002-14001 (retry) (retry) on win_rel for step "compile"
(clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2012-05-21 23:14:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gbillock@chromium.org/10347002/14001

[commit-bot: I haz the power, 2012-05-22 01:40:35]

Try job failure for 10347002-14001 (retry) on win_rel for step "compile"
(clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2012-05-22 14:04:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gbillock@chromium.org/10347002/14001

[commit-bot: I haz the power, 2012-05-22 17:42:58]

Try job failure for 10347002-14001 (retry) (retry) on win_rel for step "compile"
(clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...