refactor(loadscreen): Refactor the single player load screen to use TheDisplay for intro video playback

GeneralsMD/Code/GameEngine/Include/GameClient/Display.h

@@ -143,13 +143,15 @@ class Display : public SubsystemInterface
 
 	/// draw a video buffer fit within the screen coordinates
 	virtual void drawScaledVideoBuffer( VideoBuffer *buffer, VideoStreamInterface *stream ) = 0;
-	virtual void drawVideoBuffer( VideoBuffer *buffer, Int startX, Int startY,
-													Int endX, Int endY ) = 0;
+	virtual void drawScaledVideoBuffer() = 0;
+	virtual void drawVideoBuffer( VideoBuffer *buffer, Int startX, Int startY, Int endX, Int endY ) = 0;
+	virtual void drawVideoBuffer(Int startX, Int startY, Int endX, Int endY) = 0;
 
 	/// FullScreen video playback
 	virtual void playLogoMovie( AsciiString movieName, Int minMovieLength, Int minCopyrightLength );
 	virtual void playMovie( AsciiString movieName );
 	virtual void stopMovie( void );
+	virtual Real getMovieProgress();
 	virtual Bool isMoviePlaying(void);
 
 	/// Register debug display callback

GeneralsMD/Code/GameEngine/Include/GameClient/LoadScreen.h

@@ -110,9 +110,6 @@ class SinglePlayerLoadScreen : public LoadScreen
 
 	UnicodeString m_unicodeObjectiveLines[MAX_OBJECTIVE_LINES];
 
-	VideoBuffer *m_videoBuffer;
-	VideoStreamInterface *m_videoStream;
-
 	void moveWindows( Int frame );
 
 	AudioEventRTS m_ambientLoop;

GeneralsMD/Code/GameEngine/Source/GameClient/Display.cpp

@@ -360,6 +360,15 @@ void Display::reset()
 		v->reset();
 }
 
+//============================================================================
+// Display::getMovieProgress
+//============================================================================
+
+Real Display::getMovieProgress()
+{
+	return (Real)m_videoStream->frameIndex() / (Real)m_videoStream->frameCount() * 100.0f;
+}
+
 //============================================================================
 // Display::isMoviePlaying
 //============================================================================

GeneralsMD/Code/GameEngine/Source/GameClient/GUI/LoadScreen.cpp

@@ -182,8 +182,6 @@ SinglePlayerLoadScreen::SinglePlayerLoadScreen( void )
 	m_currentObjectiveWidthOffset = 0;
 	m_progressBar = nullptr;
 	m_percent = nullptr;
-	m_videoStream = nullptr;
-	m_videoBuffer = nullptr;
 	m_objectiveWin = nullptr;
 	for(Int i = 0; i < MAX_OBJECTIVE_LINES; ++i)
 		m_objectiveLines[i] = nullptr;
@@ -198,15 +196,6 @@ SinglePlayerLoadScreen::~SinglePlayerLoadScreen( void )
 	for(Int i = 0; i < MAX_OBJECTIVE_LINES; ++i)
 		m_objectiveLines[i] = nullptr;
 
-	delete m_videoBuffer;
-	m_videoBuffer = nullptr;
-
-	if ( m_videoStream )
-	{
-		m_videoStream->close();
-		m_videoStream = nullptr;
-	}
-
 	TheAudio->removeAudioEvent( m_ambientLoopHandle );
 	m_ambientLoopHandle = 0;
 
@@ -476,32 +465,6 @@ void SinglePlayerLoadScreen::init( GameInfo *game )
 
 */
 	m_ambientLoop.setEventName("LoadScreenAmbient");
-	// create the new stream
-	m_videoStream = TheVideoPlayer->open( TheCampaignManager->getCurrentMission()->m_movieLabel );
-	if ( m_videoStream == nullptr )
-	{
-		m_percent->winHide(TRUE);
-		return;
-	}
-
-	// Create the new buffer
-	m_videoBuffer = TheDisplay->createVideoBuffer();
-	if (	m_videoBuffer == nullptr ||
-				!m_videoBuffer->allocate(	m_videoStream->width(),
-													m_videoStream->height())
-		)
-	{
-		delete m_videoBuffer;
-		m_videoBuffer = nullptr;
-
-		if ( m_videoStream )
-		{
-			m_videoStream->close();
-			m_videoStream = nullptr;
-		}
-
-		return;
-	}
 
 	// format the progress bar: USA to blue, GLA to green, China to red
 	// and set the background image
