| 87eae9bf79a8f03ea57b79dea4d9af37ed0f7e51 |
|
20-Apr-2018 |
shawnlin <shawnlin@google.com> |
Fix the art work of lockscreen cannot be removed
The root cause is that mMediaNotificationKey is cleared right after it is set.
It makes the media art work cannot be cleared since there is no
notification that match the mMediaNotificationKey which is already
cleared to null.
Only clear media session related data inside the controller check.
Test: runtest systemui
Test: manual
Change-Id: Iae43c502b90f73e57fae7150e15ef89eb18b096c
Fixes: 78204513
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/NotificationMediaManager.java
|
| 77097800c47561e7de70f0143d715de78c702479 |
|
25-Mar-2018 |
shawnlin <shawnlin@google.com> |
Fix the background of lockscreen showing the art work of removed media notification
StatusBar inflates notification views by AsyncTask(AsyncInflationTask), and the notification entry will be added to mNotificationData only after the inflation is completed.
It makes a timing issue for media notification if following conditions are met:
1. There already exists one none-media notification which is posted by the same app as the incoming media notification.
2. Any notification posted and its inflation is completed before the media notification and then triggers the findAndUpdateMediaNotifications().
In such case, the media notification entry hasn't been added to mNotificationData yet so we can't find any notification with media controller in this list.
Then we look up MediaSessionManager and try to find any MediaController that has the same package name with any notification in mNotificationData list.
It will then find a MediaController which should belong to the media notification, but then be assigned to the none-media notification since both have the same package name.
We now move the if (medianotification != null) {} block outside the
controller to make sure the correct media notification is set after its
inflation is completed.
Test: runtest systemui
Test: manual - open chrome : 1)download a file 2)play any youtube video 3)screen off then on device
Fixes: 75988950
Change-Id: Ic7f341cd0b0cf675807d2127e61cbb651f9dff75
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/NotificationMediaManager.java
|
| 4a96b36fd9857cd3d3534ed0396ec3d7155a324c |
|
14-Dec-2017 |
Eliot Courtney <edcourtney@google.com> |
Remove NotificationPresenter#getEntryManager.
Instead, pass it along with calls to setUpWithPresenter.
Bug: 63874929
Bug: 62602530
Test: runtest systemui
Test: Compile and run
Change-Id: Icd60cf671b12b33d4565cc608c6141cd580c790e
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/NotificationMediaManager.java
|
| a6d8cf294dfb587f130fdecc5e6897e75de7bf45 |
|
20-Oct-2017 |
Eliot Courtney <edcourtney@google.com> |
Split NotificationEntryManager out of StatusBar.
NotificationEntryManager is responsible for the adding, removing, and
updating of notifications among other things, such as their inflation
and their interaction with other Notification*Manager objects.
Bug: 63874929
Bug: 62602530
Test: runtest systemui
Test: Compile and run
Change-Id: I56f8c524875900112cdf9f6120407b61e201172f
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/NotificationMediaManager.java
|
| 09322286c176af651bd1816a4540572d78e01aa1 |
|
09-Nov-2017 |
Eliot Courtney <edcourtney@google.com> |
Split NotificationUserManager out of StatusBar.
This class handles functionality related to private/public contents,
redacting notifications, whether to show notifications on the
keyguard, and user changes.
Bug: 63874929
Bug: 62602530
Test: runtest systemui
Test: Compile and run
Change-Id: If44fdf5f89c1e72c4bd5f49731fd4a158ea50552
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/NotificationMediaManager.java
|
| 3ebbccdf5c8db4ced1ecfb1358058a0b12654860 |
|
08-Nov-2017 |
Eliot Courtney <edcourtney@google.com> |
Split NotificationMediaManager out of StatusBar.
This class keeps track of the current media notification, and performs
various media notification related tasks. This is part of an ongoing
refactor to modularise StatusBar.
Bug: 63874929
Bug: 62602530
Test: runtest systemui
Test: Compile and run
Change-Id: I2f6c0c66f824e3ae24690794a5e8421c898ec355
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/NotificationMediaManager.java
|