Remove unused notification in print code to simplify before refactoring....

chrome/browser/printing/print_job.cc

 // Copyright (c) 2006-2008 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/printing/print_job.h"
 
 #include "base/message_loop.h"
 #include "base/timer.h"
 #include "chrome/browser/printing/print_job_worker.h"
 #include "chrome/browser/printing/printed_document.h"
@@ skipping 48 matching lines @@
   // Don't forget to register to our own messages.
   registrar_.Add(this, NotificationType::PRINT_JOB_EVENT,
                  Source<PrintJob>(this));
 }
 
 void PrintJob::Observe(NotificationType type,
                        const NotificationSource& source,
                        const NotificationDetails& details) {
   DCHECK_EQ(ui_message_loop_, MessageLoop::current());
   switch (type.value) {
-    case NotificationType::PRINTED_DOCUMENT_UPDATED: {
-      DCHECK(Source<PrintedDocument>(source).ptr() ==
-             document_.get());
-
-      // This notification may happens even if no job is started (i.e. print
-      // preview)
-      if (is_job_pending_ == true &&
-          Source<PrintedDocument>(source).ptr() == document_.get() &&
-          Details<PrintedPage>(details).ptr() != NULL) {
-        // Are we waiting for a page to print? The worker will know.
-        worker_->message_loop()->PostTask(FROM_HERE, NewRunnableMethod(
-            worker_.get(), &PrintJobWorker::OnNewPage));
-      }
-      break;
-    }
     case NotificationType::PRINT_JOB_EVENT: {
       OnNotifyPrintJobEvent(*Details<JobEventDetails>(details).ptr());
       break;
     }
     default: {
       break;
     }
   }
 }
 
@@ skipping 128 matching lines @@
   return is_print_dialog_box_shown_;
 }
 
 PrintedDocument* PrintJob::document() const {
   return document_.get();
 }
 
 void PrintJob::UpdatePrintedDocument(PrintedDocument* new_document) {
   if (document_.get() == new_document)
     return;
-  // Unregisters.
-  if (document_.get()) {
-    registrar_.Remove(this, NotificationType::PRINTED_DOCUMENT_UPDATED,
-                      Source<PrintedDocument>(document_.get()));
-  }
+
   document_ = new_document;
 
-  // Registers.
   if (document_.get()) {
-    registrar_.Add(this, NotificationType::PRINTED_DOCUMENT_UPDATED,
-                   Source<PrintedDocument>(document_.get()));
     settings_ = document_->settings();
   }
 
   if (worker_.get() && worker_->message_loop()) {
     DCHECK(!is_job_pending_);
     // Sync the document with the worker.
     worker_->message_loop()->PostTask(FROM_HERE, NewRunnableMethod(
         worker_.get(), &PrintJobWorker::OnDocumentChanged, document_));
   }
 }
@@ skipping 111 matching lines @@
 
 PrintedDocument* JobEventDetails::document() const {
   return document_;
 }
 
 PrintedPage* JobEventDetails::page() const {
   return page_;
 }
 
 }  // namespace printing