tree d032ff55a4149fd020a3d3f30f589ba73522f133
parent fe4c679ace3eb134d7365d3bd0fe6c2fad6ece87
author lpbeliveau-silabs <112982107+lpbeliveau-silabs@users.noreply.github.com> 1690244859 -0400
committer GitHub <noreply@github.com> 1690244859 +0000
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJkvxb7CRBK7hj4Ov3rIwAAERgIACrumhHPOdKwfcI+RB3SUw4G
 qIA1+/UPPEWeHC9VoGzrx4m676V80kakQA4Bk16LHCaoSZN0gHVq4oR3ewr9O+bH
 xXAG+gCsKljOa3hiFgWAUIEzqvyL9h18Bt3xi6qt9Dr5uWmFBjxPVEtW5bpBN4/u
 zbfNQOO+DGvEkH00wV/Bun7mOc3k9UmGYFX7JVMbeyaiRFjawFW6W6kDkBI7+XKL
 GXKR4rW5CxaCz0Hmi6bNBsSMnEXkrd5F6lTUerxH46ph5hybIf7miXXeZZdEVTEX
 eTPDqH3yQgilIMc0n3st7Nd9fpWFunGH0nGvScfF3ro3ECgsTGPKNafsl8k5T1g=
 =pSEk
 -----END PGP SIGNATURE-----
 

[ReadHandler] Integration of ReportScheduler into the ReadHandler and IM engine. (#28104)

* Integration of ReportSchedulers into the ReadHandler and IM engine.

Also added a static report scheduler to use in all test relying on the IM engine, also added test flags in ReportScheduler replace ReadHandler timing flags

Refactored the TestReadInteraction to adopt correct behavior

* Restyled by clang-format

* Restyled by gn

* Moved namespaces into class to avoid namespace pollution

* Fix missing pragma once and copyright

* Update src/app/reporting/ReportScheduler.h

Co-authored-by: mkardous-silabs <84793247+mkardous-silabs@users.noreply.github.com>

* Apply suggestions from code review

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

* Removed OnIntervalsChanged, removed TestReadInteraction from ReportScheduler's friend class and refactored the tests to use wrapper functions, refactored tests for the removal of OnIntervalsChanged, removed reportscheduler nullptr check form controllers

* Restyled by clang-format

* Fix for the TimerDelegates

And also added a default Scheduler to controllers

* Restyled by clang-format

* Added a timer Context class to allow interface between timer delegates, ReadHandlerNodes and SynchronisedReport Scheduler

* Fix using InteractionModel... error

* Added Platform::Delete() for allocated TimerDelegate and ReportScheduler on controller

* Update src/app/ReadHandler.cpp

Co-authored-by: mkardous-silabs <84793247+mkardous-silabs@users.noreply.github.com>

* Update src/app/TimerDelegates.h

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

* Addressed PR comments, next step trying to reduce RAM bloat

* Restyled by clang-format

* Removed IntrusiveList from the report Scheduler

* Removed ReadHandlerNode's inheritance of IntrusiveListBase

* Restyled by clang-format

* Fix commented code and comment syntax

* Restyled by clang-format

* Update src/app/tests/TestReportScheduler.cpp

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

* Added extra check on timerDelegate in DeviceController, removed logging causing failure in linux CI

* Restyled by clang-format

---------

Co-authored-by: Restyled.io <commits@restyled.io>
Co-authored-by: mkardous-silabs <84793247+mkardous-silabs@users.noreply.github.com>
Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>