These methods return an IEditor as a result but even if the opening fails or the editor was not opened as active.

The fix is to check for the result of the open call and only return the editor if the editor was opened actually.

This restores the behaviour we had before GRID.

fixes #64767

* lifecycle - signal READY earlier to allow commands to run

* lifecycle - move READY phase as last thing into shell

* workbench - isCreated() => isRestored()

* lifecycle - tweak phase

* remove more TPromise (for #53526)

* fix failing webview integration tests on macOS

* do not instantiate TPromise (fix #62716)

* more TPromise removal and fixes

* 💄

* 💄

We compile using the alwaysStrict flag so these directives are not needed