Add UMA metrics for blocked plugins.

Add UMA metrics for blocked plugins.

BUG=92170
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=96360

[Bernhard Bauer, 2011-08-10 14:53:52]

please review.

[jochen (gone - plz use gerrit), 2011-08-10 15:12:03]

http://codereview.chromium.org/7558024/diff/2001/chrome/renderer/chrome_conte...
File chrome/renderer/chrome_content_renderer_client.cc (right):

http://codereview.chromium.org/7558024/diff/2001/chrome/renderer/chrome_conte...
chrome/renderer/chrome_content_renderer_client.cc:463: resource,
plugin_setting);
why not record the UMA here?

[Bernhard Bauer, 2011-08-10 15:16:45]

http://codereview.chromium.org/7558024/diff/2001/chrome/renderer/chrome_conte...
File chrome/renderer/chrome_content_renderer_client.cc (right):

http://codereview.chromium.org/7558024/diff/2001/chrome/renderer/chrome_conte...
chrome/renderer/chrome_content_renderer_client.cc:463: resource,
plugin_setting);
On 2011/08/10 15:12:03, jochen wrote:
> why not record the UMA here?

UserMetrics is in content/browser/ :-/

[jochen (gone - plz use gerrit), 2011-08-10 15:19:57]

On 2011/08/10 15:16:45, Bernhard Bauer wrote:
>
http://codereview.chromium.org/7558024/diff/2001/chrome/renderer/chrome_conte...
> File chrome/renderer/chrome_content_renderer_client.cc (right):
> 
>
http://codereview.chromium.org/7558024/diff/2001/chrome/renderer/chrome_conte...
> chrome/renderer/chrome_content_renderer_client.cc:463: resource,
> plugin_setting);
> On 2011/08/10 15:12:03, jochen wrote:
> > why not record the UMA here?
> 
> UserMetrics is in content/browser/ :-/

send a ViewHostMsg_UserMetricsRecordAction then?

[Bernhard Bauer, 2011-08-10 15:48:20]

On 2011/08/10 15:19:57, jochen wrote:
> On 2011/08/10 15:16:45, Bernhard Bauer wrote:
> >
>
http://codereview.chromium.org/7558024/diff/2001/chrome/renderer/chrome_conte...
> > File chrome/renderer/chrome_content_renderer_client.cc (right):
> > 
> >
>
http://codereview.chromium.org/7558024/diff/2001/chrome/renderer/chrome_conte...
> > chrome/renderer/chrome_content_renderer_client.cc:463: resource,
> > plugin_setting);
> > On 2011/08/10 15:12:03, jochen wrote:
> > > why not record the UMA here?
> > 
> > UserMetrics is in content/browser/ :-/
> 
> send a ViewHostMsg_UserMetricsRecordAction then?

Ooh, nice! Done.

[jochen (gone - plz use gerrit), 2011-08-10 15:49:21]

LGTM, thx

[Bernhard Bauer, 2011-08-10 16:27:00]

Forgot to update chromeactions.txt. Take another look?

[jochen (gone - plz use gerrit), 2011-08-10 17:04:22]

Still LGTM

[commit-bot: I haz the power, 2011-08-11 11:28:06]

Change committed as 96360