Remove black magic and >100 lines. Unhook a lot of dead code.

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/browser/printing/printed_pages_source.h"
 #include "chrome/common/notification_service.h"
 #include "googleurl/src/gurl.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
 
 class TestSource : public printing::PrintedPagesSource {
  public:
-  virtual void RenderOnePrintedPage(printing::PrintedDocument* document,
-                                    int page_number) {
-  }
   virtual std::wstring RenderSourceName() {
     return L"";
   }
   virtual GURL RenderSourceUrl() {
     return GURL();
   }
 };
 
+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());
+    worker->printing_context().UseDefaultSettings();
+    settings_ = worker->printing_context().settings();
+    return worker;
+  }
+  virtual MessageLoop* message_loop() {
+    EXPECT_FALSE(true);
+    return NULL;
+  }
+  virtual const printing::PrintSettings& settings() const {
+    return settings_;
+  }
+  virtual int cookie() const {
+    return 42;
+  }
+ private:
+  printing::PrintSettings settings_;
+};
+
 class TestPrintJob : public printing::PrintJob {
  public:
-  TestPrintJob(printing::PrintedPagesSource* source, volatile bool* check)
-      : printing::PrintJob(source), check_(check) {
-  }
   TestPrintJob(volatile bool* check) : check_(check) {
   }
   ~TestPrintJob() {
     *check_ = true;
   }
  private:
   volatile bool* check_;
 };
 
 class TestPrintNotifObserv : public NotificationObserver {
  public:
   // NotificationObserver
   virtual void Observe(NotificationType type,
                        const NotificationSource& source,
                        const NotificationDetails& details) {
-    ASSERT_EQ(NotificationType::PRINT_JOB_EVENT, type.value);
-    printing::JobEventDetails::Type event_type =
-        Details<printing::JobEventDetails>(details)->type();
-    EXPECT_NE(printing::JobEventDetails::NEW_DOC, event_type);
-    EXPECT_NE(printing::JobEventDetails::NEW_PAGE, event_type);
-    EXPECT_NE(printing::JobEventDetails::PAGE_DONE, event_type);
-    EXPECT_NE(printing::JobEventDetails::DOC_DONE, event_type);
-    EXPECT_NE(printing::JobEventDetails::JOB_DONE, event_type);
-    EXPECT_NE(printing::JobEventDetails::ALL_PAGES_REQUESTED, event_type);
-    if (event_type == printing::JobEventDetails::USER_INIT_DONE ||
-        event_type == printing::JobEventDetails::USER_INIT_CANCELED ||
-        event_type == printing::JobEventDetails::DEFAULT_INIT_DONE ||
-        event_type == printing::JobEventDetails::FAILED) {
-      MessageLoop::current()->Quit();
-      return;
-    }
+    EXPECT_FALSE(true);
   }
 };
 
 }  // namespace
 
 TEST(PrintJobTest, SimplePrint) {
   // Test the multithreaded nature of PrintJob to make sure we can use it with
   // known livetime.
+
+  // This message loop is actually never run.
+  MessageLoop current;
+
   TestPrintNotifObserv observ;
-  MessageLoop current;
   NotificationService::current()->AddObserver(
       &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;
   TestSource source;
-  volatile bool check = false;
-  scoped_refptr<printing::PrintJob> job(new TestPrintJob(&source, &check));
-  job->GetSettings(printing::PrintJob::DEFAULTS, NULL);
-  EXPECT_EQ(MessageLoop::current(), job->message_loop());
-  current.Run();
+  job->Initialize(&owner, &source);
   job->Stop();
   job = NULL;
   EXPECT_TRUE(check);
   NotificationService::current()->RemoveObserver(
       &observ, NotificationType::ALL,
       NotificationService::AllSources());
 }
 
 TEST(PrintJobTest, SimplePrintLateInit) {
   volatile bool check = false;
@@ skipping 20 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();
   */
 }