[blockly] Fix timer remove regression

[stefan-hoehn]

Fixes #2002.

[relativeci]

Job #1116: Bundle Size — 15.75MiB (~-0.01%).

416fcf0(current) vs 26fca17 main#1115(baseline)

⚠️ Bundle contains 16 duplicate packages

Metrics (1 change)

	Current Job #1116	Baseline Job #1115
Initial JS	1.67MiB(-0.01%)	1.67MiB
Initial CSS	608.89KiB	608.89KiB
Cache Invalidation	93.94%	93.94%
Chunks	219	219
Assets	689	689
Modules	1700	1700
Duplicate Modules	90	90
Duplicate Code	1.95%	1.95%
Packages	138	138
Duplicate Packages	15	15

Total size by type (2 changes)

	Current Job #1116	Baseline Job #1115
CSS	859.34KiB	859.34KiB
Fonts	526.1KiB	526.1KiB
HTML	1.23KiB	1.23KiB
IMG	140.74KiB	140.74KiB
JS	9.25MiB (~-0.01%)	9.25MiB
Media	295.6KiB	295.6KiB
Other	4.72MiB (~-0.01%)	4.72MiB

---

View job #1116 report View main branch activity

[florian-h05]

LGTM, thanks!
I’ve just had a look a the Nashorn code, there it also does not remove the timer from this.

[florian-h05]

@kaikreuzer Can you please cherry pick this to 4.0.x before the release is done? I’ve mentioned it in the release thread, seems like you didn’t see it.

[kaikreuzer]

I indeed didn't see the message and so this unfortunately hasn't yet been backported before the 4.0.3 build. I did so now, so that it will end up in 4.0.4.
Please best always consider backporting directly when merging a PR; otherwise it also becomes very tricky with the compilation of the release notes.

[florian-h05]

I usually backport when merging, but I was away (on vacation) and only had my phone with me, I merged with GitHub web but I cannot backport from my phone. Thanks for doing it anyway.