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

Side by Side Diff: chrome/renderer/mock_printer.cc

Issue 6594121: Revert 76553 - Applying factory pattern (through NativeMetafileFactory class)... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 9 years, 9 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/renderer/mock_printer.h" 5 #include "chrome/renderer/mock_printer.h"
6 6
7 #include "base/file_util.h" 7 #include "base/file_util.h"
8 #include "base/shared_memory.h" 8 #include "base/shared_memory.h"
9 #include "chrome/common/render_messages.h" 9 #include "chrome/common/render_messages.h"
10 #include "chrome/common/render_messages_params.h" 10 #include "chrome/common/render_messages_params.h"
11 #include "ipc/ipc_message_utils.h" 11 #include "ipc/ipc_message_utils.h"
12 #include "printing/native_metafile_factory.h"
13 #include "printing/native_metafile.h"
14 #include "printing/units.h" 12 #include "printing/units.h"
15 #include "testing/gtest/include/gtest/gtest.h" 13 #include "testing/gtest/include/gtest/gtest.h"
16 14
17 MockPrinterPage::MockPrinterPage(const void* source_data, 15 MockPrinterPage::MockPrinterPage(const void* source_data,
18 uint32 source_size, 16 uint32 source_size,
19 const printing::Image& image) 17 const printing::Image& image)
20 : source_size_(source_size), 18 : source_size_(source_size),
21 image_(image) { 19 image_(image) {
22 // Create copies of the source data 20 // Create copies of the source data
23 source_data_.reset(new uint8[source_size]); 21 source_data_.reset(new uint8[source_size]);
(...skipping 102 matching lines...) Expand 10 before | Expand all | Expand 10 after
126 // We duplicate the given file handle when creating a base::SharedMemory 124 // We duplicate the given file handle when creating a base::SharedMemory
127 // instance so that its destructor closes the copy. 125 // instance so that its destructor closes the copy.
128 EXPECT_GT(params.data_size, 0U); 126 EXPECT_GT(params.data_size, 0U);
129 #if defined(OS_WIN) 127 #if defined(OS_WIN)
130 base::SharedMemory metafile_data(params.metafile_data_handle, true, 128 base::SharedMemory metafile_data(params.metafile_data_handle, true,
131 GetCurrentProcess()); 129 GetCurrentProcess());
132 #elif defined(OS_MACOSX) 130 #elif defined(OS_MACOSX)
133 base::SharedMemory metafile_data(params.metafile_data_handle, true); 131 base::SharedMemory metafile_data(params.metafile_data_handle, true);
134 #endif 132 #endif
135 metafile_data.Map(params.data_size); 133 metafile_data.Map(params.data_size);
136 scoped_ptr<printing::NativeMetafile> metafile( 134 printing::NativeMetafile metafile;
137 printing::NativeMetafileFactory::CreateMetafile()); 135 metafile.Init(metafile_data.memory(), params.data_size);
138 metafile->Init(metafile_data.memory(), params.data_size); 136 printing::Image image(metafile);
139 printing::Image image(*metafile);
140 MockPrinterPage* page_data = new MockPrinterPage(metafile_data.memory(), 137 MockPrinterPage* page_data = new MockPrinterPage(metafile_data.memory(),
141 params.data_size, 138 params.data_size,
142 image); 139 image);
143 if (!page_data) { 140 if (!page_data) {
144 printer_status_ = PRINTER_ERROR; 141 printer_status_ = PRINTER_ERROR;
145 return; 142 return;
146 } 143 }
147 144
148 scoped_refptr<MockPrinterPage> page(page_data); 145 scoped_refptr<MockPrinterPage> page(page_data);
149 pages_.push_back(page); 146 pages_.push_back(page);
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
205 if (printer_status_ != PRINTER_READY || page >= pages_.size()) 202 if (printer_status_ != PRINTER_READY || page >= pages_.size())
206 return false; 203 return false;
207 204
208 pages_[page]->image().SaveToPng(filepath); 205 pages_[page]->image().SaveToPng(filepath);
209 return true; 206 return true;
210 } 207 }
211 208
212 int MockPrinter::CreateDocumentCookie() { 209 int MockPrinter::CreateDocumentCookie() {
213 return ++current_document_cookie_; 210 return ++current_document_cookie_;
214 } 211 }
OLDNEW
« no previous file with comments | « chrome/plugin/webplugin_delegate_stub.cc ('k') | chrome/renderer/print_web_view_helper_linux.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698