 Chromium Code Reviews
 Chromium Code Reviews Issue 1737413002:
  Modified the behavior of "Hide Toolbar in Full Screen"  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master
    
  
    Issue 1737413002:
  Modified the behavior of "Hide Toolbar in Full Screen"  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master| 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..09f721d9bff09b6285ed2a56cf5d9ce3eb81d495 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" | 
| 
msw
2016/03/01 19:23:02
nit: remove
 
spqchan
2016/03/01 23:41:31
Done.
 | 
| #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" | 
| 
msw
2016/03/01 19:23:02
nit: remove
 
spqchan
2016/03/01 23:41:31
Done.
 | 
| #include "chrome/browser/ui/exclusive_access/fullscreen_controller.h" | 
| #include "chrome/common/chrome_switches.h" | 
| 
msw
2016/03/01 19:23:02
nit: remove
 
spqchan
2016/03/01 23:41:31
Done.
 | 
| +#include "chrome/common/pref_names.h" | 
| +#include "components/prefs/pref_service.h" | 
| namespace chrome { | 
| @@ -26,7 +29,10 @@ void ToggleFullscreenWithToolbarOrFallback(Browser* browser) { | 
| void ToggleFullscreenToolbar(Browser* browser) { | 
| DCHECK(browser); | 
| - browser->exclusive_access_manager()->context()->ToggleFullscreenToolbar(); | 
| + // Toggle the value of the preference. | 
| + PrefService* prefs = browser->profile()->GetPrefs(); | 
| + bool show_toolbar = prefs->GetBoolean(prefs::kShowFullscreenToolbar); | 
| + prefs->SetBoolean(prefs::kShowFullscreenToolbar, !show_toolbar); | 
| } | 
| } // namespace chrome |