Remove WEBKIT_DLL and replace it with COMPONENT_BUILD

Source/WebKit/chromium/WebKitUnitTests.gyp

 #
 # Copyright (C) 2011 Google Inc. All rights reserved.
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #         * Redistributions of source code must retain the above copyright
 # notice, this list of conditions and the following disclaimer.
 #         * Redistributions in binary form must reproduce the above
@@ skipping 45 matching lines @@
                 '<(DEPTH)/webkit/support/webkit_support.gyp:webkit_support',
             ],
             'sources': [
                 'tests/RunAllTests.cpp',
             ],
             'include_dirs': [
                 'public',
                 'src',
             ],
             'conditions': [
-                ['component=="shared_library"', {
-                    'defines': [
-                        'WEBKIT_DLL_UNITTEST',

[darin (slow to review), 2013/06/19 21:05:47]

I think this change nuked webkit_unit_tests in the component=shared_library
build.  Now, webkit_unit_tests has no tests!  The WEBKIT_DLL_UNITTEST define was
needed so that RunAllTests.cpp would know to find the tests in webkit.dll
instead of inside webkit_unit_tests.exe.

[jamesr, 2013/06/19 21:06:52]

Yikes, that's right!  Why did you remove this, Jochen?  It doesn't seem related
to the rest of the patch.

[jochen (gone - plz use gerrit), 2013/06/20 10:49:12]

OMG, that was a careless mistake :-/

Sorry about that and thanks for fixing

-                    ],
-                }, {
+                ['component!="shared_library"', {
                     'dependencies': [
                         '../../core/core.gyp:webcore',
                     ],
                     'defines': [
                         'WEBKIT_IMPLEMENTATION=1',
                         'INSIDE_WEBKIT',
                     ],
                     'sources': [
                         '<@(core_unittest_files)',
                         '<@(modules_unittest_files)',
@@ skipping 70 matching lines @@
         ['clang==1', {
             'target_defaults': {
                 'cflags': ['-Wunused-parameter'],
                 'xcode_settings': {
                     'WARNING_CFLAGS': ['-Wunused-parameter'],
                 },
             },
         }],
     ],
 }