UrlBarIcon uses now only primitives

Resolves #9753

Auditors: @bsclifton @bridiver

Test Plan:

app/renderer/components/navigation/urlBarIcon.js

@@ -111,7 +111,9 @@ class UrlBarIcon extends React.Component {
 
   onDragStart (e) {
     dndData.setupDataTransferURL(e.dataTransfer, this.props.location, this.props.title)
-    dndData.setupDataTransferBraveData(e.dataTransfer, dragTypes.TAB, this.props.activeFrame)
+    dndData.setupDataTransferBraveData(e.dataTransfer, dragTypes.TAB, {
+      tabId: this.props.tabId
+    })
   }
 
   mergeProps (state, ownProps) {
@@ -138,7 +140,7 @@ class UrlBarIcon extends React.Component {
 
     // used in other functions
     props.title = activeFrame.get('title', '')
-    props.activeFrame = activeFrame // TODO (nejc) only primitives
+    props.tabId = activeFrame.get('tabId')
 
     return props
   }

app/renderer/reducers/frameReducer.js

@@ -10,6 +10,7 @@ const Immutable = require('immutable')
 const appConstants = require('../../../js/constants/appConstants')
 const windowConstants = require('../../../js/constants/windowConstants')
 const config = require('../../../js/constants/config')
+const siteTags = require('../../../js/constants/siteTags')
 
 // Actions
 const appActions = require('../../../js/actions/appActions')
@@ -19,6 +20,7 @@ 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 siteUtil = require('../../../js/state/siteUtil')
 
 const setFullScreen = (state, action) => {
   const index = frameStateUtil.getIndexByTabId(state, action.tabId)
@@ -205,6 +207,17 @@ const frameReducer = (state, action, immutableAction) => {
     case windowConstants.WINDOW_SET_FULL_SCREEN:
       state = setFullScreen(state, action)
       break
+
+    case windowConstants.WINDOW_ON_FRAME_BOOKMARK:
+      {
+        // TODO make this an appAction that gets the bookmark data from tabState
+        const frameProps = frameStateUtil.getFrameByTabId(state, action.tabId)
+        if (frameProps) {
+          const bookmark = siteUtil.getDetailFromFrame(frameProps, siteTags.BOOKMARK)
+          appActions.addSite(bookmark, siteTags.BOOKMARK)
+        }
+        break
+      }
   }
 
   return state

js/actions/windowActions.js

@@ -1149,6 +1149,13 @@ const windowActions = {
       actionType: windowConstants.WINDOW_ON_TAB_PAGE_CONTEXT_MENU,
       index
     })
+  },
+
+  onFrameBookmark: function (tabId) {
+    dispatch({
+      actionType: windowConstants.WINDOW_ON_FRAME_BOOKMARK,
+      tabId
+    })
   }
 }
 

js/constants/windowConstants.js

@@ -102,7 +102,8 @@ const windowConstants = {
   WINDOW_ON_GO_FORWARD_LONG: _,
   WINDOW_CLOSE_OTHER_FRAMES: _,
   WINDOW_ON_CERT_ERROR: _,
-  WINDOW_ON_TAB_PAGE_CONTEXT_MENU: _
+  WINDOW_ON_TAB_PAGE_CONTEXT_MENU: _,
+  WINDOW_ON_FRAME_BOOKMARK: _
 }
 
 module.exports = mapValuesByKeys(windowConstants)

js/dnd.js

@@ -7,8 +7,6 @@ const appActions = require('./actions/appActions')
 const ReactDOM = require('react-dom')
 const dndData = require('./dndData')
 const dragTypes = require('./constants/dragTypes')
-const siteTags = require('./constants/siteTags')
-const siteUtil = require('./state/siteUtil')
 const appStoreRenderer = require('./stores/appStoreRenderer')
 const {getCurrentWindowId} = require('../app/renderer/currentWindow')
 const {ESC} = require('../app/common/constants/keyCodes.js')
@@ -137,11 +135,8 @@ module.exports.isMiddle = (domNode, clientX) => {
 module.exports.prepareBookmarkDataFromCompatible = (dataTransfer) => {
   let bookmark = dndData.getDragData(dataTransfer, dragTypes.BOOKMARK)
   if (!bookmark) {
-    const frameProps = dndData.getDragData(dataTransfer, dragTypes.TAB)
-    if (frameProps) {
-      bookmark = siteUtil.getDetailFromFrame(frameProps, siteTags.BOOKMARK)
-      appActions.addSite(bookmark, siteTags.BOOKMARK)
-    }
+    const dragData = dndData.getDragData(dataTransfer, dragTypes.TAB)
+    windowActions.onFrameBookmark(dragData.get('tabId'))
   }
   return bookmark
 }

js/dndData.js

@@ -23,7 +23,7 @@ module.exports.setupDataTransferURL = (dataTransfer, location, title) => {
 }
 
 module.exports.setupDataTransferBraveData = (dataTransfer, dragType, data) => {
-  dataTransfer.setData(`application/x-brave-${dragType}`, JSON.stringify(data.toJS()))
+  dataTransfer.setData(`application/x-brave-${dragType}`, JSON.stringify(data))
 }
 
 module.exports.shouldPrependVerticalItem = (target, clientY) => {