This change should not cause any existing tests to fail because the last line in `updateSubjectHeight` is `this.updateElementsHeight();`, so we are just removing a redundant call.

Julien, can you advise if/how we should unit test this?