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

Side by Side Diff: printing/printing.gyp

Issue 1421493002: Printing: Move some printing code out of pdf/ and into printing/. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: simplify a function, add some unit tests, rebase Created 5 years, 1 month 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
« printing/pdf_transform.cc ('K') | « printing/pdf_transform.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 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 { 5 {
6 'variables': { 6 'variables': {
7 'chromium_code': 1, 7 'chromium_code': 1,
8 }, 8 },
9 'targets': [ 9 'targets': [
10 { 10 {
(...skipping 183 matching lines...) Expand 10 before | Expand all | Expand 10 after
194 ], 194 ],
195 }], 195 }],
196 ['OS=="android"', { 196 ['OS=="android"', {
197 'sources': [ 197 'sources': [
198 'printing_context_android.cc', 198 'printing_context_android.cc',
199 'printing_context_android.h', 199 'printing_context_android.h',
200 ], 200 ],
201 'dependencies': [ 201 'dependencies': [
202 'printing_jni_headers', 202 'printing_jni_headers',
203 ], 203 ],
204 }, {
205 'sources': [
206 'pdf_transform.cc',
207 'pdf_transform.h',
208 ],
204 }], 209 }],
205 ], 210 ],
206 }, 211 },
207 { 212 {
208 'target_name': 'printing_unittests', 213 'target_name': 'printing_unittests',
209 'type': 'executable', 214 'type': 'executable',
210 'dependencies': [ 215 'dependencies': [
211 '../base/base.gyp:run_all_unittests', 216 '../base/base.gyp:run_all_unittests',
212 '../base/base.gyp:test_support_base', 217 '../base/base.gyp:test_support_base',
213 '../testing/gtest.gyp:gtest', 218 '../testing/gtest.gyp:gtest',
(...skipping 11 matching lines...) Expand all
225 'pdf_metafile_cg_mac_unittest.cc', 230 'pdf_metafile_cg_mac_unittest.cc',
226 'printed_page_unittest.cc', 231 'printed_page_unittest.cc',
227 'printing_context_win_unittest.cc', 232 'printing_context_win_unittest.cc',
228 'printing_test.h', 233 'printing_test.h',
229 'printing_utils_unittest.cc', 234 'printing_utils_unittest.cc',
230 'units_unittest.cc', 235 'units_unittest.cc',
231 ], 236 ],
232 'conditions': [ 237 'conditions': [
233 ['OS!="mac"', {'sources/': [['exclude', '_mac_unittest\\.(cc|mm?)$']]}], 238 ['OS!="mac"', {'sources/': [['exclude', '_mac_unittest\\.(cc|mm?)$']]}],
234 ['OS!="win"', {'sources/': [['exclude', '_win_unittest\\.cc$']]}], 239 ['OS!="win"', {'sources/': [['exclude', '_win_unittest\\.cc$']]}],
240 ['OS!="android"', {
241 'sources': ['pdf_transform_unittest.cc']
raymes 2015/10/26 04:04:48 Is this missing?
Lei Zhang 2015/10/26 04:39:26 yes, added.
242 }],
235 ['use_cups==1', { 243 ['use_cups==1', {
236 'defines': [ 244 'defines': [
237 'USE_CUPS', 245 'USE_CUPS',
238 ], 246 ],
239 'sources': [ 247 'sources': [
240 'backend/cups_helper_unittest.cc', 248 'backend/cups_helper_unittest.cc',
241 ], 249 ],
242 }], 250 }],
243 [ 'os_posix == 1 and OS != "mac" and OS != "android" and OS != "ios"', { 251 [ 'os_posix == 1 and OS != "mac" and OS != "android" and OS != "ios"', {
244 'conditions': [ 252 'conditions': [
(...skipping 84 matching lines...) Expand 10 before | Expand all | Expand 10 after
329 '../build/isolate.gypi', 337 '../build/isolate.gypi',
330 ], 338 ],
331 'sources': [ 339 'sources': [
332 'printing_unittests.isolate', 340 'printing_unittests.isolate',
333 ], 341 ],
334 }, 342 },
335 ], 343 ],
336 }], 344 }],
337 ] 345 ]
338 } 346 }
OLDNEW
« printing/pdf_transform.cc ('K') | « printing/pdf_transform.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698