MacViews: Fix WebDialogBrowserTest.SizeWindow to get browser_tests compiling on MacViews

chrome/chrome_tests.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.
 {
   'includes': [
     'js_unittest_vars.gypi',
   ],
   'targets': [
     {
       'target_name': 'interactive_ui_tests',
@@ skipping 1922 matching lines @@
           },
           # Other platforms only need
           # chrome_resources.gyp:{packed_extra_resources,packed_resources},
           # and can build this target standalone much faster.
           'dependencies': [
             'chrome',
             '../components/components.gyp:breakpad_stubs',
             '../third_party/ocmock/ocmock.gyp:ocmock',
           ],
           'sources': [
+            'browser/media_galleries/fileapi/iphoto_data_provider_browsertest.cc',
             'browser/renderer_host/chrome_render_widget_host_view_mac_delegate_browsertest.cc',
             'browser/renderer_host/chrome_render_widget_host_view_mac_history_swiper_browsertest.mm',
             'browser/spellchecker/spellcheck_message_filter_mac_browsertest.cc',
           ],
           'sources!': [
             # TODO(groby): This test depends on hunspell and we cannot run it on
             # Mac, which does not use hunspell by default.
             'browser/spellchecker/spellcheck_service_browsertest.cc',
+
+            # TODO(tapted): Enable toolkit-views browser_tests on Mac when their
+            # respective implementations are linked in. http://crbug.com/404980.

[msw, 2014/08/20 19:51:48]

Did you meant to mark that bug WontFix? Seems odd to mention it here if so.

[tapted, 2014/08/21 00:22:13]

nice catch - Thanks! I discovered crbug/I had simultaneously made two identical
bugs, and WontFixed the second one. But forgot I'd referenced the second one
here :/. Fixed.

+            'browser/ui/views/autofill/autofill_dialog_view_tester_views.cc',
+            'browser/ui/views/autofill/autofill_popup_base_view_browsertest.cc',
+            'browser/ui/views/extensions/extension_install_dialog_view_browsertest.cc',
+            'browser/ui/views/frame/browser_view_browsertest.cc',
+            'browser/ui/views/location_bar/zoom_bubble_view_browsertest.cc',
+            'browser/ui/views/profiles/avatar_menu_button_browsertest.cc',
+            'browser/ui/views/profiles/new_avatar_menu_button_browsertest.cc',
+            'browser/ui/views/profiles/profile_chooser_view_browsertest.cc',
+            'browser/ui/views/toolbar/browser_actions_container_browsertest.cc',
+            'browser/ui/views/translate/translate_bubble_view_browsertest.cc',
+
             # TODO(rouslan): This test depends on the custom dictionary UI,
             # which is disabled on Mac.
             'browser/ui/webui/options/edit_dictionary_browsertest.js',
             # TODO(rouslan): This test depends on hunspell and we cannot run it
             # on Mac, which does use hunspell by default.
             'browser/ui/webui/options/language_options_dictionary_download_browsertest.js',
             # ProcessSingletonMac doesn't do anything.
             'browser/process_singleton_browsertest.cc',
             # This test depends on GetCommandLineForRelaunch, which is not
             # available on Mac.
             'browser/printing/cloud_print/test/cloud_print_policy_browsertest.cc',
             # single-process mode hangs on Mac sometimes because of multiple UI
             # message loops. See 306348
             'renderer/safe_browsing/phishing_classifier_browsertest.cc',
             'renderer/safe_browsing/phishing_classifier_delegate_browsertest.cc',
             'renderer/safe_browsing/phishing_dom_feature_extractor_browsertest.cc',
           ],
-        }],
+        }],  # OS=="mac"
         ['OS=="mac" or OS=="win"', {
           'sources': [
             'browser/media_galleries/fileapi/itunes_data_provider_browsertest.cc',
             'browser/media_galleries/fileapi/picasa_data_provider_browsertest.cc',
           ],
         }],
-        ['OS=="mac"', {
-          'sources': [
-            'browser/media_galleries/fileapi/iphoto_data_provider_browsertest.cc',
-          ],
-        }],
         ['os_posix == 0 or chromeos == 1', {
           'sources!': [
             'common/time_format_browsertest.cc',
           ],
         }],
         ['OS=="android"', {
           'sources!': [
             'browser/policy/cloud/component_cloud_policy_browsertest.cc',
             'browser/prefs/pref_hash_browsertest.cc',
           ],
@@ skipping 1039 matching lines @@
             ['enable_webrtc==1', {
               'dependencies': [
                 '../third_party/libjingle/libjingle.gyp:libjingle_webrtc'
               ]
             }],
           ],
         }]
     }],
   ],  # 'conditions'
 }