Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(495)

Unified Diff: chrome/android/junit/src/org/chromium/chrome/browser/media/ui/MediaNotificationManagerTestBase.java

Issue 2837283003: [Android] Log notification shown/blocked for all other notifications (Closed)
Patch Set: Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/junit/src/org/chromium/chrome/browser/media/ui/MediaNotificationManagerTestBase.java
diff --git a/chrome/android/junit/src/org/chromium/chrome/browser/media/ui/MediaNotificationManagerTestBase.java b/chrome/android/junit/src/org/chromium/chrome/browser/media/ui/MediaNotificationManagerTestBase.java
index df9dbeea9b29ff397720aa859aa7e81bd01be821..d7a49ee1e8181dfe7016b1751bf61b2f30456f0b 100644
--- a/chrome/android/junit/src/org/chromium/chrome/browser/media/ui/MediaNotificationManagerTestBase.java
+++ b/chrome/android/junit/src/org/chromium/chrome/browser/media/ui/MediaNotificationManagerTestBase.java
@@ -35,6 +35,7 @@ import org.chromium.base.ContextUtils;
import org.chromium.chrome.browser.AppHooks;
import org.chromium.chrome.browser.AppHooksImpl;
import org.chromium.chrome.browser.media.ui.MediaNotificationManager.ListenerService;
+import org.chromium.chrome.browser.notifications.NotificationUmaTracker;
import org.chromium.content_public.common.MediaMetadata;
/**
@@ -47,12 +48,13 @@ public class MediaNotificationManagerTestBase {
MockListenerService mService;
MediaNotificationListener mListener;
AppHooksImpl mMockAppHooks;
+ NotificationUmaTracker mMockUmaTracker;
MediaNotificationInfo.Builder mMediaNotificationInfoBuilder;
static class MockMediaNotificationManager extends MediaNotificationManager {
- public MockMediaNotificationManager() {
- super(NOTIFICATION_ID);
+ public MockMediaNotificationManager(NotificationUmaTracker umaTracker) {
+ super(umaTracker, NOTIFICATION_ID);
}
}
@@ -91,8 +93,9 @@ public class MediaNotificationManagerTestBase {
new MediaNotificationManager.NotificationOptions(MockListenerService.class,
MockMediaButtonReceiver.class, NOTIFICATION_GROUP_NAME));
- MediaNotificationManager.setManagerForTesting(
- NOTIFICATION_ID, spy(new MockMediaNotificationManager()));
+ mMockUmaTracker = mock(NotificationUmaTracker.class);
+ MediaNotificationManager.setManagerForTesting(NOTIFICATION_ID,
+ spy(new MockMediaNotificationManager(mMockUmaTracker)));
mMediaNotificationInfoBuilder =
new MediaNotificationInfo.Builder()
@@ -146,6 +149,7 @@ public class MediaNotificationManagerTestBase {
clearInvocations(getManager());
clearInvocations(mService);
clearInvocations(mMockContext);
+ clearInvocations(mMockUmaTracker);
}
void setUpService() {

Powered by Google App Engine
This is Rietveld 408576698