Reverting 27389.

chrome/browser/printing/print_job_unittest.cc

 // Copyright (c) 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 "base/message_loop.h"
 #include "chrome/browser/printing/print_job.h"
 #include "chrome/browser/printing/print_job_worker.h"
 #include "chrome/common/notification_registrar.h"
 #include "chrome/common/notification_service.h"
 #include "printing/printed_pages_source.h"
@@ skipping 15 matching lines @@
 class TestPrintJobWorker : public printing::PrintJobWorker {
  public:
   explicit TestPrintJobWorker(printing::PrintJobWorkerOwner* owner)
       : printing::PrintJobWorker(owner) {
   }
   friend class TestOwner;
 };
 
 class TestOwner : public printing::PrintJobWorkerOwner {
  public:
-  virtual void AddRef() {
-    EXPECT_FALSE(true);
-  }
-  virtual void Release() {
-    EXPECT_FALSE(true);
-  }
   virtual void GetSettingsDone(const printing::PrintSettings& new_settings,
                                printing::PrintingContext::Result result) {
     EXPECT_FALSE(true);
   }
   virtual printing::PrintJobWorker* DetachWorker(
       printing::PrintJobWorkerOwner* new_owner) {
     // We're screwing up here since we're calling worker from the main thread.
     // That's fine for testing. It is actually simulating PrinterQuery behavior.
     TestPrintJobWorker* worker(new TestPrintJobWorker(new_owner));
     EXPECT_TRUE(worker->Start());
@@ skipping 45 matching lines @@
   // This message loop is actually never run.
   MessageLoop current;
 
   NotificationRegistrar registrar_;
   TestPrintNotifObserv observ;
   registrar_.Add(&observ, NotificationType::ALL,
                  NotificationService::AllSources());
   volatile bool check = false;
   scoped_refptr<printing::PrintJob> job(new TestPrintJob(&check));
   EXPECT_EQ(MessageLoop::current(), job->message_loop());
-  TestOwner owner;
+  scoped_refptr<TestOwner> owner(new TestOwner);
   TestSource source;
-  job->Initialize(&owner, &source);
+  job->Initialize(owner, &source);
   job->Stop();
   job = NULL;
   EXPECT_TRUE(check);
 }
 
 TEST(PrintJobTest, SimplePrintLateInit) {
   volatile bool check = false;
   MessageLoop current;
   scoped_refptr<printing::PrintJob> job(new TestPrintJob(&check));
   job = NULL;
@@ skipping 17 matching lines @@
   job->is_print_dialog_box_shown();
   job->document();
   // Private
   job->UpdatePrintedDocument(NULL);
   scoped_refptr<printing::JobEventDetails> event_details;
   job->OnNotifyPrintJobEvent(event_details);
   job->OnDocumentDone();
   job->ControlledWorkerShutdown();
   */
 }