When replacing one output with another, we were calling suspend
on the old output's observer object. This prevents the observer
from being scheduled in the future, but it doesn't prevent an
already-scheduled execution from proceeding. Since we have a new
definition for what an output is, there's no way another execution
of the old observer will do anything useful.
One simple way to see the difference this commit makes is to
create a simple Shiny app that sets output$plot twice in server.R,
with a println in each renderPlot. You'll see that both execute
once, whereas with this change only the most recently set renderPlot
gets executed.
b414ae512a