don’t call actions from actions

Author: bridiver

app/renderer/reducers/frameReducer.js

@@ -57,7 +57,7 @@ const closeFrame = (state, action) => {
     // This allow us to have closeTab button visible for sequential frames closing,
     // until onMouseLeave event happens.
     if (hoverState) {
-      windowActions.setTabHoverState(nextFrame.get('key'), hoverState)
+      state = frameStateUtil.setTabHoverState(state, nextFrame.get('key'), hoverState)
     }
   }
 

js/state/frameStateUtil.js

@@ -580,7 +580,17 @@ const setPreviewFrameKey = (state, frameKey, immediate = false) => {
   return state.set('previewFrameKey', newPreviewFrameKey)
 }
 
+const setTabHoverState = (state, frameKey, hoverState) => {
+  const frameIndex = getFrameIndex(state, frameKey)
+  if (frameIndex !== -1) {
+    state = state.setIn(['frames', frameIndex, 'hoverState'], hoverState)
+    state = setPreviewFrameKey(state, frameKey)
+  }
+  return state
+}
+
 module.exports = {
+  setTabHoverState,
   setPreviewFrameKey,
   getPreviewFrameKey,
   deleteTabInternalIndex,

js/stores/windowStore.js

@@ -378,11 +378,7 @@ const doAction = (action) => {
       }
     case windowConstants.WINDOW_SET_TAB_HOVER_STATE:
       {
-        const frameIndex = frameStateUtil.getFrameIndex(windowState, action.frameKey)
-        if (frameIndex !== -1) {
-          windowState = windowState.setIn(['frames', frameIndex, 'hoverState'], action.hoverState)
-          windowState = frameStateUtil.setPreviewFrameKey(windowState, action.frameKey)
-        }
+        windowState = frameStateUtil.setTabHoverState(windowState, action.frameKey, action.hoverState)
         break
       }
     case windowConstants.WINDOW_TAB_MOVE: