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

Side by Side Diff: pdfium.gyp

Issue 1557923004: Mark pdfium chromium_code. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Created 4 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 unified diff | Download patch
« no previous file with comments | « no previous file | 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 2015 PDFium Authors. All rights reserved. 1 # Copyright 2015 PDFium 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 # TODO(thakis): Enable this, pdfium:29 7 'chromium_code': 1,
8 #'chromium_code': 1,
9 'pdf_use_skia%': 0, 8 'pdf_use_skia%': 0,
10 'pdf_enable_v8%': 1, 9 'pdf_enable_v8%': 1,
11 'conditions': [ 10 'conditions': [
12 ['OS=="linux"', { 11 ['OS=="linux"', {
13 'bundle_freetype%': 0, 12 'bundle_freetype%': 0,
14 }, { # On Android there's no system FreeType. On Windows and Mac, only a 13 }, { # On Android there's no system FreeType. On Windows and Mac, only a
15 # few methods are used from it. 14 # few methods are used from it.
16 'bundle_freetype%': 1, 15 'bundle_freetype%': 1,
17 }], 16 }],
18 ], 17 ],
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
50 }], 49 }],
51 ], 50 ],
52 'msvs_disabled_warnings': [ 51 'msvs_disabled_warnings': [
53 4005, 4018, 4146, 4333, 4345, 4267, 52 4005, 4018, 4146, 4333, 4345, 4267,
54 # TODO(thestig): Fix all instances, remove this, pdfium:29 53 # TODO(thestig): Fix all instances, remove this, pdfium:29
55 4245, 4310, 4389, 4701, 4702, 4706, 4800, 54 4245, 4310, 4389, 4701, 4702, 4706, 4800,
56 ], 55 ],
57 'variables': { 56 'variables': {
58 'clang_warning_flags': [ 57 'clang_warning_flags': [
59 # TODO(thestig): Fix all instances, remove this, pdfium:29 58 # TODO(thestig): Fix all instances, remove this, pdfium:29
60 '-Wno-sign-compare', 59 '-Wno-sign-compare',
Lei Zhang 2016/01/05 17:18:09 Remove this guy too?
Nico 2016/01/05 17:35:07 No, this one is needed: pdfium isn't -Wsign-compar
Lei Zhang 2016/01/06 00:50:29 Acknowledged. "some" code cleanups, probably for a
61 ], 60 ],
62 # Make sure Chromium's build/common.gypi doesn't re-add the flag on linux.
63 'cflags_cc!': [ '-Wsign-compare' ],
64 }, 61 },
65 }, 62 },
66 'targets': [ 63 'targets': [
67 { 64 {
68 'target_name': 'pdfium', 65 'target_name': 'pdfium',
69 'type': 'static_library', 66 'type': 'static_library',
70 'dependencies': [ 67 'dependencies': [
71 'third_party/third_party.gyp:bigint', 68 'third_party/third_party.gyp:bigint',
72 'third_party/third_party.gyp:pdfium_base', 69 'third_party/third_party.gyp:pdfium_base',
73 'fdrm', 70 'fdrm',
(...skipping 728 matching lines...) Expand 10 before | Expand all | Expand 10 after
802 ], 799 ],
803 'dependencies': [ 800 'dependencies': [
804 '<(DEPTH)/v8/tools/gyp/v8.gyp:v8', 801 '<(DEPTH)/v8/tools/gyp/v8.gyp:v8',
805 '<(DEPTH)/v8/tools/gyp/v8.gyp:v8_libplatform', 802 '<(DEPTH)/v8/tools/gyp/v8.gyp:v8_libplatform',
806 ], 803 ],
807 }], 804 }],
808 ], 805 ],
809 }, 806 },
810 ], 807 ],
811 } 808 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698