It is a bit uncool to have variables of type `Action`. We do call toplevel `render*` type methods but always for a `render*Json` or reverse routing disambiguation. You might consider prying the common `Future[SimpleResult]` bit of this and `renderImage` into a separate method.