Merge converters

chrome/browser/printing/pdf_to_emf_converter.cc

 // Copyright 2014 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/pdf_to_emf_converter.h"
 
 #include <stdint.h>
 #include <windows.h>
 
 #include <memory>
@@ skipping 243 matching lines @@
   void SendStopMessage() override;
 
   // Additional message handler needed for Pdf to PostScript
   void OnPageDone(bool success);
 
   DISALLOW_COPY_AND_ASSIGN(PdfToPostScriptUtilityProcessHostClient);
 };
 
 class PdfConverterImpl : public PdfConverter {
  public:
-  PdfConverterImpl();
+  PdfConverterImpl() : weak_ptr_factory_(this) {}
 
-  ~PdfConverterImpl() override;
-
-  void Start(const scoped_refptr<base::RefCountedMemory>& data,
-             const PdfRenderSettings& conversion_settings,
-             const StartCallback& start_callback) override;
+  base::WeakPtr<PdfConverterImpl> GetWeakPtr() {
+    return weak_ptr_factory_.GetWeakPtr();
+  }
 
   void GetPage(int page_number,
                const GetPageCallback& get_page_callback) override;
 
   // Helps to cancel callbacks if this object is destroyed.
   void RunCallback(const base::Closure& callback);
 
- protected:
+  void Start(
+      const scoped_refptr<PdfConverterUtilityProcessHostClient>& utility_client,
+      const scoped_refptr<base::RefCountedMemory>& data,
+      const StartCallback& start_callback) {
+    DCHECK(!utility_client_);
+    utility_client_ = utility_client;
+    utility_client_->Start(data, start_callback);
+  }
+
+ private:
   scoped_refptr<PdfConverterUtilityProcessHostClient> utility_client_;
 
- private:
+  base::WeakPtrFactory<PdfConverterImpl> weak_ptr_factory_;
+
   DISALLOW_COPY_AND_ASSIGN(PdfConverterImpl);
 };
 
-class PdfToEmfConverterImpl : public PdfConverterImpl {
- public:
-  PdfToEmfConverterImpl();
-
-  ~PdfToEmfConverterImpl() override;
-
-  void Start(const scoped_refptr<base::RefCountedMemory>& data,
-             const PdfRenderSettings& conversion_settings,
-             const StartCallback& start_callback) override;
-
- private:
-  base::WeakPtrFactory<PdfToEmfConverterImpl> weak_ptr_factory_;
-
-  DISALLOW_COPY_AND_ASSIGN(PdfToEmfConverterImpl);
-};
-
-class PdfToPostScriptConverterImpl : public PdfConverterImpl {
- public:
-  PdfToPostScriptConverterImpl();
-  ~PdfToPostScriptConverterImpl() override;
-
-  void Start(const scoped_refptr<base::RefCountedMemory>& data,
-             const PdfRenderSettings& conversion_settings,
-             const StartCallback& start_callback) override;
-
- private:
-  base::WeakPtrFactory<PdfToPostScriptConverterImpl> weak_ptr_factory_;
-  DISALLOW_COPY_AND_ASSIGN(PdfToPostScriptConverterImpl);
-};
-
 ScopedTempFile CreateTempFile(scoped_refptr<RefCountedTempDir>* temp_dir) {
   if (!temp_dir->get())
     *temp_dir = new RefCountedTempDir();
   ScopedTempFile file;
   if (!(*temp_dir)->IsValid())
     return file;
   base::FilePath path;
   if (!base::CreateTemporaryFileInDir((*temp_dir)->GetPath(), &path)) {
     PLOG(ERROR) << "Failed to create file in "
                 << (*temp_dir)->GetPath().value();
@@ skipping 377 matching lines @@
 }
 
 void PdfToPostScriptUtilityProcessHostClient::SendStopMessage() {
   Send(new ChromeUtilityMsg_RenderPDFPagesToPostScript_Stop());
 }
 
 void PdfToPostScriptUtilityProcessHostClient::OnPageDone(bool success) {
   PdfConverterUtilityProcessHostClient::OnPageDone(success, 0.0f);
 }
 
-// Pdf Converter Impl and subclasses
-PdfConverterImpl::PdfConverterImpl() {}
-
-PdfConverterImpl::~PdfConverterImpl() {}
-
-void PdfConverterImpl::Start(const scoped_refptr<base::RefCountedMemory>& data,
-                             const PdfRenderSettings& conversion_settings,
-                             const StartCallback& start_callback) {
-  DCHECK(!utility_client_.get());
-}
-
 void PdfConverterImpl::GetPage(int page_number,
                                const GetPageCallback& get_page_callback) {
   utility_client_->GetPage(page_number, get_page_callback);
 }
 
 void PdfConverterImpl::RunCallback(const base::Closure& callback) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   callback.Run();
 }
 
-PdfToEmfConverterImpl::PdfToEmfConverterImpl() : weak_ptr_factory_(this) {}
-
-PdfToEmfConverterImpl::~PdfToEmfConverterImpl() {
-  if (utility_client_.get())
-    utility_client_->Stop();
-}
-
-void PdfToEmfConverterImpl::Start(
-    const scoped_refptr<base::RefCountedMemory>& data,
-    const PdfRenderSettings& conversion_settings,
-    const StartCallback& start_callback) {
-  DCHECK(!utility_client_.get());
-  utility_client_ = new PdfToEmfUtilityProcessHostClient(
-      weak_ptr_factory_.GetWeakPtr(), conversion_settings);
-  utility_client_->Start(data, start_callback);
-}
-
-PdfToPostScriptConverterImpl::PdfToPostScriptConverterImpl()
-    : weak_ptr_factory_(this) {}
-
-PdfToPostScriptConverterImpl::~PdfToPostScriptConverterImpl() {
-  if (utility_client_.get())
-    utility_client_->Stop();
-}
-
-void PdfToPostScriptConverterImpl::Start(
-    const scoped_refptr<base::RefCountedMemory>& data,
-    const PdfRenderSettings& conversion_settings,
-    const StartCallback& start_callback) {
-  DCHECK(!utility_client_.get());
-  utility_client_ = new PdfToPostScriptUtilityProcessHostClient(
-      weak_ptr_factory_.GetWeakPtr(), conversion_settings);
-  utility_client_->Start(data, start_callback);
-}
-
 }  // namespace
 
 PdfConverter::~PdfConverter() {}
 
 // static
-std::unique_ptr<PdfConverter> PdfConverter::CreatePdfToEmfConverter() {
-  return base::MakeUnique<PdfToEmfConverterImpl>();
+std::unique_ptr<PdfConverter> PdfConverter::StartPdfToEmfConverter(
+    const scoped_refptr<base::RefCountedMemory>& data,
+    const PdfRenderSettings& conversion_settings,
+    const StartCallback& start_callback) {
+  std::unique_ptr<PdfConverterImpl> converter =
+      base::MakeUnique<PdfConverterImpl>();
+  converter->Start(new PdfToEmfUtilityProcessHostClient(converter->GetWeakPtr(),
+                                                        conversion_settings),
+                   data, start_callback);
+  return std::move(converter);
 }
 
 // static
-std::unique_ptr<PdfConverter> PdfConverter::CreatePdfToPostScriptConverter() {
-  return base::MakeUnique<PdfToPostScriptConverterImpl>();
+std::unique_ptr<PdfConverter> PdfConverter::StartPdfToPostScriptConverter(
+    const scoped_refptr<base::RefCountedMemory>& data,
+    const PdfRenderSettings& conversion_settings,
+    const StartCallback& start_callback) {
+  std::unique_ptr<PdfConverterImpl> converter =
+      base::MakeUnique<PdfConverterImpl>();
+  converter->Start(new PdfToPostScriptUtilityProcessHostClient(
+                       converter->GetWeakPtr(), conversion_settings),
+                   data, start_callback);
+  return std::move(converter);
 }
 
 }  // namespace printing