Use addBookmark instead of addSite for bookmarks

Fix brave#11033

We currently don't have DND under automated so no extra test coverage.

The js/dnd change is if you drag a bookmark from Chrome to Brave it was
erroring with a JS error before instead of creating the bookmark.  Now
it works.

Auditors: @cezaraugusto

app/renderer/components/bookmarks/bookmarksToolbar.js

@@ -72,7 +72,7 @@ class BookmarksToolbar extends React.Component {
       const doc = parser.parseFromString(droppedHTML, 'text/html')
       const a = doc.querySelector('a')
       if (a && a.href) {
-        appActions.addSite({
+        appActions.addBookmark({
           title: a.innerText,
           location: e.dataTransfer.getData('text/plain')
         }, siteTags.BOOKMARK)
@@ -82,7 +82,7 @@ class BookmarksToolbar extends React.Component {
 
     if (e.dataTransfer.files.length > 0) {
       Array.from(e.dataTransfer.items).forEach((item) => {
-        item.getAsString((name) => appActions.addSite({ location: item.type, title: name }, siteTags.BOOKMARK))
+        item.getAsString((name) => appActions.addBookmark({ location: item.type, title: name }, siteTags.BOOKMARK))
       })
       return
     }
@@ -92,7 +92,7 @@ class BookmarksToolbar extends React.Component {
       .map((x) => x.trim())
       .filter((x) => !x.startsWith('#') && x.length > 0)
       .forEach((url) =>
-        appActions.addSite({ location: url }, siteTags.BOOKMARK))
+        appActions.addBookmark({ location: url }, siteTags.BOOKMARK))
   }
 
   onDragEnter (e) {

js/dnd.js

@@ -136,7 +136,9 @@ module.exports.prepareBookmarkDataFromCompatible = (dataTransfer) => {
   let bookmark = dndData.getDragData(dataTransfer, dragTypes.BOOKMARK)
   if (!bookmark) {
     const dragData = dndData.getDragData(dataTransfer, dragTypes.TAB)
-    windowActions.onFrameBookmark(dragData.get('tabId'))
+    if (dragData) {
+      windowActions.onFrameBookmark(dragData.get('tabId'))
+    }
   }
   return bookmark
 }