Refactors window actions, remove the whole frame prop (related to tabs)

Resolves brave#8857

Auditors: @bsclifton @bridiver

Test Plan:
- try closing a tab

app/renderer/reducers/frameReducer.js

@@ -24,7 +24,7 @@ const setFullScreen = (state, action) => {
 
 const closeFrame = (state, action) => {
   // Use the frameProps we passed in, or default to the active frame
-  const frameProps = action.frameProps
+  const frameProps = frameStateUtil.getFrameByKey(state, action.frameKey)
   const index = frameStateUtil.getFramePropsIndex(state.get('frames'), frameProps)
   if (index === -1) {
     return state
@@ -75,7 +75,7 @@ const frameReducer = (state, action) => {
       })
       break
     case windowConstants.WINDOW_CLOSE_FRAME:
-      if (!action.frameProps) {
+      if (action.frameKey < 0) {
         break
       }
       // Unless a caller explicitly specifies to close a pinned frame, then

js/actions/windowActions.js

@@ -210,13 +210,12 @@ const windowActions = {
   /**
    * Dispatches a message to close a frame
    *
-   * @param {Object[]} frames - Immutable list of of all the frames
-   * @param {Object} frameProps - The properties of the frame to close
+   * @param {Object} frameKey - Frame key of the frame to close
    */
-  closeFrame: function (frameProps) {
+  closeFrame: function (frameKey) {
     dispatch({
       actionType: windowConstants.WINDOW_CLOSE_FRAME,
-      frameProps
+      frameKey
     })
   },
 

js/components/frame.js

@@ -91,7 +91,7 @@ class Frame extends React.Component {
   }
 
   onCloseFrame () {
-    windowActions.closeFrame(this.frame)
+    windowActions.closeFrame(this.props.frameKey)
   }
 
   getFrameBraverySettings (props) {