Skip to content
Commit f9439ab4 authored by Mady Mellor's avatar Mady Mellor
Browse files

Fix experimental bubbles from HUN'ing when they shouldn't

I changed this to onNotificationAdded so that we could extract
the avatar generated for the messaging template for experimental
bubbles, however, that breaks how notification interruption state
provider (NISP) works (we get onNotificationAdded after it goes
through NISP that to check if the notif should HUN). It's technically
more correct to use onPendingEntryAdded here, so lets do that.

If the person object doesn't have an avatar we'll end up using
the small or large icon from the notification instead, this at
least works for most apps I've tried with experimental bubbles.

Test: have no hangouts bubbles, get a hangouts bubble, note that
      it doesn't HUN
Fixes: 148609860

Change-Id: I6e19e49b8b5f13969c30410450826bf480fb9d34
parent 8ac84650
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment