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

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

Issue 2646523003: Refactor pdf_to_emf_converter (Closed)
Patch Set: Created 3 years, 11 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.cc
diff --git a/chrome/browser/printing/print_job.cc b/chrome/browser/printing/print_job.cc
index 70bdb9004589a6d943d0d19c9ef62c9328cff8eb..58e376e14935b94ee9c41898c14f4af42e95326c 100644
--- a/chrome/browser/printing/print_job.cc
+++ b/chrome/browser/printing/print_job.cc
@@ -231,18 +231,18 @@ class PrintJob::PdfToEmfState {
pages_in_progress_(0),
page_size_(page_size),
content_area_(content_area),
- converter_(PdfToEmfConverter::CreateDefault()) {}
+ converter_(PdfConverter::CreatePdfToEmfConverter()) {}
void Start(const scoped_refptr<base::RefCountedMemory>& data,
const PdfRenderSettings& conversion_settings,
bool print_text_with_gdi,
- const PdfToEmfConverter::StartCallback& start_callback) {
+ const PdfConverter::StartCallback& start_callback) {
converter_->Start(data, conversion_settings, print_text_with_gdi,
start_callback);
}
void GetMorePages(
- const PdfToEmfConverter::GetPageCallback& get_page_callback) {
+ const PdfConverter::GetPageCallback& get_page_callback) {
const int kMaxNumberOfTempFilesPerDocument = 3;
while (pages_in_progress_ < kMaxNumberOfTempFilesPerDocument &&
current_page_ < page_count_) {
@@ -252,7 +252,7 @@ class PrintJob::PdfToEmfState {
}
void OnPageProcessed(
- const PdfToEmfConverter::GetPageCallback& get_page_callback) {
+ const PdfConverter::GetPageCallback& get_page_callback) {
--pages_in_progress_;
GetMorePages(get_page_callback);
// Release converter if we don't need this any more.
@@ -270,7 +270,7 @@ class PrintJob::PdfToEmfState {
int pages_in_progress_;
gfx::Size page_size_;
gfx::Rect content_area_;
- std::unique_ptr<PdfToEmfConverter> converter_;
+ std::unique_ptr<PdfConverter> converter_;
};
void PrintJob::AppendPrintedPage(int page_number) {

Powered by Google App Engine
This is Rietveld 408576698