Provide a link to restart chrome when changing gpu settings

chrome/browser/ui/webui/options/browser_options_handler.cc

Index: chrome/browser/ui/webui/options/browser_options_handler.cc
===================================================================
--- chrome/browser/ui/webui/options/browser_options_handler.cc	(revision 189066)
+++ chrome/browser/ui/webui/options/browser_options_handler.cc	(working copy)
@@ -24,6 +24,7 @@
 #include "chrome/browser/chrome_page_zoom.h"
 #include "chrome/browser/custom_home_pages_table_model.h"
 #include "chrome/browser/download/download_prefs.h"
+#include "chrome/browser/lifetime/application_lifetime.h"
 #include "chrome/browser/net/url_fixer_upper.h"
 #include "chrome/browser/prefs/session_startup_pref.h"
 #include "chrome/browser/printing/cloud_print/cloud_print_proxy_service.h"
@@ -366,6 +367,12 @@
 #endif
     { "hardwareAccelerationModeCheckbox",
       IDS_OPTIONS_SYSTEM_ENABLE_HARDWARE_ACCELERATION_MODE },
+    { "hardwareAccelerationModeChangeRestartPrefix",
+      IDS_OPTIONS_SYSTEM_ENABLE_HARDWARE_ACCELERATION_MODE_RESTART_PREFIX },
+    { "hardwareAccelerationModeChangeRestart",
+      IDS_OPTIONS_SYSTEM_ENABLE_HARDWARE_ACCELERATION_MODE_RESTART },
+    { "hardwareAccelerationModeChangeRestartPostfix",
+      IDS_OPTIONS_SYSTEM_ENABLE_HARDWARE_ACCELERATION_MODE_RESTART_POSTFIX },

[Dan Beam, 2013/03/20 22:17:55]

shouldn't these be in the

  #if !defined(OS_CHROMEOS)

section?

[Zhenyao Mo, 2013/03/20 23:21:49]

Done.

     // Strings with product-name substitutions.
 #if !defined(OS_CHROMEOS)
     { "syncOverview", IDS_SYNC_OVERVIEW, IDS_PRODUCT_NAME },
@@ -602,6 +609,10 @@
       base::Bind(&BrowserOptionsHandler::PerformFactoryResetRestart,
                  base::Unretained(this)));
 #endif
+  web_ui()->RegisterMessageCallback(
+      "restartBrowser",
+      base::Bind(&BrowserOptionsHandler::HandleRestartBrowser,
+                 base::Unretained(this)));

[Dan Beam, 2013/03/20 22:17:55]

^ shouldn't this be defined only in

  #if !defined(OS_CHROMEOS)

as well?

[Zhenyao Mo, 2013/03/20 23:21:49]

Done.

 }
 
 void BrowserOptionsHandler::OnStateChanged() {
@@ -1232,6 +1243,10 @@
   }
 }
 
+void BrowserOptionsHandler::HandleRestartBrowser(const ListValue* args) {
+  chrome::AttemptRestart();
+}
+
 #if !defined(OS_CHROMEOS)
 void BrowserOptionsHandler::ShowNetworkProxySettings(const ListValue* args) {
   content::RecordAction(UserMetricsAction("Options_ShowProxySettings"));