fix #8374 - Open image in new tab should recognize focus prefs

It causes a crash on linux FYI: @bridiver
brave · Apr 26, 2017 · 687bfc3 · 687bfc3
1 parent 576921b
commit 687bfc3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/js/contextMenus.js b/js/contextMenus.js
@@ -1027,6 +1027,7 @@ function mainTemplateInit (nodeProps, frame, tab) {
   }
 
   if (isImage) {
+    const active = getSetting(settings.SWITCH_TO_NEW_TABS) === true
     template.push(
       {
         label: locale.translation('openImageInNewTab'),
@@ -1035,7 +1036,8 @@ function mainTemplateInit (nodeProps, frame, tab) {
             appActions.createTabRequested({
               url: nodeProps.srcURL,
               openerTabId: frame.get('tabId'),
-              partition: getPartitionFromNumber(frame.get('partitionNumber'), isPrivate)
+              partition: getPartitionFromNumber(frame.get('partitionNumber'), isPrivate),
+              active: active
             })
           }
         }

diff --git a/test/lib/brave.js b/test/lib/brave.js
@@ -970,7 +970,7 @@ var exports = {
         ? 'node_modules/electron-prebuilt/dist/brave.exe'
         : './node_modules/.bin/electron',
       env,
-      args: ['./', '--debug=5858', '--enable-logging', '--v=1'],
+      args: ['./', '--enable-logging', '--v=1'],
       requireName: 'devTools'
     })
     return this.app.start()