Use browser process for closing tabs for tab page close

Fix #11028 Auditors: NejcZdovc
brave · Sep 22, 2017 · aadb3bb · aadb3bb
1 parent 0c0524c
commit aadb3bb
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/app/renderer/reducers/contextMenuReducer.js b/app/renderer/reducers/contextMenuReducer.js
@@ -95,7 +95,9 @@ const onTabPageMenu = function (state, action) {
   }, {
     label: locale.translation('closeTabPage'),
     click: () => {
-      windowActions.closeFrames(tabPageFrames)
+      tabPageFrames
+        .map((frame) => frame.get('tabId'))
+        .forEach((tabId) => appActions.tabCloseRequested(tabId))
     }
   }]
 

diff --git a/app/renderer/reducers/frameReducer.js b/app/renderer/reducers/frameReducer.js
@@ -239,6 +239,8 @@ const frameReducer = (state, action, immutableAction) => {
 
     case windowConstants.WINDOW_CLOSE_FRAME:
       state = closeFrame(state, action)
+      const activeFrame = frameStateUtil.getActiveFrame(state)
+      state = frameStateUtil.updateTabPageIndex(state, activeFrame.get('tabId'))
       break
 
     case windowConstants.WINDOW_SET_FULL_SCREEN: