allocator cleanup: remove dependencies on allocator from all targets

third_party/WebKit/Source/platform/blink_platform_tests.gyp

 #
 # Copyright (C) 2013 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 35 matching lines @@
         'blink_platform.gyp:blink_platform',
       ],
       'defines': [
         'INSIDE_BLINK',
       ],
       'sources': [
         'heap/RunAllTests.cpp',
         '<@(platform_heap_test_files)',
       ],
       'conditions': [
-        ['os_posix==1 and OS!="mac" and OS!="android" and OS!="ios" and use_allocator!="none"', {
-          'dependencies': [
-            '<(DEPTH)/base/base.gyp:base',

[petrcermak, 2016/01/28 14:58:50]

Is the dependency on base supposed to be removed as well?

[Primiano Tucci (use gerrit), 2016/01/28 15:20:03]

Yup it is redundant as they get it via test_support_base. There is really no
reason why they should additionally depend on base, but only on some platforms.
But thanks for double-checking. This is precisely the reason why I asked for a
review. 

-            '<(DEPTH)/base/allocator/allocator.gyp:allocator',
-          ]
-        }],
-        ['OS=="win" and component!="shared_library" and win_use_allocator_shim==1', {
-          'dependencies': [
-            '<(DEPTH)/base/allocator/allocator.gyp:allocator',
-          ]
-        }],
         ['OS=="android"', {
           'type': 'shared_library',
           'dependencies': [
             '<(DEPTH)/testing/android/native_test.gyp:native_test_native_code',
             '<(DEPTH)/tools/android/forwarder2/forwarder.gyp:forwarder2',
           ],
         }],
       ],
     },
     {
@@ skipping 20 matching lines @@
         'INSIDE_BLINK',
       ],
       'include_dirs': [
         '<(SHARED_INTERMEDIATE_DIR)/blink',
       ],
       'sources': [
         'testing/RunAllTests.cpp',
         '<@(platform_test_files)',
       ],
       'conditions': [
-        ['os_posix==1 and OS!="mac" and OS!="android" and OS!="ios" and use_allocator!="none"', {
-          'dependencies': [
-            '<(DEPTH)/base/base.gyp:base',

[petrcermak, 2016/01/28 14:58:50]

ditto

[Primiano Tucci (use gerrit), 2016/01/28 15:20:03]

Ack.

-            '<(DEPTH)/base/allocator/allocator.gyp:allocator',
-          ]
-        }],
         ['OS=="android" and gtest_target_type == "shared_library"', {
           'type': 'shared_library',
           'dependencies': [
             '<(DEPTH)/testing/android/native_test.gyp:native_test_native_code',
             '<(DEPTH)/tools/android/forwarder2/forwarder.gyp:forwarder2',
           ],
         }],
       ],
     },
     {
@@ skipping 103 matching lines @@
             '../../../../build/isolate.gypi',
           ],
           'sources': [
             'blink_platform_unittests.isolate',
           ],
         }
       ],
     }],
   ],
 }