move some logic from mainform to moviesession, remove TasMovie check that makes no sense

BizHawk.Client.Common/movie/MovieSession.cs

@@ -227,6 +227,11 @@ namespace BizHawk.Client.Common
 		/// <exception cref="MoviePlatformMismatchException"><paramref name="record"/> is <see langword="false"/> and <paramref name="movie"/>.<see cref="IMovie.SystemID"/> does not match <paramref name="emulator"/>.<see cref="IEmulator.SystemId"/></exception>
 		public void QueueNewMovie(IMovie movie, bool record, IEmulator emulator)
 		{
+			if (movie.IsActive() && movie.Changes)
+			{
+				movie.Save();
+			}
+
 			if (!record) // The semantics of record is that we are starting a new movie, and even wiping a pre-existing movie with the same path, but non-record means we are loading an existing movie into playback mode
 			{
 				movie.Load(false);

BizHawk.Client.EmuHawk/MainForm.Movie.cs

@@ -11,13 +11,6 @@ namespace BizHawk.Client.EmuHawk
 	{
 		public bool StartNewMovie(IMovie movie, bool record)
 		{
-			// SuuperW: Check changes. adelikat: this could break bk2 movies
-			// TODO: Clean up the saving process
-			if (movie.IsActive() && (movie.Changes || !(movie is TasMovie)))
-			{
-				movie.Save();
-			}
-
 			try
 			{
 				MovieSession.QueueNewMovie(movie, record, Emulator);