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

…tart fix brave#2543
darkdh · Jul 17, 2016 · 897dddc · 897dddc
1 parent 1d15bfe
commit 897dddc
Show file tree

Hide file tree

Showing 5 changed files with 35 additions and 2 deletions.
diff --git a/app/extensions/brave/locales/en-US/app.properties b/app/extensions/brave/locales/en-US/app.properties
@@ -149,3 +149,4 @@ 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 ?
diff --git a/app/index.js b/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: ['Yes', '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)
     })
@@ -633,10 +655,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

diff --git a/app/locale.js b/app/locale.js
@@ -165,7 +165,8 @@ var rendererIdentifiers = function () {
     'historySuggestionTitle',
     'aboutPagesSuggestionTitle',
     'searchSuggestionTitle',
-    'topSiteSuggestionTitle'
+    'topSiteSuggestionTitle',
+    'prefsRestart'
   ]
 }
 

diff --git a/js/about/preferences.js b/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 () {

diff --git a/js/constants/messages.js b/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 */