prefs that require browser restart should show prompt for browser res…

…tart

fix #2543

app/extensions/brave/locales/en-US/app.properties

@@ -149,3 +149,7 @@ flashTitle=Flash Object Blocked
 flashRightClick=Right-click to run Adobe Flash
 flashSubtext=from {{source}} on {{site}}.
 flashExpirationText=Approvals reset 7 days after last visit.
+prefsRestart=Do you want to restart now?
+yes=Yes
+no=No
+neverForThisSite=Never for this site

app/index.js

@@ -63,6 +63,8 @@ let throttleKeytar = false
 // Map of password notification bar messages to their callbacks
 const passwordCallbacks = {}
 
+const prefsRestartCallbacks = {}
+
 /**
  * Gets the master key for encrypting login credentials from the OS keyring.
  */
@@ -381,6 +383,26 @@ app.on('ready', () => {
       app.quit()
     })
 
+    ipcMain.on(messages.PREFS_RESTART, () => {
+      var message = locale.translation('prefsRestart')
+
+      appActions.showMessageBox({
+        buttons: [locale.translation('yes'), locale.translation('no')],
+        options: {
+          persist: false
+        },
+        message
+      })
+      prefsRestartCallbacks[message] = (buttonIndex, persist) => {
+        if (buttonIndex === 0) {
+          app.relaunch({args: process.argv.slice(1) + ['--relaunch']})
+          app.exit(0)
+        } else {
+          appActions.hideMessageBox(message)
+        }
+      }
+    })
+
     ipcMain.on(messages.UPDATE_APP_MENU, (e, args) => {
       Menu.init(AppStore.getState().get('settings'), args)
     })
@@ -597,7 +619,11 @@ app.on('ready', () => {
       if (!(message in passwordCallbacks)) {
         // Notification not shown already
         appActions.showMessageBox({
-          buttons: ['Yes', 'No', 'Never for this site'],
+          buttons: [
+            locale.translation('yes'),
+            locale.translation('no'),
+            locale.translation('neverForThisSite')
+          ],
           options: {
             persist: false,
             advancedText: '[Password settings]',
@@ -633,10 +659,13 @@ app.on('ready', () => {
       }
     })
 
-    ipcMain.on(messages.NOTIFICATION_RESPONSE, (e, message, buttonIndex) => {
+    ipcMain.on(messages.NOTIFICATION_RESPONSE, (e, message, buttonIndex, persist) => {
       if (passwordCallbacks[message]) {
         passwordCallbacks[message](buttonIndex)
       }
+      if (prefsRestartCallbacks[message]) {
+        prefsRestartCallbacks[message](buttonIndex, persist)
+      }
     })
 
     // Setup the crash handling

app/locale.js

@@ -165,7 +165,11 @@ var rendererIdentifiers = function () {
     'historySuggestionTitle',
     'aboutPagesSuggestionTitle',
     'searchSuggestionTitle',
-    'topSiteSuggestionTitle'
+    'topSiteSuggestionTitle',
+    'prefsRestart',
+    'yes',
+    'no',
+    'neverForThisSite'
   ]
 }
 

js/about/preferences.js

@@ -357,6 +357,7 @@ class ShieldsTab extends ImmutableComponent {
 class SecurityTab extends ImmutableComponent {
   onToggleFlash (e) {
     aboutActions.setResourceEnabled(flash, e.target.value)
+    ipc.send(messages.PREFS_RESTART)
   }
   render () {
     return <div>
@@ -591,6 +592,10 @@ class AboutPreferences extends React.Component {
       settings: this.state.settings.set(key, value)
     })
     aboutActions.changeSetting(key, value)
+    if (key === settings.DO_NOT_TRACK || key === settings.HARDWARE_ACCELERATION_ENABLED ||
+      key === settings.PDFJS_ENABLED) {
+      ipc.send(messages.PREFS_RESTART)
+    }
   }
 
   render () {

js/constants/messages.js

@@ -41,6 +41,7 @@ const messages = {
   SHORTCUT_PREV_TAB: _,
   // Misc application events
   QUIT_APPLICATION: _,
+  PREFS_RESTART: _,
   UPDATE_APP_MENU: _, /** @arg {Object} args menu args to update */
   CERT_ERROR: _, /** @arg {Object} details of certificate error */
   LOGIN_REQUIRED: _, /** @arg {Object} details of the login required request */