When code runner extension gets installed, disable the play button icon immediately

[karrtikr]

For #7613

[codecov-io]

Codecov Report

Merging #7616 into master will decrease coverage by 0.32%.
The diff coverage is 87.5%.

@@            Coverage Diff             @@
##           master    #7616      +/-   ##
==========================================
- Coverage   58.68%   58.35%   -0.33%     
==========================================
  Files         493      493              
  Lines       21990    21989       -1     
  Branches     3531     3525       -6     
==========================================
- Hits        12905    12832      -73     
- Misses       8279     8351      +72     
  Partials      806      806

Impacted Files	Coverage Δ
src/client/common/types.ts	100% <ø> (ø)	⬆️
src/client/terminals/activation.ts	83.87% <100%> (-0.98%)	⬇️
src/client/common/application/extensions.ts	50% <50%> (-10%)	⬇️
src/client/testing/serviceRegistry.ts	48.64% <0%> (-46.85%)	⬇️
src/client/testing/codeLenses/main.ts	44.44% <0%> (-33.34%)	⬇️
...t/datascience/jupyter/jupyterDataRateLimitError.ts	66.66% <0%> (-33.34%)	⬇️
src/client/formatters/serviceRegistry.ts	75% <0%> (-25%)	⬇️
src/client/common/utils/icons.ts	83.33% <0%> (-16.67%)	⬇️
src/client/api.ts	78.57% <0%> (-14.29%)	⬇️
src/client/datascience/cellFactory.ts	75.86% <0%> (-13.8%)	⬇️
... and 3 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8bc45c3...7db0739. Read the comment docs.

[karthiknadig]

For my info. Why not await here?

[karrtikr]

executeCommand returns a thenable type object (by design), not a promise.
So although it's an async operation, we can't use await.

[DonJayamanne]

I'd say we can wait, but there's no need to wait. I.e. no point delaying other code when we don't care how long it takes to complete.

[karrtikr]

Wait, I just noticed we can do that, not sure why I didn't use it in the first place.