Falling off the ob_start stack

function my_output($blah) {
  ob_start();
  really_long_function_call();
  return ob_get_clean();
}

Had been turned into:

function my_output($blah) {
  ob_start();
  if($cache = cached('really_long_function_call') {
    return $cache;
  really_long_function_call();
  return ob_get_clean();
}

The ob_start() stack was now out of whack. Whenever the top level layout code then tried to close an ob_start(), it would actually close the level that had been started in my_output(), which would cause page elements to show up out of order.

To make things more interesting, when PHP reaches the end of a program, it closes all open buffers and flushes their output out to its output. This really through us off as we spent the first hour of debugging into looking why the buffered output was being flushed when it shouldn't, when this actually wasn't happening.

This is one of those standard cases that when you suspect the bug is in the system libraries instead of your code, you're probably doing something wrong.