To @peterflynn's assumption that the dot may not get cleared. It actually does get cleared, because showing a custom viewer always clears the current editor, by briefly going to the NoEditor view state. At that point the dot is cleared. The title is updated again when the custom viewer is actually shown, but at that point the dot is already gone. That said, I still think the else block should be there, because it would be bad to assume the view state dance a requirement for this code to work correctly. Yet, with the current CustomViewer implementation the code will never be executed.