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

Side by Side Diff: third_party/WebKit/Source/platform/wtf/BUILD.gn

Issue 2760573002: Change the target type of platform_wtf to source_set. (Closed)
Patch Set: Created 3 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
« 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 2017 The Chromium Authors. All rights reserved. 1 # Copyright 2017 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 # The below is a temporary setup during the WTF migration project: 5 # The below is a temporary setup during the WTF migration project:
6 # https://groups.google.com/a/chromium.org/d/msg/blink-dev/tLdAZCTlcAA/bYXVT8gYC AAJ 6 # https://groups.google.com/a/chromium.org/d/msg/blink-dev/tLdAZCTlcAA/bYXVT8gYC AAJ
7 # 7 #
8 # We are moving wtf/ files to platform/wtf/ incrementally, thus, conceptually, 8 # We are moving wtf/ files to platform/wtf/ incrementally, thus, conceptually,
9 # the "wtf" target in wtf/BUILD.gn is being split into two, in a way that 9 # the "wtf" target in wtf/BUILD.gn is being split into two, in a way that
10 # only wtf/ can refer the contents in platform/wtf/. 10 # only wtf/ can refer the contents in platform/wtf/.
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
50 if (is_component_build) { 50 if (is_component_build) {
51 # Chromium windows multi-dll build enables C++ exceptions and this causes 51 # Chromium windows multi-dll build enables C++ exceptions and this causes
52 # wtf to generate 4291 warning due to operator new/delete 52 # wtf to generate 4291 warning due to operator new/delete
53 # implementations. Disable the warning for chromium windows multi-dll 53 # implementations. Disable the warning for chromium windows multi-dll
54 # build. 54 # build.
55 cflags += [ "/wd4291" ] 55 cflags += [ "/wd4291" ]
56 } 56 }
57 } 57 }
58 } 58 }
59 59
60 component("platform_wtf") { 60 source_set("platform_wtf") {
61 sources = [ 61 sources = [
62 "AddressSanitizer.h", 62 "AddressSanitizer.h",
63 "Alignment.h", 63 "Alignment.h",
64 "AutoReset.h", 64 "AutoReset.h",
65 "BitwiseOperations.h", 65 "BitwiseOperations.h",
66 "ByteSwap.h", 66 "ByteSwap.h",
67 "CPU.h", 67 "CPU.h",
68 "CheckedNumeric.h", 68 "CheckedNumeric.h",
69 "Compiler.h", 69 "Compiler.h",
70 "ConditionalDestructor.h", 70 "ConditionalDestructor.h",
(...skipping 73 matching lines...) Expand 10 before | Expand all | Expand 10 after
144 # "text/AtomicStringCF.cpp", 144 # "text/AtomicStringCF.cpp",
145 # "text/StringImplCF.cpp", 145 # "text/StringImplCF.cpp",
146 # ] 146 # ]
147 } 147 }
148 148
149 if (remove_webcore_debug_symbols) { 149 if (remove_webcore_debug_symbols) {
150 configs -= [ "//build/config/compiler:default_symbols" ] 150 configs -= [ "//build/config/compiler:default_symbols" ]
151 configs += [ "//build/config/compiler:no_symbols" ] 151 configs += [ "//build/config/compiler:no_symbols" ]
152 } 152 }
153 } 153 }
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