Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Delta Between Two Patch Sets: adblockplus.gyp

Issue 29323611: Issue #1234, #2058 - Rewrite log facility, improving thread implementation
Left Patch Set: Created Aug. 19, 2015, 5:42 p.m.
Right Patch Set: rebase only Created July 27, 2016, 9:11 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « no previous file | src/plugin/ActiveQueue.h » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 { 1 {
2 'includes': [ 2 'includes': [
3 'defaults.gypi', 3 'defaults.gypi',
4 'common/common.gypi', 4 'common/common.gypi',
5 ], 5 ],
6 6
7 'variables': { 7 'variables': {
8 'build_type%': 'devbuild', 8 'build_type%': 'devbuild',
9 'build_version%': '', 9 'build_version%': '',
10 }, 10 },
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
43 'src/shared/AutoHandle.cpp', 43 'src/shared/AutoHandle.cpp',
44 'src/shared/AutoHandle.h', 44 'src/shared/AutoHandle.h',
45 'src/shared/Communication.cpp', 45 'src/shared/Communication.cpp',
46 'src/shared/Communication.h', 46 'src/shared/Communication.h',
47 'src/shared/CriticalSection.h', 47 'src/shared/CriticalSection.h',
48 'src/shared/Dictionary.cpp', 48 'src/shared/Dictionary.cpp',
49 'src/shared/Dictionary.h', 49 'src/shared/Dictionary.h',
50 'src/shared/Utils.cpp', 50 'src/shared/Utils.cpp',
51 'src/shared/Utils.h', 51 'src/shared/Utils.h',
52 'src/shared/Version.h', 52 'src/shared/Version.h',
53 ] 53 'src/shared/MsHTMLUtils.cpp',
54 'src/shared/MsHTMLUtils.h',
55 ],
56 'include_dirs': [
57 '$(ADBLOCKPLUS_ATL)/include',
58 ],
54 }, 59 },
55 60
56 { 61 {
57 'target_name': 'AdblockPlusEngine', 62 'target_name': 'AdblockPlusEngine',
58 'type': 'executable', 63 'type': 'executable',
59 'dependencies': [ 64 'dependencies': [
60 'shared', 65 'shared',
61 'libadblockplus/libadblockplus.gyp:libadblockplus', 66 'libadblockplus/libadblockplus.gyp:libadblockplus',
62 ], 67 ],
63 'sources': [ 68 'sources': [
64 'src/engine/Main.cpp', 69 'src/engine/Main.cpp',
70 'src/engine/NotificationWindow.cpp',
71 'src/engine/NotificationWindow.h',
65 'src/engine/Debug.cpp', 72 'src/engine/Debug.cpp',
66 'src/engine/UpdateInstallDialog.cpp', 73 'src/engine/UpdateInstallDialog.cpp',
67 'src/engine/Updater.cpp', 74 'src/engine/Updater.cpp',
68 'src/engine/engine.rc', 75 'src/engine/engine.rc',
76 ],
77 'include_dirs': [
78 '$(ADBLOCKPLUS_ATL)/include',
69 ], 79 ],
70 'libraries': [ 80 'libraries': [
71 '-ladvapi32', 81 '-ladvapi32',
72 '-lole32', 82 '-lole32',
73 '-luser32', 83 '-luser32',
74 '-lshell32', 84 '-lshell32',
75 '-lshlwapi', 85 '-lshlwapi',
76 ], 86 ],
77 'msbuild_toolset': 'v110_xp', 87 'msbuild_toolset': 'v110_xp',
78 'msvs_settings': { 88 'msvs_settings': {
79 'VCLinkerTool': { 89 'VCLinkerTool': {
80 'DelayLoadDLLs': ['Shell32.dll'], 90 'DelayLoadDLLs': ['Shell32.dll'],
91 'conditions': [[
92 'target_arch=="ia32"', {
93 'AdditionalLibraryDirectories': [
94 '$(ADBLOCKPLUS_ATL)/lib',
95 ],
96 }, {
97 'AdditionalLibraryDirectories': [
98 '$(ADBLOCKPLUS_ATL)/lib/amd64',
99 ],
100 }
101 ]],
81 }, 102 },
82 }, 103 },
83 }, 104 },
84 105
85 { 106 {
86 'target_name': 'AdblockPlus', 107 'target_name': 'AdblockPlus',
87 'type': 'shared_library', 108 'type': 'shared_library',
88 'dependencies': [ 109 'dependencies': [
89 'shared', 110 'shared',
90 'libadblockplus/libadblockplus.gyp:libadblockplus', 111 'libadblockplus/libadblockplus.gyp:libadblockplus',
91 ], 112 ],
92 'sources': [ 113 'sources': [
93 'src/plugin/ActiveQueue.h', 114 'src/plugin/ActiveQueue.h',
94 'src/plugin/AdblockPlus.def', 115 'src/plugin/AdblockPlus.def',
95 'src/plugin/AdblockPlus.idl', 116 'src/plugin/AdblockPlus.idl',
96 'src/plugin/AdblockPlus.rc', 117 'src/plugin/AdblockPlus.rc',
97 'src/plugin/AdblockPlus.rgs', 118 'src/plugin/AdblockPlus.rgs',
98 'src/plugin/AdblockPlusClient.cpp', 119 'src/plugin/AdblockPlusClient.cpp',
99 'src/plugin/AdblockPlusClient.h', 120 'src/plugin/AdblockPlusClient.h',
100 'src/plugin/AdblockPlusDomTraverser.cpp', 121 'src/plugin/AdblockPlusDomTraverser.cpp',
101 'src/plugin/AdblockPlusDomTraverser.h', 122 'src/plugin/AdblockPlusDomTraverser.h',
102 'src/plugin/AdblockPlusGuids.h', 123 'src/plugin/AdblockPlusGuids.h',
103 'src/plugin/ATL_Deprecate.cpp',
104 'src/plugin/ATL_Deprecate.h', 124 'src/plugin/ATL_Deprecate.h',
105 'src/plugin/Config.h', 125 'src/plugin/Config.h',
126 'src/plugin/Instances.h',
106 'src/plugin/NotificationMessage.cpp', 127 'src/plugin/NotificationMessage.cpp',
107 'src/plugin/NotificationMessage.h', 128 'src/plugin/NotificationMessage.h',
108 'src/plugin/Placeholder.h', 129 'src/plugin/Placeholder.h',
109 'src/plugin/Plugin.cpp', 130 'src/plugin/Plugin.cpp',
110 'src/plugin/Plugin.h', 131 'src/plugin/Plugin.h',
111 'src/plugin/PluginClass.cpp', 132 'src/plugin/PluginClass.cpp',
112 'src/plugin/PluginClass.h', 133 'src/plugin/PluginClass.h',
113 'src/plugin/PluginClientFactory.cpp', 134 'src/plugin/PluginClientFactory.cpp',
114 'src/plugin/PluginClientFactory.h', 135 'src/plugin/PluginClientFactory.h',
115 'src/plugin/PluginDebug.cpp', 136 'src/plugin/PluginDebug.cpp',
116 'src/plugin/PluginDebug.h', 137 'src/plugin/PluginDebug.h',
117 'src/plugin/PluginDomTraverserBase.h', 138 'src/plugin/PluginDomTraverserBase.h',
118 'src/plugin/PluginErrorCodes.h', 139 'src/plugin/PluginErrorCodes.h',
119 'src/plugin/PluginFilter.cpp', 140 'src/plugin/PluginFilter.cpp',
120 'src/plugin/PluginFilter.h', 141 'src/plugin/PluginFilter.h',
121 'src/plugin/PluginMimeFilterClient.cpp', 142 'src/plugin/PluginMimeFilterClient.cpp',
122 'src/plugin/PluginMimeFilterClient.h', 143 'src/plugin/PluginMimeFilterClient.h',
123 'src/plugin/PluginMutex.cpp', 144 'src/plugin/PluginMutex.cpp',
124 'src/plugin/PluginMutex.h', 145 'src/plugin/PluginMutex.h',
125 'src/plugin/PluginPassthroughObject.h',
126 'src/plugin/PluginSettings.cpp', 146 'src/plugin/PluginSettings.cpp',
127 'src/plugin/PluginSettings.h', 147 'src/plugin/PluginSettings.h',
128 'src/plugin/PluginStdAfx.cpp', 148 'src/plugin/PluginStdAfx.cpp',
129 'src/plugin/PluginStdAfx.h', 149 'src/plugin/PluginStdAfx.h',
130 'src/plugin/PluginSystem.cpp', 150 'src/plugin/PluginSystem.cpp',
131 'src/plugin/PluginSystem.h', 151 'src/plugin/PluginSystem.h',
132 'src/plugin/PluginTabBase.cpp', 152 'src/plugin/PluginTabBase.cpp',
133 'src/plugin/PluginTabBase.h', 153 'src/plugin/PluginTabBase.h',
134 'src/plugin/PluginUserSettings.cpp', 154 'src/plugin/PluginUserSettings.cpp',
135 'src/plugin/PluginUserSettings.h', 155 'src/plugin/PluginUserSettings.h',
136 'src/plugin/PluginUtil.cpp', 156 'src/plugin/PluginUtil.cpp',
137 'src/plugin/PluginUtil.h', 157 'src/plugin/PluginUtil.h',
138 'src/plugin/PluginWbPassThrough.cpp', 158 'src/plugin/PluginWbPassThrough.cpp',
139 'src/plugin/PluginWbPassThrough.h', 159 'src/plugin/PluginWbPassThrough.h',
140 'src/plugin/ProtocolCF.h',
141 'src/plugin/ProtocolCF.inl',
142 'src/plugin/ProtocolImpl.h',
143 'src/plugin/ProtocolImpl.inl',
144 'src/plugin/Resource.h', 160 'src/plugin/Resource.h',
145 'src/plugin/SinkPolicy.h', 161 'src/plugin/passthroughapp/PassthroughObject.h',
146 'src/plugin/SinkPolicy.inl', 162 'src/plugin/passthroughapp/ProtocolCF.h',
163 'src/plugin/passthroughapp/ProtocolCF.inl',
164 'src/plugin/passthroughapp/ProtocolImpl.h',
165 'src/plugin/passthroughapp/ProtocolImpl.inl',
166 'src/plugin/passthroughapp/SinkPolicy.h',
167 'src/plugin/passthroughapp/SinkPolicy.inl',
147 ], 168 ],
148 'include_dirs': [ 169 'include_dirs': [
149 '$(WindowsSDK_IncludePath)', 170 '$(ADBLOCKPLUS_ATL)/include',
150 '$(VCInstallDir)atlmfc/include',
151 '$(WINDDKDIR)/inc/atl71',
152 ], 171 ],
153 # See "Adding Visual Style Support to an Extension, Plug-in, MMC Snap-in or a DLL 172 # See "Adding Visual Style Support to an Extension, Plug-in, MMC Snap-in or a DLL
154 # That Is Brought into a Process" on the link here: 173 # That Is Brought into a Process" on the link here:
155 # http://msdn.microsoft.com/en-us/library/windows/desktop/bb773175%28v=vs.85 %29.aspx#using_manifests 174 # http://msdn.microsoft.com/en-us/library/windows/desktop/bb773175%28v=vs.85 %29.aspx#using_manifests
156 'defines': ['ISOLATION_AWARE_ENABLED'], 175 'defines': ['ISOLATION_AWARE_ENABLED'],
157 'libraries': [ 176 'libraries': [
158 '-lwinhttp', 177 '-lwinhttp',
159 '-lshell32', 178 '-lshell32',
160 '-lComctl32', 179 '-lComctl32',
161 '-lGdi32', 180 '-lGdi32',
162 ], 181 ],
163 'configurations': {
164 # 'libraries' is not allowed under 'configurations' :-(
165 'Debug': {
166 'msvs_settings': {
167 'VCLinkerTool': {
168 'AdditionalDependencies': ['atlsd.lib'],
169 },
170 },
171 },
172 'Release': {
173 'msvs_settings': {
174 'VCLinkerTool': {
175 'AdditionalDependencies': ['atls.lib'],
176 },
177 },
178 },
179 },
180 'msvs_settings': { 182 'msvs_settings': {
181 'VCLinkerTool': { 183 'VCLinkerTool': {
182 'conditions': [[ 184 'conditions': [[
183 'target_arch=="ia32"', { 185 'target_arch=="ia32"', {
184 'AdditionalLibraryDirectories': [ 186 'AdditionalLibraryDirectories': [
185 '$(VCInstallDir)atlmfc/lib', 187 '$(ADBLOCKPLUS_ATL)/lib',
186 '$(WindowsSDK_LibraryPath_x86)',
187 '$(WINDDKDIR)/lib/ATL/i386',
188 ], 188 ],
189 }, { 189 }, {
190 'AdditionalLibraryDirectories': [ 190 'AdditionalLibraryDirectories': [
191 '$(VCInstallDir)atlmfc/lib/amd64', 191 '$(ADBLOCKPLUS_ATL)/lib/amd64',
192 '$(WindowsSDK_LibraryPath_x64)',
193 '$(WINDDKDIR)/lib/ATL/amd64',
194 ], 192 ],
195 } 193 }
196 ]], 194 ]],
197 'DelayLoadDLLs': ['Shell32.dll'], 195 'DelayLoadDLLs': ['Shell32.dll'],
198 }, 196 },
199 }, 197 },
200 }, 198 },
201 199
202 { 200 {
203 'target_name': 'tests', 201 'target_name': 'tests',
204 'type': 'executable', 202 'type': 'executable',
205 'dependencies': [ 203 'dependencies': [
206 'shared', 204 'shared',
207 'libadblockplus/third_party/googletest.gyp:googletest_main', 205 'libadblockplus/third_party/googletest.gyp:googletest_main',
208 ], 206 ],
209 'sources': [ 207 'sources': [
210 'test/CommunicationTest.cpp', 208 'test/CommunicationTest.cpp',
211 'test/DictionaryTest.cpp', 209 'test/DictionaryTest.cpp',
212 'test/UtilTest.cpp', 210 'test/UtilTest.cpp',
213 'test/UtilGetQueryStringTest.cpp', 211 'test/UtilGetQueryStringTest.cpp',
214 'test/UtilGetSchemeAndHierarchicalPartTest.cpp', 212 'test/UtilGetSchemeAndHierarchicalPartTest.cpp',
215 ], 213 ],
216 'defines': ['WINVER=0x0501'], 214 'defines': ['WINVER=0x0501'],
217 'link_settings': { 215 'link_settings': {
218 'libraries': ['-ladvapi32', '-lshell32', '-lole32'], 216 'libraries': ['-ladvapi32', '-lshell32', '-lole32', '-loleaut32'],
219 }, 217 },
220 'msvs_settings': { 218 'msvs_settings': {
221 'VCLinkerTool': { 219 'VCLinkerTool': {
222 'SubSystem': '1', # Console 220 'SubSystem': '1', # Console
223 'EntryPointSymbol': 'mainCRTStartup', 221 'EntryPointSymbol': 'mainCRTStartup',
224 }, 222 },
225 }, 223 },
226 }, 224 },
227 225
228 { 226 {
229 'target_name': 'tests_plugin', 227 'target_name': 'tests_plugin',
230 'type': 'executable', 228 'type': 'executable',
231 'dependencies': [ 229 'dependencies': [
232 'shared', 230 'shared',
233 'libadblockplus/libadblockplus.gyp:libadblockplus', 231 'libadblockplus/libadblockplus.gyp:libadblockplus',
234 'libadblockplus/third_party/googletest.gyp:googletest_main', 232 'libadblockplus/third_party/googletest.gyp:googletest_main',
235 ], 233 ],
236 'sources': [ 234 'sources': [
235 'src/plugin/PluginDebug.cpp',
236 'test/plugin/DebugTest.cpp',
237 'src/plugin/Instances.h',
238 'test/plugin/InstancesTest.cpp',
237 'src/plugin/PluginUserSettings.cpp', 239 'src/plugin/PluginUserSettings.cpp',
238 'src/plugin/PluginUserSettings.h', 240 'src/plugin/PluginUserSettings.h',
239 'test/plugin/UserSettingsTest.cpp', 241 'test/plugin/UserSettingsTest.cpp',
242 'src/plugin/PluginUtil.h',
243 'test/plugin/UtilTest.cpp',
240 # 244 #
241 # required only for linking 245 # required only for linking
242 # 246 #
243 'src/plugin/AdblockPlusClient.cpp', 247 'src/plugin/AdblockPlusClient.cpp',
244 'src/plugin/AdblockPlusDomTraverser.cpp', 248 'src/plugin/AdblockPlusDomTraverser.cpp',
245 'src/plugin/ATL_Deprecate.cpp',
246 'src/plugin/NotificationMessage.cpp', 249 'src/plugin/NotificationMessage.cpp',
247 'src/plugin/Plugin.cpp', 250 'src/plugin/Plugin.cpp',
248 'src/plugin/PluginClientFactory.cpp', 251 'src/plugin/PluginClientFactory.cpp',
249 'src/plugin/PluginClass.cpp', 252 'src/plugin/PluginClass.cpp',
250 'src/plugin/PluginDebug.cpp',
251 'src/plugin/PluginFilter.cpp', 253 'src/plugin/PluginFilter.cpp',
252 'src/plugin/PluginMimeFilterClient.cpp', 254 'src/plugin/PluginMimeFilterClient.cpp',
253 'src/plugin/PluginMutex.cpp', 255 'src/plugin/PluginMutex.cpp',
254 'src/plugin/PluginSettings.cpp', 256 'src/plugin/PluginSettings.cpp',
255 'src/plugin/PluginSystem.cpp', 257 'src/plugin/PluginSystem.cpp',
256 'src/plugin/PluginTabBase.cpp', 258 'src/plugin/PluginTabBase.cpp',
257 'src/plugin/PluginUtil.cpp', 259 'src/plugin/PluginUtil.cpp',
258 'src/plugin/PluginWbPassthrough.cpp', 260 'src/plugin/PluginWbPassthrough.cpp',
259 ], 261 ],
260 'include_dirs': [ 262 'include_dirs': [
261 '$(WINDDKDIR)/inc/atl71', 263 '$(ADBLOCKPLUS_ATL)/include',
262 ], 264 ],
263 'defines': [ 265 'defines': [
264 'WINVER=0x0501', 266 'WINVER=0x0501',
265 'PRODUCT_ADBLOCKPLUS' 267 'PRODUCT_ADBLOCKPLUS'
266 ], 268 ],
267 'link_settings': { 269 'link_settings': {
268 'libraries': ['-ladvapi32', '-lshell32', '-lole32', '-lComctl32', '-lGdi32 '], 270 'libraries': ['-ladvapi32', '-lshell32', '-lole32', '-lComctl32', '-lGdi32 '],
269 }, 271 },
270 'msvs_settings': { 272 'msvs_settings': {
271 'VCLinkerTool': { 273 'VCLinkerTool': {
272 'SubSystem': '1', # Console 274 'SubSystem': '1', # Console
273 'EntryPointSymbol': 'mainCRTStartup', 275 'EntryPointSymbol': 'mainCRTStartup',
274 'conditions': [[ 276 'conditions': [[
275 'target_arch=="ia32"', { 277 'target_arch=="ia32"', {
276 'AdditionalLibraryDirectories': [ 278 'AdditionalLibraryDirectories': [
277 '$(VCInstallDir)atlmfc/lib', 279 '$(ADBLOCKPLUS_ATL)/lib',
278 '$(WindowsSDK_LibraryPath_x86)',
279 '$(WINDDKDIR)/lib/ATL/i386',
280 ], 280 ],
281 }, { 281 }, {
282 'AdditionalLibraryDirectories': [ 282 'AdditionalLibraryDirectories': [
283 '$(VCInstallDir)atlmfc/lib/amd64', 283 '$(ADBLOCKPLUS_ATL)/lib/amd64',
284 '$(WindowsSDK_LibraryPath_x64)',
285 '$(WINDDKDIR)/lib/ATL/amd64',
286 ], 284 ],
287 } 285 }
288 ]], 286 ]],
289 },
290 },
291 'configurations': {
292 # 'libraries' is not allowed under 'configurations' :-(
293 'Debug': {
294 'msvs_settings': {
295 'VCLinkerTool': {
296 'AdditionalDependencies': ['atlsd.lib'],
297 },
298 },
299 },
300 'Release': {
301 'msvs_settings': {
302 'VCLinkerTool': {
303 'AdditionalDependencies': ['atls.lib'],
304 },
305 },
306 }, 287 },
307 }, 288 },
308 }, 289 },
309 290
310 { 291 {
311 'target_name': 'tests_plugin_2', 292 'target_name': 'tests_plugin_2',
312 'type': 'executable', 293 'type': 'executable',
313 'dependencies': [ 294 'dependencies': [
314 'libadblockplus/third_party/googletest.gyp:googletest_main', 295 'libadblockplus/third_party/googletest.gyp:googletest_main',
315 ], 296 ],
(...skipping 12 matching lines...) Expand all
328 'msvs_settings': { 309 'msvs_settings': {
329 'VCLinkerTool': { 310 'VCLinkerTool': {
330 'SubSystem': '1', # Console 311 'SubSystem': '1', # Console
331 'EntryPointSymbol': 'mainCRTStartup', 312 'EntryPointSymbol': 'mainCRTStartup',
332 }, 313 },
333 }, 314 },
334 }, 315 },
335 316
336 ] 317 ]
337 } 318 }
LEFTRIGHT

Powered by Google App Engine
This is Rietveld