@@ -530,9 +493,16 @@ void SinglePlayerLoadScreen::init( GameInfo *game )
 		// TheSuperHackers @bugfix Originally this movie render loop stopped rendering when the game window was inactive.
 		// This either skipped the movie or caused decompression artifacts. Now the video just keeps playing until it done.
 
-		Int progressUpdateCount = m_videoStream->frameCount() / FRAME_FUDGE_ADD;
-		Int shiftedPercent = -FRAME_FUDGE_ADD + 1;
-		while (m_videoStream->frameIndex() < m_videoStream->frameCount() - 1 )
+		// Hide the background window while the movie is playing
+		backgroundWin->winEnable(false);
+
+		// Show video progress bar
+		m_progressBar->winEnable(true);
+
+		// TheSuperHackers @info We now make use of movie playback within the display object
+		TheDisplay->playMovie(TheCampaignManager->getCurrentMission()->m_movieLabel);
+
+		while(TheDisplay->isMoviePlaying())
 		{
 			// TheSuperHackers @feature User can now skip video by pressing ESC
 			if (TheKeyboard)
@@ -548,51 +518,31 @@ void SinglePlayerLoadScreen::init( GameInfo *game )
 
 			TheGameEngine->serviceWindowsOS();
 
-			if(!m_videoStream->isFrameReady())
-			{
-				Sleep(1);
-				continue;
-			}
-
-			m_videoStream->frameDecompress();
-			m_videoStream->frameRender(m_videoBuffer);
-
-// PULLED FROM THE MISSION DISK
-//			moveWindows( m_videoStream->frameIndex());
-
-			m_videoStream->frameNext();
-
-			if(m_videoBuffer)
-				m_loadScreen->winGetInstanceData()->setVideoBuffer(m_videoBuffer);
-			if(m_videoStream->frameIndex() % progressUpdateCount == 0)
-			{
-				shiftedPercent++;
-				if(shiftedPercent >0)
-					shiftedPercent = 0;
-				Int percent = (shiftedPercent + FRAME_FUDGE_ADD)/1.3;
-				UnicodeString per;
-				per.format(L"%d%%",percent);
-				TheMouse->setCursorTooltip(UnicodeString::TheEmptyString);
-				GadgetProgressBarSetProgress(m_progressBar, percent);
-				GadgetStaticTextSetText(m_percent, per);
-
-			}
-			TheWindowManager->update();
+			// Update the video progress in the progress bar
+			Int percent = TheDisplay->getMovieProgress();
+			UnicodeString per;
+			per.format(L"%d%%",percent);
+			TheMouse->setCursorTooltip(UnicodeString::TheEmptyString);
+			GadgetProgressBarSetProgress(m_progressBar, percent);
+			GadgetStaticTextSetText(m_percent, per);
 
 			// redraw all views, update the GUI
+			TheWindowManager->update();
+			TheDisplay->update();
 			TheDisplay->draw();
 		}
 
 		// let the background image show through
-		m_videoStream->close();
-		m_videoStream = nullptr;
-		m_loadScreen->winGetInstanceData()->setVideoBuffer( nullptr );
+		TheDisplay->stopMovie();
+		backgroundWin->winEnable(true);
 		TheDisplay->draw();
 	}
 	else
 	{
 		// if we're min spec'ed don't play a movie
 
+		backgroundWin->winEnable(true);
+
 		Int delay = mission->m_voiceLength * 1000;
 		Int begin = timeGetTime();
 		Int currTime = begin;
@@ -613,6 +563,10 @@ void SinglePlayerLoadScreen::init( GameInfo *game )
 		TheDisplay->draw();
 
 	}
+
+	GadgetProgressBarSetProgress(m_progressBar, 0);
+	GadgetStaticTextSetText(m_percent, UnicodeString::TheEmptyString);
+
 	setFPMode();
 	m_percent->winHide(TRUE);
 	m_ambientLoopHandle = TheAudio->addAudioEvent(&m_ambientLoop);

