Don't allow tab preview after close if config is off

Fix #9536
Auditors: @bsclifton

app/renderer/reducers/frameReducer.js

@@ -20,6 +20,8 @@ const frameStateUtil = require('../../../js/state/frameStateUtil')
 const {getCurrentWindowId} = require('../currentWindow')
 const {getSourceAboutUrl, getSourceMagnetUrl} = require('../../../js/lib/appUrlUtil')
 const {isURL, isPotentialPhishingUrl, getUrlFromInput} = require('../../../js/lib/urlutil')
+const settings = require('../../../js/constants/settings')
+const {getSetting} = require('../../../js/settings')
 
 const setFullScreen = (state, action) => {
   const index = frameStateUtil.getFrameIndex(state, action.frameProps.get('key'))
@@ -37,6 +39,7 @@ const closeFrame = (state, action) => {
 
   const frameProps = frameStateUtil.getFrameByKey(state, action.frameKey)
   const hoverState = state.getIn(['frames', index, 'hoverState'])
+  const framePreviewEnabled = getSetting(settings.SHOW_TAB_PREVIEWS)
 
   state = state.merge(frameStateUtil.removeFrame(
     state,
@@ -52,14 +55,14 @@ const closeFrame = (state, action) => {
 
   const nextFrame = frameStateUtil.getFrameByIndex(state, index)
 
-  if (nextFrame) {
-    // After closing a tab, preview the next frame as long as there is one
-    windowActions.setPreviewFrame(nextFrame.get('key'))
+  if (nextFrame && hoverState) {
     // Copy the hover state if tab closed with mouse as long as we have a next frame
     // This allow us to have closeTab button visible for sequential frames closing,
     // until onMouseLeave event happens.
-    if (hoverState) {
-      windowActions.setTabHoverState(nextFrame.get('key'), hoverState)
+    windowActions.setTabHoverState(nextFrame.get('key'), hoverState)
+    if (framePreviewEnabled && index !== nextFrame) {
+      // After closing a tab, preview the next frame as long as there is one
+      windowActions.setPreviewFrame(nextFrame.get('key'))
     }
   }