Call WindowBase::onFrame() for every active window

apps/openmw/mwgui/companionwindow.cpp

@@ -116,8 +116,9 @@ void CompanionWindow::setPtr(const MWWorld::Ptr& npc)
     setTitle(npc.getClass().getName(npc));
 }
 
-void CompanionWindow::onFrame()
+void CompanionWindow::onFrame(float dt)
 {
+    checkReferenceAvailable();
     updateEncumbranceBar();
 }
 

apps/openmw/mwgui/companionwindow.hpp

@@ -23,7 +23,7 @@ namespace MWGui
         virtual void resetReference();
 
         void setPtr(const MWWorld::Ptr& npc);
-        void onFrame ();
+        void onFrame (float dt);
 
     private:
         ItemView* mItemView;

apps/openmw/mwgui/console.hpp

@@ -42,6 +42,8 @@ namespace MWGui
             virtual void onOpen();
             virtual void onClose();
 
+            void onFrame(float dt) { checkReferenceAvailable(); }
+
             void setFont(const std::string &fntName);
 
             void onResChange(int width, int height);

apps/openmw/mwgui/container.hpp

@@ -36,6 +36,8 @@ namespace MWGui
         void setPtr(const MWWorld::Ptr& container);
         virtual void onClose();
 
+        void onFrame(float dt) { checkReferenceAvailable(); }
+
         virtual void resetReference();
 
     private:

apps/openmw/mwgui/dialogue.cpp

@@ -630,9 +630,10 @@ namespace MWGui
         MWBase::Environment::get().getWindowManager()->removeGuiMode(GM_Dialogue);
     }
 
-    void DialogueWindow::onFrame()
+    void DialogueWindow::onFrame(float dt)
     {
-        if(mMainWidget->getVisible() && mPtr.getTypeName() == typeid(ESM::NPC).name())
+        checkReferenceAvailable();
+        if(mPtr.getTypeName() == typeid(ESM::NPC).name())
         {
             int disp = MWBase::Environment::get().getMechanicsManager()->getDerivedDisposition(mPtr);
             mDispositionBar->setProgressRange(100);

apps/openmw/mwgui/dialogue.hpp

@@ -117,7 +117,7 @@ namespace MWGui
         void clearChoices();
 
         void goodbye();
-        void onFrame();
+        void onFrame(float dt);
 
         // make sure to call these before setKeywords()
         void setServices(int services) { mServices = services; }

apps/openmw/mwgui/enchantingdialog.hpp

@@ -21,6 +21,8 @@ namespace MWGui
 
         virtual void onOpen();
 
+        void onFrame(float dt) { checkReferenceAvailable(); }
+
         void setSoulGem (const MWWorld::Ptr& gem);
         void setItem (const MWWorld::Ptr& item);
 

apps/openmw/mwgui/inventorywindow.cpp

@@ -589,11 +589,8 @@ namespace MWGui
         mEncumbranceBar->setValue(static_cast<int>(encumbrance), static_cast<int>(capacity));
     }
 
-    void InventoryWindow::onFrame()
+    void InventoryWindow::onFrame(float dt)
     {
-        if (!mMainWidget->getVisible())
-            return;
-
         updateEncumbranceBar();
     }
 

apps/openmw/mwgui/inventorywindow.hpp

@@ -40,7 +40,7 @@ namespace MWGui
             /// start trading, disables item drag&drop
             void setTrading(bool trading);
 
-            void onFrame();
+            void onFrame(float dt);
 
             void pickUpObject (MWWorld::Ptr object);
 

apps/openmw/mwgui/mainmenu.cpp

@@ -197,7 +197,7 @@ namespace MWGui
         }
     }
 
