refactor(loadscreen): Use TheDisplay for video playback of single player intro videos

Author: Mauller

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

@@ -151,6 +151,7 @@ class Display : public SubsystemInterface
 	virtual void playLogoMovie( AsciiString movieName, Int minMovieLength, Int minCopyrightLength );
 	virtual void playMovie( AsciiString movieName );
 	virtual void stopMovie( void );
+	virtual Real getMovieProgress(); ///< returns the playback progress in the range 0.0 to 1.0
 	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,16 @@ void Display::reset()
 		v->reset();
 }
 
+//============================================================================
+// Display::getMovieProgress
+//============================================================================
+
+Real Display::getMovieProgress()
+{
+	return m_videoStream && m_videoStream->frameCount() > 0 ?
+		(Real)m_videoStream->frameIndex() / (Real)m_videoStream->frameCount() : 0.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 percentValue = TheDisplay->getMovieProgress() * 100.0f;
+			UnicodeString percentText;
+			percentText.format(L"%d%%", percentValue);
+			TheMouse->setCursorTooltip(UnicodeString::TheEmptyString);
+			GadgetProgressBarSetProgress(m_progressBar, percentValue);
+			GadgetStaticTextSetText(m_percent, percentText);
 
 			// 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/Source/W3DDevice/GameClient/GUI/W3DGameWindow.cpp

@@ -379,6 +379,7 @@ void W3DGameWinDefaultDraw( GameWindow *window, WinInstanceData *instData )
 	}
 
 	// if we have a video buffer, draw the video buffer
+	// else if the display has a movie playing then we need to draw the displays video buffer
 	if ( instData->m_videoBuffer )
 	{
 		ICoord2D pos, size;
@@ -387,6 +388,14 @@ void W3DGameWinDefaultDraw( GameWindow *window, WinInstanceData *instData )
 
 		TheDisplay->drawVideoBuffer( instData->m_videoBuffer, pos.x, pos.y, pos.x + size.x, pos.y + size.y );
 	}
+	else 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);
+	}
 
 }