Remove Loaded property from IMovie and refactor accordingly, it was only needed for a hack that had faulty logic

BizHawk.Client.Common/movie/IMovie.cs

@@ -17,7 +17,6 @@ namespace BizHawk.Client.Common
 		bool IsRecording { get; }
 		bool IsFinished { get; }
 		bool Changes { get; }
-		bool Loaded { get; }
 
 		#endregion
 

BizHawk.Client.Common/movie/Movie.cs

@@ -46,7 +46,7 @@ namespace BizHawk.Client.Common
 
 		public int InputLogLength
 		{
-			get { return Loaded ? _log.Length : _preloadFramecount; }
+			get { return _log.Length; }
 		}
 
 		public double FrameCount

BizHawk.Client.Common/movie/TasMovie.cs

@@ -116,11 +116,6 @@ namespace BizHawk.Client.Common
 
 		public bool Changes { get; private set; }
 
-		public bool Loaded
-		{
-			get { throw new NotImplementedException(); }
-		}
-
 		public TimeSpan Time
 		{
 			get
@@ -160,7 +155,7 @@ namespace BizHawk.Client.Common
 			}
 			else
 			{
-				_mode = Moviemode.Finished;
+				_mode = Moviemode.Finished; // TODO: never do this in a TasMovie!
 				return String.Empty;
 			}
 		}
@@ -183,7 +178,6 @@ namespace BizHawk.Client.Common
 		public void SwitchToPlay()
 		{
 			_mode = Moviemode.Play;
-			Save();
 		}
 
 		public void StartNewPlayback()

BizHawk.Client.EmuHawk/MainForm.Rewind.cs

@@ -603,13 +603,19 @@ namespace BizHawk.Client.EmuHawk
 		{
 			for (int i = 0; i < frames; i++)
 			{
-				if (RewindBuf.Count == 0 || (Global.MovieSession.Movie.Loaded && Global.MovieSession.Movie.FrameCount == 0))
+				if (RewindBuf.Count == 0 || (Global.MovieSession.Movie.IsActive && Global.MovieSession.Movie.InputLogLength == 0))
+				{
 					return;
+				}
 
 				if (LastState.Length < 0x10000)
+				{
 					Rewind64K();
+				}
 				else
+				{
 					RewindLarge();
+				}
 			}
 		}
 

BizHawk.Client.EmuHawk/tools/TAStudio/TAStudio.cs

@@ -11,8 +11,6 @@ namespace BizHawk.Client.EmuHawk
 {
 	public partial class TAStudio : Form, IToolForm
 	{
-		// TOOD: clean up Input painting logic (lock to a column, lock to on or off)
-
 		private const string MarkerColumnName = "MarkerColumn";
 		private const string FrameColumnName = "FrameColumn";