[sync] Componentize sync: Part 4: Fix includes in third_party/protobuf

third_party/protobuf/protobuf.gyp

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 {
   'conditions': [
     ['use_system_protobuf==0', {
       'conditions': [
         ['OS!="win"', {
           'variables': {
@@ skipping 163 matching lines @@
           'include_dirs': [
             '<(config_h_dir)',
             'src',
           ],
           # This macro must be defined to suppress the use of dynamic_cast<>,
           # which requires RTTI.
           'defines': [
             'GOOGLE_PROTOBUF_NO_RTTI',
             'GOOGLE_PROTOBUF_NO_STATIC_INITIALIZER',
           ],
-
+          # On Win component builds, we must dynamically link protobuf_lite to
+          # avoid warning C4275: "non dll-interface class used as base for
+          # dll-interface class". See the "DLLs vs. static linking" section in
+          # third_party/protobuf/vsprojects/readme.txt.
+          'conditions': [
+           ['OS=="win" and component=="shared_library"', {
+             'defines': [
+               'LIBPROTOBUF_EXPORTS',
+               'LIBPROTOC_EXPORTS',
+               'PROTOBUF_USE_DLLS',
+             ],
+           }],
+          ],
           'direct_dependent_settings': {
             'include_dirs': [
               '<(config_h_dir)',
               'src',
             ],
             'defines': [
               'GOOGLE_PROTOBUF_NO_RTTI',
               'GOOGLE_PROTOBUF_NO_STATIC_INITIALIZER',
             ],
+            # On Win component builds, we must dynamically link protobuf_lite to

[akalin, 2012/12/13 02:04:50]

it looks like the comment for the main define block is omitted when copied to
this one.  Perhaps copy it also?  Or remove this copied comment block.

[Raghu Simha, 2012/12/13 20:07:49]

Nice catch. I think it makes sense to delete the copied block since it's right
below the original. Done.

+            # avoid warning C4275: "non dll-interface class used as base for
+            # dll-interface class". See the "DLLs vs. static linking" section in
+            # third_party/protobuf/vsprojects/readme.txt.
+            'conditions': [
+             ['OS=="win" and component=="shared_library"', {
+               'defines': [
+                 'LIBPROTOBUF_EXPORTS',
+                 'LIBPROTOC_EXPORTS',
+                 'PROTOBUF_USE_DLLS',
+               ],
+             }],
+            ],
           },
         },
         # This is the full, heavy protobuf lib that's needed for c++ .proto's
         # that don't specify the LITE_RUNTIME option.  The protocol
         # compiler itself (protoc) falls into that category.
         #
         # DO NOT LINK AGAINST THIS TARGET IN CHROME CODE  --agl
         {
           'target_name': 'protobuf_full_do_not_use',
           'type': 'static_library',
@@ skipping 267 matching lines @@
           'toolsets': ['host', 'target'],
         },
         {
           'target_name': 'py_proto',
           'type': 'none',
         },
       ],
     }],
   ],
 }