Move printing.* prefs from local state to profile

chrome/browser/ui/browser_commands.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/browser_commands.h"
 
 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/bookmarks/bookmark_editor.h"
@@ skipping 635 matching lines @@
 }
 
 void ShowChromeToMobileBubble(Browser* browser) {
   // Only show the bubble if the window is active, otherwise we may get into
   // weird situations where the bubble is deleted as soon as it is shown.
   if (browser->window()->IsActive())
     browser->window()->ShowChromeToMobileBubble();
 }
 
 void Print(Browser* browser) {
-  if (g_browser_process->local_state()->GetBoolean(
-          prefs::kPrintPreviewDisabled)) {
+  if (browser->profile()->GetPrefs()->GetBoolean(
+      prefs::kPrintPreviewDisabled)) {
     GetActiveTabContents(browser)->print_view_manager()->PrintNow();
   } else {
     GetActiveTabContents(browser)->print_view_manager()->
         PrintPreviewNow();
   }
 }
 
 bool CanPrint(const Browser* browser) {
-  // LocalState can be NULL in tests.
-  if (g_browser_process->local_state() &&
-      !g_browser_process->local_state()->GetBoolean(prefs::kPrintingEnabled)) {
-    return false;
-  }
-
+  // Do not print when printing is disabled via pref or policy.
   // Do not print when a constrained window is showing. It's confusing.
   // Do not print if instant extended API is enabled and mode is NTP.
-  return !(HasConstrainedWindow(browser) ||
+  return browser->profile()->GetPrefs()->GetBoolean(prefs::kPrintingEnabled) &&
+      !(HasConstrainedWindow(browser) ||
       GetContentRestrictions(browser) & content::CONTENT_RESTRICTION_PRINT ||
       IsNTPModeForInstantExtendedAPI(browser));
 }
 
 void AdvancedPrint(Browser* browser) {
   GetActiveTabContents(browser)->print_view_manager()->
       AdvancedPrintNow();
 }
 
 bool CanAdvancedPrint(const Browser* browser) {
-  // LocalState can be NULL in tests.
-  if (g_browser_process->local_state() &&
-      !g_browser_process->local_state()->GetBoolean(prefs::kPrintingEnabled)) {
-    return false;
-  }
-
-  // It is always possible to advanced print when print preview is visible.
-  return PrintPreviewShowing(browser) || CanPrint(browser);
+  // If printing is not disabled via pref or policy, it is always possible to
+  // advanced print when the print preview is visible.
+  return browser->profile()->GetPrefs()->GetBoolean(prefs::kPrintingEnabled) &&
+      (PrintPreviewShowing(browser) || CanPrint(browser));
 }
 
 void PrintToDestination(Browser* browser) {
   GetActiveTabContents(browser)->print_view_manager()->PrintToDestination();
 }
 
 void EmailPageLocation(Browser* browser) {
   content::RecordAction(UserMetricsAction("EmailPageLocation"));
   WebContents* wc = GetActiveWebContents(browser);
   DCHECK(wc);
@@ skipping 280 matching lines @@
   if (!tab_contents)
     tab_contents = new TabContents(contents);
   app_browser->tab_strip_model()->AppendTabContents(tab_contents, true);
 
   contents->GetMutableRendererPrefs()->can_accept_load_drops = false;
   contents->GetRenderViewHost()->SyncRendererPrefs();
   app_browser->window()->Show();
 }
 
 }  // namespace chrome