Modified the behavior of "Hide Toolbar in Full Screen"

chrome/browser/ui/browser_commands_mac.cc

Index: chrome/browser/ui/browser_commands_mac.cc
diff --git a/chrome/browser/ui/browser_commands_mac.cc b/chrome/browser/ui/browser_commands_mac.cc
index b6fb858b044b48dcdd79ae225c9c0afc782b0cdd..b83711a9725b7558d3abe879651c2dd363dbef40 100644
--- a/chrome/browser/ui/browser_commands_mac.cc
+++ b/chrome/browser/ui/browser_commands_mac.cc
@@ -6,11 +6,14 @@
 
 #include "base/command_line.h"
 #include "chrome/browser/fullscreen.h"
+#include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_commands.h"
 #include "chrome/browser/ui/exclusive_access/exclusive_access_context.h"
 #include "chrome/browser/ui/exclusive_access/fullscreen_controller.h"
 #include "chrome/common/chrome_switches.h"
+#include "chrome/common/pref_names.h"
+#include "components/prefs/pref_service.h"
 
 namespace chrome {
 
@@ -26,7 +29,9 @@ void ToggleFullscreenWithToolbarOrFallback(Browser* browser) {
 
 void ToggleFullscreenToolbar(Browser* browser) {
   DCHECK(browser);
-  browser->exclusive_access_manager()->context()->ToggleFullscreenToolbar();
+  PrefService* prefs = browser->profile()->GetPrefs();
+  bool show_toolbar = !prefs->GetBoolean(prefs::kShowFullscreenToolbar);

[msw, 2016/03/01 00:07:15]

This '!' inversion is confusing... did you mean to get kHideFullscreenToolbar
instead? or is the inversion intended? can you add a comment or explain?

[spqchan, 2016/03/01 17:50:18]

Done.

+  prefs->SetBoolean(prefs::kShowFullscreenToolbar, show_toolbar);
 }
 
 }  // namespace chrome