tree 06117a946fedc679edcf881600a33a9182076304
parent 94c3882388c77e214f502aad94a518a340bb7f24
author lpbeliveau-silabs <112982107+lpbeliveau-silabs@users.noreply.github.com> 1691695161 -0400
committer GitHub <noreply@github.com> 1691695161 -0400
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJk1Tg5CRBK7hj4Ov3rIwAAtycIAF8uu/XMUCWRqLeONInwzaBr
 GZl6wZao1CJcZJCXMHm8VCl1NP7MWJ1ctrqvBeeilClCrPBR3nXlDBfp6qlUZzif
 Z0yFrnQ33uS0IfGjQHBiEoYetu3PYBKaUXIZ0Cqq8G8fo/bjC4Yz01CkGzLUvo3C
 Dko819KvNOgzNs2XPfmXAMyZoDkWInMzg8EIIcz7Eex/HqqwL7vmLxaxTUNqPR5T
 fGrp7z8I6717HhfogAiTsoJnUWwomxcD/TvMnPOhsoqjrdvkIeOSUmT0w6cpbaMN
 XL4dtLkFOjN+ldE5kEqPlhE2PsqC9jE1m56n/uqCouzusfVRAzSvQ+BfB+SiTX4=
 =kfMs
 -----END PGP SIGNATURE-----
 

[ReadHandler] Removed Scheduling of report from OnReadHandlerCreated  (#28536)

* Removed Scheduling of report from OnReadHandlerCreated since it caused immediate scheduling before the intervals are negotiated

* Separated Read reports from Subscription reports and renamed flags and accessors for clarity

* Apply suggestions from code review

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

* Put back rescheduling in the OnReadHandlerSubscribed, added a CanEmitReadReport() method for readhandler for read request. Fixed call order in TestReportScheduler tests

* Removed redundancy by removing un-necessary OnSubscriptionAction(), added comment for OnReadHandlerSubscribed and modified reporting condition for Read reports to be sent independently from the report scheduler

* Apply suggestions from code review

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

* Appplied change to method name and fixe condition in SetStateFlag

* Update src/app/ReadHandler.h

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

* Removed un-necessary check in SetStateFlag

---------

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>