FPSCounter: Flush the logs every second and close them when the renderer is shut down.

We also move `FPSCounter::Update()` to RenderBase, where it is called after `SwapImpl()`, does this affect our results?
Of which I haven't been given any hypothetical valid reasons.

Technically it already had this constraint prior to this, and there were no
problems.
On Jul 16, 2014 3:17 PM, "Jules Blok" <notifications@github.com> wrote:

> In Source/Core/VideoCommon/FPSCounter.h:
>
> > @@ -4,12 +4,32 @@
> >
> >  #pragma once
> >
> > -namespace FPSCounter
> > +#include <fstream>
> > +
> > +#include "Common/Timer.h"
> > +
> > +class FPSCounter
>
> Why would we put such constraints on it right away? Inheriting this class
> will work just fine and there may be valid reasons to do so.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dolphin-emu/dolphin/pull/595/files#r15021045>.
>