Commit cc3af8b9 authored by zqzhang's avatar zqzhang Committed by Commit bot

[Media, UI] Don't record click UMA when content intent is null

BUG=607397

Review-Url: https://codereview.chromium.org/1927863003
Cr-Commit-Position: refs/heads/master@{#390351}
parent adf651b1
......@@ -142,8 +142,10 @@ public class CastSessionImpl implements MediaNotificationListener, CastSession {
}
Intent contentIntent = Tab.createBringTabToFrontIntent(tabId);
contentIntent.putExtra(MediaNotificationUma.INTENT_EXTRA_NAME,
MediaNotificationUma.SOURCE_PRESENTATION);
if (contentIntent != null) {
contentIntent.putExtra(MediaNotificationUma.INTENT_EXTRA_NAME,
MediaNotificationUma.SOURCE_PRESENTATION);
}
mNotificationBuilder = new MediaNotificationInfo.Builder()
.setPaused(false)
.setOrigin(origin)
......
......@@ -114,8 +114,10 @@ public class MediaSessionTabHelper {
}
Intent contentIntent = Tab.createBringTabToFrontIntent(mTab.getId());
contentIntent.putExtra(MediaNotificationUma.INTENT_EXTRA_NAME,
MediaNotificationUma.SOURCE_MEDIA);
if (contentIntent != null) {
contentIntent.putExtra(MediaNotificationUma.INTENT_EXTRA_NAME,
MediaNotificationUma.SOURCE_MEDIA);
}
mNotificationInfoBuilder =
new MediaNotificationInfo.Builder()
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment