tree 148da2a13cb6dd1cadbeb3af16eb266c33b09970
parent d0fd3f7240f44aabdfcaa4f15bd8c7facdfc0bf1
author Srujan Gaddam <58529443+srujzs@users.noreply.github.com> 1756147189 -0700
committer GitHub <noreply@github.com> 1756147189 -0700
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsFcBAABCAAQBQJorK31CRC1aQ7uu5UhlAAAiS4QAGmTxm+3tzi4q777MreJzvJI
 9SuU7boNk3DEL6GMZPlp/LT+i6Rkn1zf+Ay1xczA5naasTYp2nnEPOsvxBbP/hFu
 C7mp4zHHP3l4dfxDVio7gC0477v3w/KneyinWUgNc/W5cCWwKccE7qLfbaxdg7mU
 PDBn+l5OtycK1Atj0wgehjRT3b8NuDkRU6bwyrpMGEZS4dLHPqPyNgukfhGVUVy/
 aU2JaoHNSLssUiU+LTpdTqAnolGJyjmp9aB6ecN4XhmEeHm3qoh3RiCHGwK51dXt
 EJrrt4IE0nR4y8LTJTEpLRajqP0DbRCY8NthUthym3DFc8sdmbUVXho4N/NXhowR
 1EtEu+xxDDxpYjbLQRnaJ6G6gXPJyTZ78GNK/Ii4ox4OQA0yRPd8Sdwk6izxyf/v
 1M5T5iyr+LN4JJO4EHFk1A+hNQPP2IHEHXOYOWAHQSbyLnXstzX0PQ93a+9fJMU2
 tb4BZF9p2SimQ1p40CEqyNFc6aO//w5W8u4OW0B1L9VxMrdjLY44BNUZFAEFNDxj
 m1l1gf0Zcw3dquAPuD3jngsYZ69hTmPnE2joFBpQaIO+87OASaM80A1v8mmV1dyA
 SHIx9Oj2qC5vVQcYnfm7U77zFxNVky3JnusnHSS7y/pXdi/0QyivXCqSlydQ8+N5
 yij51K0N5P3dlCUFchZ2
 =izNT
 -----END PGP SIGNATURE-----
 

Delay edits to work around lack of millisecond precision in `stat` and migrate `pageSource` tests to use console logs (#2670)

Currently, CI is red due to two issues on Windows:

- hot_reload_breakpoints_test: On Windows, dart:io doesn't have enough precision to provide a millisecond-based stat. Instead, it just rounds to the nearest second. We use this to determine what files were modified in a recompile whereas we use DateTime.now (which does have millisecond precision) to mark the timestamp of the previous compile. This then potentially leads to frontend_server_common mistakenly not recompiling some files.

To fix this, we stage all the edits at once (if possible) and delay for 1 second to guarantee that stat will always give a time after the previous compile if the file was modified.

- hot_reload_test: This and a number of tests use innerHTML, document.body.append, and the pageSource API to modify the page text and then read it. A previous race condition where we tried to compute the page before main was done executing was resolved by waiting for a log at the end of main. However, this is insufficient, because the changes to the page still may not bubble up by the time we try to calculate the page source, which was the issue with hot_reload_test.

We could do a busy-loop calculating the page until we get the text we want, but that's erroneous and a better solution is to just use console logs, which fixes the original issue. As part of this, a WebkitDebugger is always initialized in the test context so we can inspect console logs regardless of whether DWDS debugging is enabled.