History log of /frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationInflater.java
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
6dceace0eddd08156e6b71c17e3de4ed5f4f2f41 16-May-2018 Jason Monk <jmonk@google.com> Reduce usages of main looper in sysui tests

Push over to a standard testable looper, and a testable looper +
setAsMain when needed.

Also make tests more synchronous and single threaded as possible.
This will make them more deterministic and speeds them up noticeably.

Test: runtest systemui
Bug: 79550837
Change-Id: Iab0eb794329d7b1de95aef904ec08ecae7dadc98
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationInflater.java
c7f5a827d69e68931cd9f57cf7e158bc0055b7e7 21-Mar-2018 Selim Cinek <cinek@google.com> Updated the styling for notifications

The new notifications now have different font weights
and colors to make the shade more readable.

Fixes: 69168591
Test: runtest systemui
Change-Id: I0b635724fa122d292841e56efa84aa57fa364300
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationInflater.java
aa9db1f34fe8b4a2d143c1379ec6c0c304bbd40b 28-Feb-2018 Selim Cinek <cinek@google.com> Insetting heads up notifications

Heads up notifications are now corretly respecting insets.
instead of overlapping with any possible notches, we're
insetting heads up notifications and splitting the main
content from the header.

Fixes: 72748440
Test: runtest systemui
Change-Id: Ie53ea31fef4e468239c4346f9d1f192bcb26e11d
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationInflater.java
fc8073c4b606e18366660cea5cbe570c9bef5a51 17-Aug-2017 Selim Cinek <cinek@google.com> Fixed a bug with decorated custom views

Even if their layout still maches, the content was unrecoverably
destroyed, so we can't just reapply it anymore.

Fixes: 62911941
Test: runtest -x packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/NotificationInflaterTest.java

Change-Id: I3a50c96484686958570ac5e4949df3ad4b8a421a
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationInflater.java
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
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationInflater.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
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationInflater.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
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationInflater.java
ae8de8c77c51b7b26ba92f2cce76bb5dd74bea73 20-May-2017 Selim Cinek <cinek@google.com> Fixed an issue when inflating too many notifications

The pool of queued asynctask was limited and should really
not be, because we might be inflating a lot views at the
same time.

Test: runtest systemui
Change-Id: I3e724985e01483531051a3e9e255faa69d2591d2
Fixes: 38261831
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationInflater.java
ac5f02749a595d39711beb4a1defb01949eb548a 03-May-2017 Selim Cinek <cinek@google.com> Fixed the contrast for low-priority notifications

The low priority notifications had their contrast calculated
against the white background and weren't satisfying
our contrast requirements.
This also aligns the case where the app had no colors better,
and ensures that it's actually using the secondary text
color there.

Test: existing tests pass
Change-Id: Ic11e9d06783e60998f35e0eb7f6f29fb1d86c7df
Fixes: 37444266
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationInflater.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
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationInflater.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
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationInflater.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
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationInflater.java
0847acd4d48deb716de99d259364bc99b592a8bc 25-Apr-2017 Selim Cinek <cinek@google.com> Implementing gradual image fade for the media template

Test: play music
Merged-In: I525513ce1da1237c4edad32c0ed31e79d5eacd32
Change-Id: I525513ce1da1237c4edad32c0ed31e79d5eacd32
Fixes: 36561228
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationInflater.java
5fb73f86299d9cc616ca741f8c7c4af2485cc273 21-Apr-2017 Selim Cinek <cinek@google.com> Extracting the notification colors based on the album art

Media notifications are now extracting the background and
foreground colors from the album art.

Test: manual, play different songs
Bug: 36561228
Merged-In: I9c3c962fa59eb70ef9b2d4893b939be6e1ee1ab0
Change-Id: I9c3c962fa59eb70ef9b2d4893b939be6e1ee1ab0
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationInflater.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
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationInflater.java
1a1ecfcf5ae32482aee23ebc7c4389daf164cadd 17-Apr-2017 Adrian Roos <roosa@google.com> AOD: Implement redacted notificatinos

Test: Turn on "show sensitive notifications on lockscreen", receive notification, observe AOD
Bug: 36610748
Change-Id: I981ef4cea7cefc76dcd55ebad9d05c758927dabb
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationInflater.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
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationInflater.java
c478f9023d7649d85e60759c9e6c0e033774293b 23-Feb-2017 Selim Cinek <cinek@google.com> Creating the right views for low-priority group children

Previously we always created the low-priority views even if it
was in a group, but now we use the collapsed or the singleline
one based on the group state.

Test: runtest systemui
Bug: 35125708
Change-Id: I49d418912f65c3bc8b672d383eb6a0526fcb8993
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationInflater.java
1a48babb5ed30e8a3eb9645355d6455b21041406 18-Feb-2017 Selim Cinek <cinek@google.com> Move the inflation away from the statusbar

Since the notifications need to inflate more dynamically
based on their own state, the inflation is moved away
from the statusbar.
This also improves the apply inplace logic, that was
reinflating all views even if only a single notification
layout was changed.

Test: runtest systemui
Bug: 35125708
Change-Id: I42a33065ab60b7c45ca979ae2d7baa1518bf92b7
/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationInflater.java