| d246bed148f5505bd65627027dfe27313b03fd4c |
20-Jun-2017 |
Selim Cinek <cinek@google.com> |
Retrying the remoteview application on the ui thread
Because of various reasons, the async inflation could spuriously
fail, leading to dropped notifications.
We're now retrying them on the UI thread before we fail completely.
Test: runtest -x packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/NotificationInflaterTest.java
Bug: 38190555
Change-Id: I13feaeaad925b72798a22194add16528c9438412
otificationInflaterTest.java
|
| 131f1a4bc491c07f0c3f643ac5b2551061799024 |
06-Jun-2017 |
Selim Cinek <cinek@google.com> |
Fixed a bug where the navigation notification would flicker
Change-Id: Id68ba339407287d67d8db888515389e79e09ece3
Fixes: 62339936
Test: runtest -x packages/SystemUI/tests/src/com/android/systemui/statusbar/NotificationCustomViewWrapperTest.java
otificationViewWrapperTest.java
|
| 3d5fd7a95993db3a96bd1ee712dc391ee57b9955 |
26-May-2017 |
Selim Cinek <cinek@google.com> |
Merge "Fixed memory leak with the inflater" into oc-dev
|
| 67ff248168fb77dadd81886df134cbfe048a6001 |
25-May-2017 |
Selim Cinek <cinek@google.com> |
Fixed memory leak with the inflater
Because min priority children could be removed from
their parents after the removal, a new inflation task
would be started, leading to the view being instantly
readded again. This lead to memory leaks.
It also fixes a bug where the inflation would not inflate
enough views that could lead to crashes / wrong layouts.
Finally there was a indexing error when handling removal
of group summaries.
Test: runtest -x packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/NotificationInflaterTest.java
Change-Id: Iac242946bd30060967ee7877560d40e63f39f996
Fixes: 62067645
otificationInflaterTest.java
|
| 25a52b65b2cac1f49f37f4532cfa62282432957a |
23-May-2017 |
Jason Monk <jmonk@google.com> |
Add test to verify all sysui tests extend the right stuff
And then make it pass.
Test: runtest systemui
Fixes: 38500852
Change-Id: I377164219e4a4f4d1a99f11ef06da03c2b527eb0
otificationInflaterTest.java
otificationViewWrapperTest.java
isualStabilityManagerTest.java
|
| 70950def80d1aa5938c104e125635e0d42933ba7 |
23-May-2017 |
Geoffrey Pitsch <gpitsch@google.com> |
Change Ignores to FlakyTest so we can check if they're fixed
Test: runtest systemui
Bug: 38500852
Change-Id: I84fd8178086dbd4e35c72e7ef2780248a00791e8
otificationInflaterTest.java
|
| 351a321371c307e92ee73ea6d179de41518d1ef3 |
22-May-2017 |
Geoffrey Pitsch <gpitsch@google.com> |
Ignore a bunch of systemui tests until flakes are found
Test: runtest systemui
Bug: 38500852
Change-Id: I54f0e868bd9432758ff21a344631b6888bc83a4b
otificationInflaterTest.java
|
| 0b84204185377c1d95c0847eaf3eaab8f1a53c62 |
12-May-2017 |
Selim Cinek <cinek@google.com> |
Merge "Fixing the colorization of legacy media notifications" into oc-dev
|
| 454a5fc4bb5cb9d5242cf88d53cee5d811f02a3a |
12-May-2017 |
Selim Cinek <cinek@google.com> |
Fixing flakey tests
Sometimes if the system is slow or the process isn't scheduled in time,
our test may take a long time. We still want to test the async
flow, and exclude the flakyness in the rest of the cases, so we
removed the timeout.
Test: runtest systemui
Change-Id: I562d0798d274ca93d9ae0644f44504428d84023c
Fixes: 38184803
otificationInflaterTest.java
|
| a7679b6f3f0fc3e7c40102b732cfa51a9830a891 |
11-May-2017 |
Selim Cinek <cinek@google.com> |
Fixing the colorization of legacy media notifications
Previously the colorization wouldn't work if the notification
was not targeting N and above, since for those the
remoteviews would be built into the notification and we couldn't
colorize it.
Test: runtest -x packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/MediaNotificationProcessorTest.java
Fixes: 38147364
Fixes: 37743600
Change-Id: Iabad02a4515c42676c0660293e58cf62d5d8ff88
ediaNotificationProcessorTest.java
ropertyAnimatorTest.java
isualStabilityManagerTest.java
|
| 0f66a4cc16ec1a927c90ac559c73c80ddcb5ee71 |
29-Apr-2017 |
Selim Cinek <cinek@google.com> |
Moving Row inflation to the background too
Previously only the contentview inflation was on the
background, now the inflation of the row is too.
Test: runtest systemui
Bug: 34888292
Change-Id: I3adc6b3311217421c9de5c37794397b8a3fd665d
otificationInflaterTest.java
|
| 01d3da63cef1f82db182c6995264bf3ea3371dcc |
29-Apr-2017 |
Selim Cinek <cinek@google.com> |
Moving the inflation to the background
The inflation of the notifications is moved to
the background. This should improve the general
performance when adding / removing groups and alike.
Test: runtest systemui
Fixes: 34888292
Change-Id: Ieb19a09a5a97d496d9319d917a5e317a3ad76fc4
otificationInflaterTest.java
|
| dc1231c3b9fd72790f767a605136e3f05a2af3e2 |
28-Apr-2017 |
Selim Cinek <cinek@google.com> |
Fixes a bug where the asynctask wasn't removed when inflating
Test: runtest systemui
Bug: 36561228
Change-Id: I9b5099d4a32be1f18db2ab1f8c94fe6f35a51b18
otificationInflaterTest.java
|
| 2630dc7ecf16a965321cfc615104f372cd2b0e96 |
21-Apr-2017 |
Selim Cinek <cinek@google.com> |
Moved notification builder creation to the background
In preparation of future colorization work, the creation
of the notification builder is moved to the background.
Test: manual, add notifications, update them etc.
Test: runtest systemui
Bug: 36561228
Merged-In: Iaec5febf4d8d9da348d77e0d4f6f61b9746fae16
Change-Id: Iaec5febf4d8d9da348d77e0d4f6f61b9746fae16
otificationInflaterTest.java
otificationinflaterTest.java
|
| 2d691791e7e673dc824c434850970e3493deac65 |
07-Apr-2017 |
Adrian Roos <roosa@google.com> |
Merge "Revert "Ignore a broken test until it can be fixed."" into oc-dev
|
| ad27aa11b1af13c70f69cfff58e7a078eb33b8e2 |
07-Apr-2017 |
TreeHugger Robot <treehugger-gerrit@google.com> |
Merge "NotiViewWrapper: Fix test that breaks on master" into oc-dev
|
| 2f6bef2c17acde80a857e37ae34a2b6801d5b197 |
07-Apr-2017 |
Adrian Roos <roosa@google.com> |
Revert "Ignore a broken test until it can be fixed."
This reverts commit 1ee48434efc8129c604b20eae90605b0098f674e.
Change-Id: Ie51d2931a6bafb187902c6ac845066ca70937f2c
otificationViewWrapperTest.java
|
| 4c720d2c1377889fa3482fca9655e9e4720f1a5c |
05-Apr-2017 |
Adrian Roos <roosa@google.com> |
NotiViewWrapper: Fix test that breaks on master
oc-dev allows constructing views with a null context, but
that doesn't work any longer on master.
Bug: 36430936
Change-Id: If069f81d54e3e3b9145ca5187b87888cd6173c0f
Merged-In: If069f81d54e3e3b9145ca5187b87888cd6173c0f
otificationViewWrapperTest.java
|
| 1ee48434efc8129c604b20eae90605b0098f674e |
06-Apr-2017 |
Jason Monk <jmonk@google.com> |
Ignore a broken test until it can be fixed.
Test: runtest systemui
Bug: 37074875
Change-Id: Id5561a3ded2242c711c2edccefb1647437d6f963
otificationViewWrapperTest.java
|
| 7c68e29171dca0cb2a914a9c701707d5c562cb03 |
05-Apr-2017 |
Adrian Roos <roosa@google.com> |
NotificationViewWrapper: Fix crash for custom views
Bug: 36430936
Test: runtest -x packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/NotificationViewWrapperTest.java
Change-Id: Ibdf9cf90fcb69a8af8a3b60596143e3be83fe46e
Merged-In: Ibdf9cf90fcb69a8af8a3b60596143e3be83fe46e
otificationViewWrapperTest.java
|
| 10790672a98debafb8882971e94ff26aec9a6bfa |
09-Mar-2017 |
Selim Cinek <cinek@google.com> |
Fixed a bug where the increased height wasn't applied correctly
The wrong value was used for the increased heads-up and the
increased contentViewHeight.
Test: runtest -x packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/NotificationinflaterTest.java
Change-Id: Ia72b2de890d89d4cffb40eaa653a62f426945451
otificationinflaterTest.java
|