Remove the old task manager view

chrome/chrome_tests_unit.gypi

Index: chrome/chrome_tests_unit.gypi
diff --git a/chrome/chrome_tests_unit.gypi b/chrome/chrome_tests_unit.gypi
index ba10ca96472c7d52bc4d5f8d8bf2b12e47e91a32..5fdcec50a0505685e08cb11a3d482309a39ede1c 100644
--- a/chrome/chrome_tests_unit.gypi
+++ b/chrome/chrome_tests_unit.gypi
@@ -720,7 +720,9 @@
       'browser/task_management/task_manager_observer_unittest.cc',
       'browser/task_management/test_task_manager.cc',
       'browser/task_management/test_task_manager.h',
-      # Old Task Manager Tests Sources:
+    ],
+    'chrome_unit_tests_old_task_manager_mac_sources': [
+      # Old Task Manager Tests Sources (enabled only on mac):
       'browser/task_manager/task_manager_unittest.cc',

[tapted, 2016/04/26 01:54:04]

Move these to chrome_unit_tests_cocoa_sources (with the comment) - we don't need
the extra variable

[afakhry, 2016/04/26 22:22:44]

Done.

       'browser/task_manager/task_manager_util_unittest.cc',
     ],
@@ -2442,6 +2444,13 @@
         ['enable_task_manager==1', {

[tapted, 2016/04/26 01:54:04]

It's weird to have this for test targets but not the main browser in
chrome_browser.gypi [we should probably remove enable_task_manager and just use
!android, but that's orthogonal). I guess this CL doesn't make things
significantly worse.

           'sources': [ '<@(chrome_unit_tests_task_manager_sources)' ],
           'dependencies': [ '../gpu/gpu.gyp:gpu' ],
+          'conditions': [
+            ['OS == "mac"', {
+              'sources': [
+                '<@(chrome_unit_tests_old_task_manager_mac_sources)'
+              ],
+            }],
+          ],
         }],
         ['enable_mdns==1', {
           'sources': [ '<@(chrome_unit_tests_mdns_sources)' ],