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

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

Issue 2831223002: [Media>UI] Migrate media notification tests to JUnit tests (Closed)
Patch Set: fixed tests 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
« no previous file with comments | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaSessionTabHelper.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java b/chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java
index 5ac335281983bae2b59daa07c1327161fc431aa1..815761f11efc5f6a21b487996a14f94597b8335c 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java
@@ -588,24 +588,6 @@ public class MediaNotificationManager {
sManagers.put(notificationId, manager);
}
- @VisibleForTesting
- @Nullable
- static ChromeNotificationBuilder getNotificationBuilderForTesting(int notificationId) {
- MediaNotificationManager manager = getManager(notificationId);
- if (manager == null) return null;
-
- return manager.mNotificationBuilder;
- }
-
- @VisibleForTesting
- @Nullable
- static MediaNotificationInfo getMediaNotificationInfoForTesting(
- int notificationId) {
- MediaNotificationManager manager = getManager(notificationId);
-
- return (manager == null) ? null : manager.mMediaNotificationInfo;
- }
-
private static boolean isRunningN() {
return (sOverrideIsRunningNForTesting != null)
? sOverrideIsRunningNForTesting
« no previous file with comments | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaSessionTabHelper.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698