Remove "#pragma comment(lib" in components.

components/crash.gypi

 # Copyright 2013 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.
 
 {
   'targets': [
     {
       # GN version: //components/crash/core/browser
       'target_name': 'crash_core_browser',
       'type': 'static_library',
@@ skipping 280 matching lines @@
               ],
             }],
             ['OS=="win"', {
               'dependencies': [
                 '../breakpad/breakpad.gyp:breakpad_handler',
               ],
               'include_dirs': [
                 '..',
                 '../breakpad/src',
               ],
+              'all_dependent_settings': {

[Nico, 2016/04/01 15:18:41]

Actually, the all_dependent_blocks probably aren't really needed -- for .lib
files they'll be forwarded to the next linkable, and downstream linkables should
specify these libs themselves if they need them explicitly, right?

[jam, 2016/04/01 15:32:07]

In the other places where I've been adding .libs, I've seen the pattern where
they all specify them twice in all_dependent_settings and also outside of it
like this. i.e.
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/chrome_brow...

granted, I don't particularly care about gyp files at this point. i only care
about making sure that gn files are close to perfect :)

+                'msvs_settings': {
+                  'VCLinkerTool': {
+                    'AdditionalDependencies': [
+                      'userenv.lib',
+                    ],
+                  },
+                },
+              },
+              'msvs_settings': {
+                'VCLinkerTool': {
+                  'AdditionalDependencies': [
+                    'userenv.lib',
+                  ],
+                },
+              },
             }],
           ],
         },
         {
           # GN version: //components/crash/content/app:test_support
           'target_name': 'crash_test_support',
           'type': 'none',
           'dependencies': [
             'crash_component_lib',
           ],
@@ skipping 55 matching lines @@
                 '../base/base.gyp:base_win64',
                 '../breakpad/breakpad.gyp:breakpad_handler_win64',
                 '../breakpad/breakpad.gyp:breakpad_sender_win64',
                 '../sandbox/sandbox.gyp:sandbox_win64',
               ],
               'configurations': {
                 'Common_Base': {
                   'msvs_target_platform': 'x64',
                 },
               },
+              'conditions': [
+                ['OS=="win"', {
+                  'all_dependent_settings': {
+                    'msvs_settings': {
+                      'VCLinkerTool': {
+                        'AdditionalDependencies': [
+                          'userenv.lib',
+                        ],
+                      },
+                    },
+                  },
+                  'msvs_settings': {
+                    'VCLinkerTool': {
+                      'AdditionalDependencies': [
+                        'userenv.lib',
+                      ],
+                    },
+                  },
+                }],
+              ],
             },
             {
               # GN version: //components/crash/content/tools:crash_service
               'target_name': 'breakpad_crash_service_win64',
               'type': 'static_library',
               'dependencies': [
                 '../base/base.gyp:base_win64',
                 '../breakpad/breakpad.gyp:breakpad_handler_win64',
                 '../breakpad/breakpad.gyp:breakpad_sender_win64',
               ],
@@ skipping 59 matching lines @@
                   ],
                 }],
               ],
             },
           ],
         }],
       ],
     }],
   ],
 }