GeneralsMD/Code/GameEngineDevice/Include/W3DDevice/GameClient/W3DDisplay.h

@@ -115,8 +115,9 @@ class W3DDisplay : public Display
 
 	/// draw a video buffer fit within the screen coordinates
 	virtual void drawScaledVideoBuffer( VideoBuffer *buffer, VideoStreamInterface *stream );
-	virtual void drawVideoBuffer( VideoBuffer *buffer, Int startX, Int startY,
-													Int endX, Int endY );
+	virtual void drawScaledVideoBuffer();
+	virtual void drawVideoBuffer( VideoBuffer *buffer, Int startX, Int startY, Int endX, Int endY );
+	virtual void drawVideoBuffer( Int startX, Int startY, Int endX, Int endY );
 
 	virtual VideoBuffer*	createVideoBuffer( void ) ;							///< Create a video buffer that can be used for this display
 

GeneralsMD/Code/GameEngineDevice/Source/W3DDevice/GameClient/GUI/W3DGameWindow.cpp

@@ -388,6 +388,16 @@ void W3DGameWinDefaultDraw( GameWindow *window, WinInstanceData *instData )
 		TheDisplay->drawVideoBuffer( instData->m_videoBuffer, pos.x, pos.y, pos.x + size.x, pos.y + size.y );
 	}
 
+	// if the display has a movie playing then we need to draw the displays video buffer
+	if (TheDisplay->isMoviePlaying())
+	{
+		ICoord2D pos, size;
+		window->winGetScreenPosition(&pos.x, &pos.y);
+		window->winGetSize(&size.x, &size.y);
+
+		TheDisplay->drawVideoBuffer(pos.x, pos.y, pos.x + size.x, pos.y + size.y);
+	}
+
 }
 
 // W3DGameWindow::winDrawBorder ===============================================

GeneralsMD/Code/GameEngineDevice/Source/W3DDevice/GameClient/W3DDisplay.cpp

@@ -2844,6 +2844,11 @@ void W3DDisplay::drawScaledVideoBuffer( VideoBuffer *buffer, VideoStreamInterfac
 	drawVideoBuffer( buffer, startX, startY, endX, endY );
 }
 
+void W3DDisplay::drawScaledVideoBuffer()
+{
+	drawScaledVideoBuffer(m_videoBuffer, m_videoStream);
+}
+
 //============================================================================
 // W3DDisplay::drawVideoBuffer
 //============================================================================
@@ -2861,6 +2866,11 @@ void W3DDisplay::drawVideoBuffer( VideoBuffer *buffer, Int startX, Int startY, I
 
 }
 
+void W3DDisplay::drawVideoBuffer( Int startX, Int startY, Int endX, Int endY )
+{
+	drawVideoBuffer(m_videoBuffer, startX, startY, endX, endY);
+}
+
 // W3DDisplay::setClipRegion ============================================
 /** Set the clipping region for images.
   @todo: Make this work for all primitives, not just drawImage. */

GeneralsMD/Code/Tools/GUIEdit/Include/GUIEditDisplay.h

@@ -98,9 +98,11 @@ class GUIEditDisplay : public Display
 	virtual VideoBuffer*	createVideoBuffer( void ) { return nullptr; }
 
 	/// draw a video buffer fit within the screen coordinates
-	virtual void drawScaledVideoBuffer( VideoBuffer *buffer, VideoStreamInterface *stream ) { }
-	virtual void drawVideoBuffer( VideoBuffer *buffer, Int startX, Int startY,
-																Int endX, Int endY ) { }
+	virtual void drawScaledVideoBuffer(VideoBuffer* buffer, VideoStreamInterface* stream) { }
+	virtual void drawScaledVideoBuffer() { }
+	virtual void drawVideoBuffer(VideoBuffer* buffer, Int startX, Int startY, Int endX, Int endY) { }
+	virtual void drawVideoBuffer(Int startX, Int startY, Int endX, Int endY) { }
+
 	virtual void takeScreenShot(void){ }
 	virtual void toggleMovieCapture(void) {}