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

Unified Diff: printing/printing.gyp

Issue 149212: Move printing related stuff to the root printing project from the browser pro... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 5 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: printing/printing.gyp
===================================================================
--- printing/printing.gyp (revision 20026)
+++ printing/printing.gyp (working copy)
@@ -14,9 +14,13 @@
'target_name': 'printing',
'type': '<(library)',
'dependencies': [
+ '../app/app.gyp:app_base', # Only required for Font support
'../base/base.gyp:base',
'../base/base.gyp:base_gfx',
-
+ '../build/temp_gyp/googleurl.gyp:googleurl',
+ '../skia/skia.gyp:skia',
+ '../third_party/icu38/icu38.gyp:icui18n',
+ '../third_party/icu38/icu38.gyp:icuuc',
],
'msvs_guid': '9E5416B9-B91B-4029-93F4-102C1AD5CAF4',
'include_dirs': [
@@ -26,6 +30,25 @@
'emf_win.cc',
'emf_win.h',
'native_metafile.h',
+ 'page_number.cc',
+ 'page_number.h',
+ 'page_overlays.cc',
+ 'page_overlays.h',
+ 'page_range.cc',
+ 'page_range.h',
+ 'page_setup.cc',
+ 'page_setup.h',
+ 'print_settings.cc',
+ 'print_settings.h',
+ 'printed_document.cc',
+ 'printed_document.h',
+ 'printed_page.cc',
+ 'printed_page.h',
+ 'printed_pages_source.h',
+ 'printing_context.h',
+ 'printing_context_linux.cc',
+ 'printing_context_mac.cc',
+ 'printing_context_win.cc',
'units.cc',
'units.h',
],
@@ -38,7 +61,12 @@
['OS!="linux"', {'sources/': [['exclude', '_linux\\.cc$']]}],
['OS!="mac"', {'sources/': [['exclude', '_mac\\.(cc|mm?)$']]}],
['OS!="win"', {
- 'sources/': [['exclude', '_win\\.cc$']]
+ 'sources/': [
+ ['exclude', '_win\\.cc$'],
+ ['exclude',
+ 'printed_document.cc',
+ ]
+ ]
}, { # else: OS=="win"
'sources/': [['exclude', '_posix\\.cc$']]
}],
@@ -55,17 +83,31 @@
],
'sources': [
'emf_win_unittest.cc',
+ 'printing_test.h',
+ 'page_number_unittest.cc',
+ 'page_overlays_unittest.cc',
+ 'page_range_unittest.cc',
+ 'page_setup_unittest.cc',
+ 'printing_context_win_unittest.cc',
'units_unittest.cc',
],
'conditions': [
['OS!="linux"', {'sources/': [['exclude', '_linux_unittest\\.cc$']]}],
['OS!="mac"', {'sources/': [['exclude', '_mac_unittest\\.(cc|mm?)$']]}],
['OS!="win"', {
- 'sources/': [['exclude', '_win_unittest\\.cc$']]
+ 'sources/': [
+ ['exclude', '_win_unittest\\.cc$'],
+
+ # Most of the printing functionailty is Windows only for now.
+ ['exclude', '.*'],
+ ['include', 'page_range_unittest.cc'],
+ ['include', 'page_setup_unittest.cc'],
+ ['include', 'units_unittest.cc'],
+ ]
}, { # else: OS=="win"
'sources/': [['exclude', '_posix_unittest\\.cc$']]
}],
],
},
],
-}
+}

Powered by Google App Engine
This is Rietveld 408576698