Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2310)

Unified Diff: chrome/browser/printing/print_job_manager.cc

Issue 10808006: Move printing.* prefs from local state to profile (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased just in case as the CL is a few days old now plus I got my branches all mixed up a bit. Created 8 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/printing/print_job_manager.cc
diff --git a/chrome/browser/printing/print_job_manager.cc b/chrome/browser/printing/print_job_manager.cc
index e3abfd103604c2e07c4f95666a95f91000cdf60e..154f735423d20a45a83b596f42edd3c5aade4fb2 100644
--- a/chrome/browser/printing/print_job_manager.cc
+++ b/chrome/browser/printing/print_job_manager.cc
@@ -4,18 +4,13 @@
#include "chrome/browser/printing/print_job_manager.h"
-#include "chrome/browser/prefs/pref_service.h"
#include "chrome/browser/printing/print_job.h"
#include "chrome/browser/printing/printer_query.h"
#include "chrome/common/chrome_notification_types.h"
-#include "chrome/common/pref_names.h"
-#include "content/public/browser/browser_thread.h"
#include "content/public/browser/notification_service.h"
#include "printing/printed_document.h"
#include "printing/printed_page.h"
-using content::BrowserThread;
-
namespace printing {
PrintJobManager::PrintJobManager() {
@@ -28,12 +23,6 @@ PrintJobManager::~PrintJobManager() {
queued_queries_.clear();
}
-void PrintJobManager::InitOnUIThread(PrefService* prefs) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- printing_enabled_.Init(prefs::kPrintingEnabled, prefs, NULL);
- printing_enabled_.MoveToThread(BrowserThread::IO);
-}
-
void PrintJobManager::OnQuit() {
StopJobs(true);
registrar_.RemoveAll();
@@ -88,11 +77,6 @@ void PrintJobManager::PopPrinterQuery(int document_cookie,
}
}
-// static
-void PrintJobManager::RegisterPrefs(PrefService* prefs) {
- prefs->RegisterBooleanPref(prefs::kPrintingEnabled, true);
-}
-
void PrintJobManager::Observe(int type,
const content::NotificationSource& source,
const content::NotificationDetails& details) {
@@ -164,8 +148,4 @@ void PrintJobManager::OnPrintJobEvent(
}
}
-bool PrintJobManager::printing_enabled() const {
- return *printing_enabled_;
-}
-
} // namespace printing
« no previous file with comments | « chrome/browser/printing/print_job_manager.h ('k') | chrome/browser/printing/print_preview_tab_controller_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698