-    void MainMenu::update(float dt)
+    void MainMenu::onFrame(float dt)
     {
         if (mVideo)
         {

apps/openmw/mwgui/mainmenu.hpp

@@ -36,7 +36,7 @@ namespace MWGui
 
             virtual void setVisible (bool visible);
 
-            void update(float dt);
+            void onFrame(float dt);
 
         private:
             const VFS::Manager* mVFS;

apps/openmw/mwgui/spellbuyingwindow.hpp

@@ -28,6 +28,8 @@ namespace MWGui
             void setPtr(const MWWorld::Ptr& actor);
             void setPtr(const MWWorld::Ptr& actor, int startOffset);
 
+            void onFrame(float dt) { checkReferenceAvailable(); }
+
         protected:
             MyGUI::Button* mCancelButton;
             MyGUI::TextBox* mPlayerGold;

apps/openmw/mwgui/spellcreationdialog.hpp

@@ -152,6 +152,8 @@ namespace MWGui
 
         virtual void onOpen();
 
+        void onFrame(float dt) { checkReferenceAvailable(); }
+
         void setPtr(const MWWorld::Ptr& actor);
 
     protected:

apps/openmw/mwgui/spellwindow.cpp

@@ -69,15 +69,12 @@ namespace MWGui
 
     void SpellWindow::onFrame(float dt) 
     {
-        if (mMainWidget->getVisible())
+        NoDrop::onFrame(dt);
+        mUpdateTimer += dt;
+        if (0.5f < mUpdateTimer)
         {
-            NoDrop::onFrame(dt);
+            mUpdateTimer = 0;
-            mUpdateTimer += dt;
+            mSpellView->incrementalUpdate();
-            if (0.5f < mUpdateTimer)
-            {
-                mUpdateTimer = 0;
-                mSpellView->incrementalUpdate();
-            }
         }
     }
 

apps/openmw/mwgui/statswindow.cpp

@@ -297,9 +297,6 @@ namespace MWGui
 
     void StatsWindow::onFrame (float dt)
     {
-        if (!mMainWidget->getVisible())
-            return;
-
         NoDrop::onFrame(dt);
 
         MWWorld::Ptr player = MWMechanics::getPlayer();

apps/openmw/mwgui/tradewindow.cpp

@@ -142,6 +142,11 @@ namespace MWGui
         onFilterChanged(mFilterAll);
     }
 
+    void TradeWindow::onFrame(float dt)
+    {
+        checkReferenceAvailable();
+    }
+
     void TradeWindow::onFilterChanged(MyGUI::Widget* _sender)
     {
         if (_sender == mFilterAll)

apps/openmw/mwgui/tradewindow.hpp

@@ -29,6 +29,8 @@ namespace MWGui
 
             void setPtr(const MWWorld::Ptr& actor);
 
+            void onFrame(float dt);
+
             void borrowItem (int index, size_t count);
             void returnItem (int index, size_t count);
 

apps/openmw/mwgui/trainingwindow.cpp

@@ -195,6 +195,7 @@ namespace MWGui
 
     void TrainingWindow::onFrame(float dt)
     {
+        checkReferenceAvailable();
         mTimeAdvancer.onFrame(dt);
 
         if (mFadeTimeRemaining <= 0)

apps/openmw/mwgui/windowbase.hpp

@@ -29,6 +29,9 @@ namespace MWGui
         /// Open this object in the GUI, for windows that support it
         virtual void setPtr(const MWWorld::Ptr& ptr) {}
 
+        /// Called every frame if the window is in an active GUI mode
+        virtual void onFrame(float duration) {}
+
         /// Notify that window has been made visible
         virtual void onOpen() {}
         /// Notify that window has been hidden

apps/openmw/mwgui/windowmanagerimp.cpp

@@ -890,12 +890,19 @@ namespace MWGui
 
     void WindowManager::onFrame (float frameDuration)
     {
+        if (!mGuiModes.empty())
+        {
+            GuiModeState& state = mGuiModeStates[mGuiModes.back()];
+            for (WindowBase* window : state.mWindows)
+                window->onFrame(frameDuration);
+        }
+        if (!mCurrentModals.empty())
+            mCurrentModals.top()->onFrame(frameDuration);
+
         mMessageBoxManager->onFrame(frameDuration);
 
         mToolTips->onFrame(frameDuration);
 
-        mMenu->update(frameDuration);
-
         if (mLocalMapRender)
             mLocalMapRender->cleanupCameras();
 
@@ -909,34 +916,10 @@ namespace MWGui
             mDragAndDrop->mDraggedWidget->setPosition(MyGUI::InputManager::getInstance().getMousePosition());
         }
 
-        mDialogueWindow->onFrame();
-
-        mInventoryWindow->onFrame();
-
         updateMap();
 
-        mStatsWindow->onFrame(frameDuration);
-        mMap->onFrame(frameDuration);
-        mSpellWindow->onFrame(frameDuration);
-
-        mWaitDialog->onFrame(frameDuration);
-
         mHud->onFrame(frameDuration);
 
-        mTrainingWindow->onFrame (frameDuration);
-
-        mTrainingWindow->checkReferenceAvailable();
-        mDialogueWindow->checkReferenceAvailable();
-        mTradeWindow->checkReferenceAvailable();
-        mSpellBuyingWindow->checkReferenceAvailable();
-        mSpellCreationDialog->checkReferenceAvailable();
-        mEnchantingDialog->checkReferenceAvailable();
-        mContainerWindow->checkReferenceAvailable();
-        mCompanionWindow->checkReferenceAvailable();
-        mConsole->checkReferenceAvailable();
-        mCompanionWindow->onFrame();
-        mJailScreen->onFrame(frameDuration);
-
         if (mWerewolfFader)
             mWerewolfFader->update(frameDuration);
         mBlindnessFader->update(frameDuration);