[MediaNotification] Add UMA to record user clicking media notifications

[MediaNotification] Add UMA to record user clicking media notifications

This CL adds UMA to record how often the user clicks media notifications
to go back to Chrome.

BUG=605156
Committed: https://crrev.com/6126ee74cd5a98a55b6a1d3991fb0018ca425136
Cr-Commit-Position: refs/heads/master@{#389090}

[Zhiqiang Zhang (Slow), 2016-04-20 16:27:43]

Description was changed from

==========
[MediaNotification] Add UMA to record user clicking media notifications

This CL adds UMA to record how often the user clicks media notifications
to go back to Chrome.

BUG=599937
==========

to

==========
[MediaNotification] Add UMA to record user clicking media notifications

This CL adds UMA to record how often the user clicks media notifications
to go back to Chrome.

BUG=605156
==========

[Zhiqiang Zhang (Slow), 2016-04-20 17:20:32]

zqzhang@chromium.org changed reviewers:
+ avayvod@chromium.org, dfalcantara@chromium.org, mlamouri@chromium.org

[Zhiqiang Zhang (Slow), 2016-04-20 17:20:32]

PTAL
+dfalcantara to look at:
ChromeLauncherActivity.java
.../chrome/browser/metrics/*.java
java_sources.gni

+mlamouri/avayvod to look at:
.../chrome/browser/media/*

[gone, 2016-04-20 22:17:14]

https://chromiumcodereview.appspot.com/1909483003/diff/40001/chrome/android/j...
File
chrome/android/java/src/org/chromium/chrome/browser/media/remote/CastNotificationControl.java
(right):

https://chromiumcodereview.appspot.com/1909483003/diff/40001/chrome/android/j...
chrome/android/java/src/org/chromium/chrome/browser/media/remote/CastNotificationControl.java:103:
Intent contentIntent = new Intent(mContext, ExpandedControllerActivity.class)
Don't do this weird indentation if you're saving the Intent anyway.

Intent contentIntent = new Intent(mContext, ExpandedControllerActivity.class);
contentIntent.putExtra(LaunchFromNotificationUMA.INTENT_EXTRA_NAME, 
        LaunchFromNotificationUMA.SOURCE_MEDIA_FLING);

https://chromiumcodereview.appspot.com/1909483003/diff/40001/chrome/android/j...
File
chrome/android/java/src/org/chromium/chrome/browser/media/router/cast/CastSessionImpl.java
(right):

https://chromiumcodereview.appspot.com/1909483003/diff/40001/chrome/android/j...
chrome/android/java/src/org/chromium/chrome/browser/media/router/cast/CastSessionImpl.java:144:
new MediaNotificationInfo.Builder()
Undo this extra indentation.  The change doesn't help readability at all.

https://chromiumcodereview.appspot.com/1909483003/diff/40001/chrome/android/j...
chrome/android/java/src/org/chromium/chrome/browser/media/router/cast/CastSessionImpl.java:153:
.setContentIntent(Tab.createBringTabToFrontIntent(tabId).putExtra(
This is hard to read.  Temporarily store the Intent as another variable before
constructing the Builder instead of adding yet another level of indentation.

https://chromiumcodereview.appspot.com/1909483003/diff/40001/chrome/android/j...
File
chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java
(right):

https://chromiumcodereview.appspot.com/1909483003/diff/40001/chrome/android/j...
chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java:558:
// intent extras will stay the same for the same tab.
1) Don't stick random comments in between a function's arguments.  Put it above
the call.

2) Talk with aberent about this change.  IIRC there was another PendingIntent
related bug where an argument like this was set to 0.

https://chromiumcodereview.appspot.com/1909483003/diff/40001/chrome/android/j...
File
chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaSessionTabHelper.java
(right):

https://chromiumcodereview.appspot.com/1909483003/diff/40001/chrome/android/j...
chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaSessionTabHelper.java:127:
Tab.createBringTabToFrontIntent(mTab.getId())
Same comment about formatting as above.  This is indented a rather ridiculous 5
levels deep now.  Store the Intent before the builder is called and edit it
there.

https://chromiumcodereview.appspot.com/1909483003/diff/40001/chrome/android/j...
File
chrome/android/java/src/org/chromium/chrome/browser/metrics/LaunchFromNotificationUMA.java
(right):

https://chromiumcodereview.appspot.com/1909483003/diff/40001/chrome/android/j...
chrome/android/java/src/org/chromium/chrome/browser/metrics/LaunchFromNotificationUMA.java:12:
public class LaunchFromNotificationUMA {
LaunchFromNotificationUma, not UMA.  media/ uniquely breaks the naming
convention when it shouldn't.

https://chromiumcodereview.appspot.com/1909483003/diff/40001/chrome/android/j...
chrome/android/java/src/org/chromium/chrome/browser/metrics/LaunchFromNotificationUMA.java:19:
public static final String INTENT_EXTRA_NAME = "Launch.NotificationSource";
This doesn't have to match the histogram name.  All Intent extras should be
prefaced with the package name so that we don't get conflicts.

Use something like this instead:
public static final String EXTRA_NOTIFICATION_SOURCE =
"org.chromium.chrome.browser.metrics.LaunchFromNotificationUMA.NOTIFICATION_SOURCE";

https://chromiumcodereview.appspot.com/1909483003/diff/40001/chrome/android/j...
chrome/android/java/src/org/chromium/chrome/browser/metrics/LaunchFromNotificationUMA.java:21:
private static final LaunchMetrics
Formatting is super wacky here.  can you import
LaunchMetrics.EnumeratedHistogramSample instead?

https://chromiumcodereview.appspot.com/1909483003/diff/40001/chrome/android/j...
chrome/android/java/src/org/chromium/chrome/browser/metrics/LaunchFromNotificationUMA.java:33:
if (source == SOURCE_INVALID) return;
if (source <= SOURCE_INVALID || source >= SOURCE_MAX)

https://chromiumcodereview.appspot.com/1909483003/diff/40001/chrome/android/j...
File chrome/android/java_sources.gni (right):

https://chromiumcodereview.appspot.com/1909483003/diff/40001/chrome/android/j...
chrome/android/java_sources.gni:454:
"java/src/org/chromium/chrome/browser/metrics/LaunchFromNotificationUMA.java",
This class should be capitalized like "Uma" like all the other non-Media metrics
classes.

[Zhiqiang Zhang (Slow), 2016-04-21 10:36:47]

https://codereview.chromium.org/1909483003/diff/40001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java
(right):

https://codereview.chromium.org/1909483003/diff/40001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java:558:
// intent extras will stay the same for the same tab.
> 2) Talk with aberent about this change.  IIRC there was another PendingIntent
> related bug where an argument like this was set to 0.

+aberent, can you give me some details of the bug related to this PendingIntent
flag?

[Zhiqiang Zhang (Slow), 2016-04-21 10:37:53]

zqzhang@chromium.org changed reviewers:
+ aberent@chromium.org

[Zhiqiang Zhang (Slow), 2016-04-21 10:37:54]

https://codereview.chromium.org/1909483003/diff/40001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java
(right):

https://codereview.chromium.org/1909483003/diff/40001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java:558:
// intent extras will stay the same for the same tab.
> 2) Talk with aberent about this change.  IIRC there was another PendingIntent
> related bug where an argument like this was set to 0.

+aberent, can you give me some details of the bug related to this PendingIntent
flag?

[mlamouri (slow - plz ping), 2016-04-21 12:37:26]

Couple of comments see below. Otherwise, you might want to use `git cl format`
to prevent formatting issues ;)

https://codereview.chromium.org/1909483003/diff/40001/tools/metrics/histogram...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/1909483003/diff/40001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:18734: +<histogram
name="Launch.NotificationSource" enum="LaunchNotificationSource">
The name is slightly confusing: should it be more explicitly about Media? Like
Media.Notification.Click? and the source would be MediaNotificationSource?

https://codereview.chromium.org/1909483003/diff/40001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:18735: + 
<owner>mlamouri@chromium.org,avayvod@chromium.org</owner>
You should add your name here :)

[Zhiqiang Zhang (Slow), 2016-04-21 16:36:08]

On 2016/04/21 12:37:26, Mounir Lamouri wrote:
> Couple of comments see below. Otherwise, you might want to use `git cl format`
> to prevent formatting issues ;)
> 

That's what I did and it wasn't that smart :/

[Zhiqiang Zhang (Slow), 2016-04-21 16:36:51]

zqzhang@chromium.org changed reviewers:
- aberent@chromium.org

[Zhiqiang Zhang (Slow), 2016-04-21 16:36:53]

PTAL. Addressed reviewer's comments with replies.

https://codereview.chromium.org/1909483003/diff/40001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/media/remote/CastNotificationControl.java
(right):

https://codereview.chromium.org/1909483003/diff/40001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/media/remote/CastNotificationControl.java:103:
Intent contentIntent = new Intent(mContext, ExpandedControllerActivity.class)
On 2016/04/20 22:17:14, dfalcantara wrote:
> Don't do this weird indentation if you're saving the Intent anyway.
> 
> Intent contentIntent = new Intent(mContext, ExpandedControllerActivity.class);
> contentIntent.putExtra(LaunchFromNotificationUMA.INTENT_EXTRA_NAME, 
>         LaunchFromNotificationUMA.SOURCE_MEDIA_FLING);

Done.

https://codereview.chromium.org/1909483003/diff/40001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/media/router/cast/CastSessionImpl.java
(right):

https://codereview.chromium.org/1909483003/diff/40001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/media/router/cast/CastSessionImpl.java:144:
new MediaNotificationInfo.Builder()
On 2016/04/20 22:17:14, dfalcantara wrote:
> Undo this extra indentation.  The change doesn't help readability at all.

Done.

https://codereview.chromium.org/1909483003/diff/40001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/media/router/cast/CastSessionImpl.java:153:
.setContentIntent(Tab.createBringTabToFrontIntent(tabId).putExtra(
On 2016/04/20 22:17:14, dfalcantara wrote:
> This is hard to read.  Temporarily store the Intent as another variable before
> constructing the Builder instead of adding yet another level of indentation.

Done.

https://codereview.chromium.org/1909483003/diff/40001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java
(right):

https://codereview.chromium.org/1909483003/diff/40001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java:558:
// intent extras will stay the same for the same tab.
On 2016/04/20 22:17:14, dfalcantara wrote:
> 1) Don't stick random comments in between a function's arguments.  Put it
above
> the call.
> 
> 2) Talk with aberent about this change.  IIRC there was another PendingIntent
> related bug where an argument like this was set to 0.

I talked with aberent. We found the CL you mentioned:
https://codereview.chromium.org/1707783002/
However it's about setting the tabId in the pending intent, not the last flag.
So it's unrelated to my CL.

https://codereview.chromium.org/1909483003/diff/40001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaSessionTabHelper.java
(right):

https://codereview.chromium.org/1909483003/diff/40001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaSessionTabHelper.java:127:
Tab.createBringTabToFrontIntent(mTab.getId())
On 2016/04/20 22:17:14, dfalcantara wrote:
> Same comment about formatting as above.  This is indented a rather ridiculous
5
> levels deep now.  Store the Intent before the builder is called and edit it
> there.

Done.

https://codereview.chromium.org/1909483003/diff/40001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/metrics/LaunchFromNotificationUMA.java
(right):

https://codereview.chromium.org/1909483003/diff/40001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/metrics/LaunchFromNotificationUMA.java:12:
public class LaunchFromNotificationUMA {
On 2016/04/20 22:17:14, dfalcantara wrote:
> LaunchFromNotificationUma, not UMA.  media/ uniquely breaks the naming
> convention when it shouldn't.

Done.

https://codereview.chromium.org/1909483003/diff/40001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/metrics/LaunchFromNotificationUMA.java:19:
public static final String INTENT_EXTRA_NAME = "Launch.NotificationSource";
On 2016/04/20 22:17:14, dfalcantara wrote:
> This doesn't have to match the histogram name.  All Intent extras should be
> prefaced with the package name so that we don't get conflicts.
> 
> Use something like this instead:
> public static final String EXTRA_NOTIFICATION_SOURCE =
>
"org.chromium.chrome.browser.metrics.LaunchFromNotificationUMA.NOTIFICATION_SOURCE";

Done.

https://codereview.chromium.org/1909483003/diff/40001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/metrics/LaunchFromNotificationUMA.java:21:
private static final LaunchMetrics
On 2016/04/20 22:17:14, dfalcantara wrote:
> Formatting is super wacky here.  can you import
> LaunchMetrics.EnumeratedHistogramSample instead?

Done.

https://codereview.chromium.org/1909483003/diff/40001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/metrics/LaunchFromNotificationUMA.java:33:
if (source == SOURCE_INVALID) return;
On 2016/04/20 22:17:14, dfalcantara wrote:
> if (source <= SOURCE_INVALID || source >= SOURCE_MAX)

Done.

https://codereview.chromium.org/1909483003/diff/40001/chrome/android/java_sou...
File chrome/android/java_sources.gni (right):

https://codereview.chromium.org/1909483003/diff/40001/chrome/android/java_sou...
chrome/android/java_sources.gni:454:
"java/src/org/chromium/chrome/browser/metrics/LaunchFromNotificationUMA.java",
On 2016/04/20 22:17:14, dfalcantara wrote:
> This class should be capitalized like "Uma" like all the other non-Media
metrics
> classes.

Done.

https://codereview.chromium.org/1909483003/diff/40001/tools/metrics/histogram...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/1909483003/diff/40001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:18734: +<histogram
name="Launch.NotificationSource" enum="LaunchNotificationSource">
On 2016/04/21 12:37:26, Mounir Lamouri wrote:
> The name is slightly confusing: should it be more explicitly about Media? Like
> Media.Notification.Click? and the source would be MediaNotificationSource?

Done.

https://codereview.chromium.org/1909483003/diff/40001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:18735: + 
<owner>mlamouri@chromium.org,avayvod@chromium.org</owner>
On 2016/04/21 12:37:26, Mounir Lamouri wrote:
> You should add your name here :)

Done.

[Zhiqiang Zhang (Slow), 2016-04-21 16:36:54]

zqzhang@chromium.org changed reviewers:
- aberent@chromium.org

[Zhiqiang Zhang (Slow), 2016-04-21 16:36:54]

PTAL. Addressed reviewer's comments with replies.

[Zhiqiang Zhang (Slow), 2016-04-21 16:39:31]

zqzhang@chromium.org changed reviewers:
+ isherman@chromium.org

[Zhiqiang Zhang (Slow), 2016-04-21 16:39:31]

+isherman@ to look at histograms.xml

[gone, 2016-04-21 17:16:35]

Yeah, never use git cl format on Java files.  It just ruins everything.

lgtm

https://codereview.chromium.org/1909483003/diff/60001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/media/remote/CastNotificationControl.java
(right):

https://codereview.chromium.org/1909483003/diff/60001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/media/remote/CastNotificationControl.java:104:
.putExtra(MediaNotificationUma.INTENT_EXTRA_NAME,
You still did the wonky thing where you called putExtra on the same line as the
construction of the Intent.  Since you're stuck with two lines anyway, use this
if the putExtra line fits within 100 chars.

Intent contentIntent = new Intent(mContext, ExpandedControllerActivity.class);
contentIntent.putExtra(MediaNotificationUma.INTENT_EXTRA_NAME,
MediaNotificationUma.SOURCE_MEDIA_FLING);

https://codereview.chromium.org/1909483003/diff/60001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java
(right):

https://codereview.chromium.org/1909483003/diff/60001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java:555:
// intent extras will stay the same for the same tab.
Don't randomly place TODOs in the middle of block comments.  Move it to the end.

https://codereview.chromium.org/1909483003/diff/60001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/metrics/MediaNotificationUma.java
(right):

https://codereview.chromium.org/1909483003/diff/60001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/metrics/MediaNotificationUma.java:22:
public static final String INTENT_EXTRA_NAME =
You may eventually add other extras.  Differentiate now.

EXTRA_CLICK_SOURCE

https://codereview.chromium.org/1909483003/diff/60001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/metrics/MediaNotificationUma.java:26:
new LaunchMetrics.EnumeratedHistogramSample("Media.Notification.Click",
SOURCE_MAX);
Media.Notification.ClickSource?  Click alone sounds like the thing was clicked
on.

[Ilya Sherman, 2016-04-22 08:19:06]

histograms lgtm

[Zhiqiang Zhang (Slow), 2016-04-22 10:14:34]

Addressed dan's comments.
Need approval from avayvod/mlamouri.

https://codereview.chromium.org/1909483003/diff/60001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/media/remote/CastNotificationControl.java
(right):

https://codereview.chromium.org/1909483003/diff/60001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/media/remote/CastNotificationControl.java:104:
.putExtra(MediaNotificationUma.INTENT_EXTRA_NAME,
On 2016/04/21 17:16:35, dfalcantara wrote:
> You still did the wonky thing where you called putExtra on the same line as
the
> construction of the Intent.  Since you're stuck with two lines anyway, use
this
> if the putExtra line fits within 100 chars.
> 
> Intent contentIntent = new Intent(mContext, ExpandedControllerActivity.class);
> contentIntent.putExtra(MediaNotificationUma.INTENT_EXTRA_NAME,
> MediaNotificationUma.SOURCE_MEDIA_FLING);

Done.

https://codereview.chromium.org/1909483003/diff/60001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java
(right):

https://codereview.chromium.org/1909483003/diff/60001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java:555:
// intent extras will stay the same for the same tab.
On 2016/04/21 17:16:35, dfalcantara wrote:
> Don't randomly place TODOs in the middle of block comments.  Move it to the
end.

Done.

https://codereview.chromium.org/1909483003/diff/60001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/metrics/MediaNotificationUma.java
(right):

https://codereview.chromium.org/1909483003/diff/60001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/metrics/MediaNotificationUma.java:22:
public static final String INTENT_EXTRA_NAME =
On 2016/04/21 17:16:35, dfalcantara wrote:
> You may eventually add other extras.  Differentiate now.
> 
> EXTRA_CLICK_SOURCE

Done.

https://codereview.chromium.org/1909483003/diff/60001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/metrics/MediaNotificationUma.java:26:
new LaunchMetrics.EnumeratedHistogramSample("Media.Notification.Click",
SOURCE_MAX);
On 2016/04/21 17:16:35, dfalcantara wrote:
> Media.Notification.ClickSource?  Click alone sounds like the thing was clicked
> on.

Done.

[mlamouri (slow - plz ping), 2016-04-22 10:32:50]

lgtm

Sorry for the wrong tip with regards to `git cl format` :(

https://codereview.chromium.org/1909483003/diff/80001/tools/metrics/histogram...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/1909483003/diff/80001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:20974: +<histogram
name="Media.Notification.ClickSource"
"Media.Notification.Click" is probably better. We are recording the clicks and
it happens that we also record the sources.

https://codereview.chromium.org/1909483003/diff/80001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:20980: +    Records which kind of
notification the user clicked to go back to Chrome.
Same as above. Maybe something like:
"Records when the user click a media notification to go back to Chrome."

[Zhiqiang Zhang (Slow), 2016-04-22 12:41:47]

https://codereview.chromium.org/1909483003/diff/80001/tools/metrics/histogram...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/1909483003/diff/80001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:20974: +<histogram
name="Media.Notification.ClickSource"
On 2016/04/22 10:32:50, Mounir Lamouri wrote:
> "Media.Notification.Click" is probably better. We are recording the clicks and
> it happens that we also record the sources.

Done.

https://codereview.chromium.org/1909483003/diff/80001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:20980: +    Records which kind of
notification the user clicked to go back to Chrome.
On 2016/04/22 10:32:50, Mounir Lamouri wrote:
> Same as above. Maybe something like:
> "Records when the user click a media notification to go back to Chrome."

Rephrased to "The type of media notification the user clicked to go back to
Chrome"

[Zhiqiang Zhang (Slow), 2016-04-22 12:41:53]

The CQ bit was checked by zqzhang@chromium.org

[Zhiqiang Zhang (Slow), 2016-04-22 12:41:54]

The patchset sent to the CQ was uploaded after l-g-t-m from
dfalcantara@chromium.org, isherman@chromium.org, mlamouri@chromium.org
Link to the patchset: https://codereview.chromium.org/1909483003/#ps100001
(title: "renaming")

[commit-bot: I haz the power, 2016-04-22 12:42:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1909483003/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1909483003/100001

[commit-bot: I haz the power, 2016-04-22 13:51:04]

Description was changed from

==========
[MediaNotification] Add UMA to record user clicking media notifications

This CL adds UMA to record how often the user clicks media notifications
to go back to Chrome.

BUG=605156
==========

to

==========
[MediaNotification] Add UMA to record user clicking media notifications

This CL adds UMA to record how often the user clicks media notifications
to go back to Chrome.

BUG=605156
==========

[commit-bot: I haz the power, 2016-04-22 13:51:05]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-04-22 19:46:48]

Description was changed from

==========
[MediaNotification] Add UMA to record user clicking media notifications

This CL adds UMA to record how often the user clicks media notifications
to go back to Chrome.

BUG=605156
==========

to

==========
[MediaNotification] Add UMA to record user clicking media notifications

This CL adds UMA to record how often the user clicks media notifications
to go back to Chrome.

BUG=605156

Committed: https://crrev.com/6126ee74cd5a98a55b6a1d3991fb0018ca425136
Cr-Commit-Position: refs/heads/master@{#389090}
==========

[commit-bot: I haz the power, 2016-04-22 19:46:50]

Patchset 6 (id:??) landed as
https://crrev.com/6126ee74cd5a98a55b6a1d3991fb0018ca425136
Cr-Commit-Position: refs/heads/master@{#389090}