tree d01bc849a96f65bde5622b86060298654f1d7be3
parent b3f8db67cf1ae09c20d4a8caa7dd901e95b1dcd7
author lpbeliveau-silabs <112982107+lpbeliveau-silabs@users.noreply.github.com> 1689265244 -0400
committer GitHub <noreply@github.com> 1689265244 +0000
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJksCRcCRBK7hj4Ov3rIwAAQywIAFjLkeM7djuHNmYIc6kb1teI
 BIrTytopRQ/hCQzrFLqzb7lQ1+RTTFJJTxvAmZTxQtfxIBgsxej8DTHEFyjC017K
 0xRbX9c6G3dIMLi9Z/58DFYUyPfAPmqS6MCxiCLLM8jy3u7clUfLaTbj2P0z0npX
 2eyO6G4wXJOPXATJByauso17JJkm/r5SmD/cC+WyozjQQ1N8MOBdltLw9Cjih7rX
 +xa8nJQEiEoyiwDU1FEp3yTuwtRudmyQ0vXZxjQTShj8eDQbxAV/z9M61THQX2fo
 vOf/vLKkCYFHJJRn7vIZsfFpv8SwxZ1i1o/VdyxpyHg5UaiS0f0KEF21Leb15As=
 =Ujbk
 -----END PGP SIGNATURE-----
 

[ReadHandler] Report Scheduler class (#27553)

* Added a new class that will handle the scheduling of reports.

* Restyled by clang-format

* Removed un-necessary define in TestReportScheduler and applied refactor of SetReportingIntervals to SetMaxReportingIntervals to platform code

* Added TimerDelegate and wrapper functions around calls to Timer. Remove unnecessary checks for nullptr

* Added VerifyOrReturn after NL_TEST_ASSERTS for nullptr

* Completed TimerDelegate class and modified ReadHandlerNodes so they carry their own callback

* Modified TimerDelegate to allow to pass different objects as context

* ifdefing out ScheduleRun() to debug failing CI

* Added issue # to TODOs, refactored Min/Max Intervals to Min/Max Timestamp

* Clarified some comments regarding timing

* Restyled by whitespace

* Restyled by clang-format

* Added interface to GetMonotonicTimestamp in the timer delegate

* Apply suggestions from code review

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

* Completed renaming to eliminate compiling error, moved TestReporScehduler in reporting namespace, addressed some low hanging fruits

* Removed useless objects from tests as well as useless typecasting, and unnecessary check

* Fixed comment about private methods used in ReportScheduler as a friend class

* Changed to SetMinReportInterval to SetMinReportingIntervalForTests, removed the IsChunkedReport from comment about friend class, added a mock timestamp and timer to test to better control time in simulation for specific timing test cases

* Apply suggestions from code review

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

* Restyled by clang-format

* Removed all calls to ReadHandler States to prevent Engine calls from the Test as it seems to impact the CI

---------

Co-authored-by: Restyled.io <commits@restyled.io>
Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>