fixed: only lock in CXBMCRenderManager::UpdateResolution() when the renderer has been reconfigured, saves a lock per render

(cherry picked from commit 1620c60d253b0bd91fb8c4915edc193c89f71ee8)

git-svn-id: https://xbmc.svn.sourceforge.net/svnroot/xbmc/branches/Dharma@34626 568bbfeb-2a22-0410-94d2-cc84cf5bfa90

2 files changed, 3 insertions, 2 deletions

diff --git a/xbmc/Application.cpp b/xbmc/Application.cpp
index 0332fc0beb..788c650dab 100644
--- a/xbmc/Application.cpp
+++ b/xbmc/Application.cpp
@@ -2039,9 +2039,10 @@ void CApplication::Render()
   g_graphicsContext.Flip();
   CTimeUtils::UpdateFrameTime();
   g_infoManager.UpdateFPS();
-  g_renderManager.UpdateResolution();
   g_graphicsContext.Unlock();
 
+  g_renderManager.UpdateResolution();
+
   // yield to other threads, so any thread needing
   // gfx context will get a timeslice. Newer os's
   // doesn't automatically prempt the unlocking thread
diff --git a/xbmc/cores/VideoRenderers/RenderManager.cpp b/xbmc/cores/VideoRenderers/RenderManager.cpp
index 7d39a28c8d..15a0be1c06 100644
--- a/xbmc/cores/VideoRenderers/RenderManager.cpp
+++ b/xbmc/cores/VideoRenderers/RenderManager.cpp
@@ -505,9 +505,9 @@ void CXBMCRenderManager::Recover()
 
 void CXBMCRenderManager::UpdateResolution()
 {
-  CRetakeLock<CExclusiveLock> lock(m_sharedSection);
   if (m_bReconfigured)
   {
+    CRetakeLock<CExclusiveLock> lock(m_sharedSection);
     if (g_graphicsContext.IsFullScreenVideo() && g_graphicsContext.IsFullScreenRoot())
     {
       RESOLUTION res = GetResolution();