Commit 27bcdc2
committed
notif [nfc]: Add Android-only asserts in NotificationDisplayManager methods
NotificationDisplayManager's public methods only get invoked on
Android, and their implementations basically assume the platform is
Android. So, add these asserts, to make that assumption clearer and
to fail early if we accidentally call one of these on iOS.
When we eventually take more hands-on control of notifications on
iOS, we might either adapt this class to support iOS too, or make a
new class for iOS and make this one's name and interface clearer
that it's Android-only.
Discussion:
https://chat.zulip.org/#narrow/channel/243-mobile-team/topic/NotificationDisplayManager.20Android-only/near/20991961 parent 9350346 commit 27bcdc2
1 file changed
+7
-0
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
217 | 217 | | |
218 | 218 | | |
219 | 219 | | |
| 220 | + | |
220 | 221 | | |
221 | 222 | | |
222 | 223 | | |
223 | 224 | | |
| 225 | + | |
224 | 226 | | |
225 | 227 | | |
226 | 228 | | |
| |||
471 | 473 | | |
472 | 474 | | |
473 | 475 | | |
| 476 | + | |
| 477 | + | |
474 | 478 | | |
475 | 479 | | |
476 | 480 | | |
| |||
499 | 503 | | |
500 | 504 | | |
501 | 505 | | |
| 506 | + | |
502 | 507 | | |
503 | 508 | | |
504 | 509 | | |
| |||
527 | 532 | | |
528 | 533 | | |
529 | 534 | | |
| 535 | + | |
| 536 | + | |
530 | 537 | | |
531 | 538 | | |
532 | 539 | | |
| |||
0 commit comments