MacViews: Enable views based External Protocol dialog behind secondary-ui-md flag.

chrome/test/BUILD.gn

 # Copyright 2014 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.
 
 import("//build/config/chrome_build.gni")
 import("//build/config/compiler/compiler.gni")
 import("//build/config/crypto.gni")
 import("//build/config/features.gni")
 import("//build/config/ui.gni")
 import("//chrome/common/features.gni")
@@ skipping 2119 matching lines @@
       deps += [ "//ui/events:test_support" ]
     }
     if (toolkit_views) {
       sources += [
         "../browser/ui/global_error/global_error_service_browsertest.cc",
         "../browser/ui/views/frame/browser_non_client_frame_view_browsertest_win.cc",
         "../browser/ui/views/frame/browser_window_property_manager_browsertest_win.cc",
         "../browser/ui/views/select_file_dialog_extension_browsertest.cc",
       ]
       deps += [ "//ui/views" ]
+      if (!is_chromeos) {
+        sources +=
+            [ "../browser/ui/views/external_protocol_dialog_browsertest.cc" ]
+      }
       if (!is_chromeos && (!is_mac || mac_views_browser)) {
-        sources += [
-          "../browser/ui/views/external_protocol_dialog_browsertest.cc",
-          "../browser/ui/views/profiles/profile_chooser_view_browsertest.cc",
-        ]
+        sources += [ "../browser/ui/views/profiles/profile_chooser_view_browsertest.cc" ]
       }
       if (!is_mac || mac_views_browser) {
         sources += [
           # TODO(tapted): Move these to chrome_browser_tests_views_sources when the
           # the corresponding files are moved in chrome_browser_ui.gypi (i.e. out of
           # chrome_browser_ui_views_non_mac_sources). http://crbug.com/404979.
           "../browser/ui/views/autofill/autofill_popup_base_view_browsertest.cc",
           "../browser/ui/views/autofill/card_unmask_prompt_view_tester_views.cc",
           "../browser/ui/views/autofill/card_unmask_prompt_view_tester_views.h",
           "../browser/ui/views/autofill/password_generation_popup_view_tester_views.cc",
@@ skipping 194 matching lines @@
         "../browser/extensions/api/vpn_provider/vpn_provider_apitest.cc",
         "../browser/ui/ash/launcher/arc_app_launcher_browsertest.cc",
         "../browser/ui/views/arc_app_dialog_view_browsertest.cc",
         "../browser/ui/webui/options/chromeos/accounts_options_browsertest.cc",
         "../browser/ui/webui/options/chromeos/guest_mode_options_browsertest.cc",
         "../browser/ui/webui/options/chromeos/guest_mode_options_ui_browsertest.cc",
         "../browser/ui/webui/options/chromeos/shared_options_browsertest.cc",
         "//ui/base/ime/chromeos/input_method_whitelist.cc",
         "//ui/base/ime/chromeos/input_method_whitelist.h",
       ]
       sources -= [

[tapted, 2017/01/24 04:02:43]

it would be nice to be consistent... I suppose this is the most correct section
to duplicate the browsertest file

[karandeepb, 2017/01/24 06:12:23]

Done.

         "../../apps/load_and_launch_browsertest.cc",
         "../browser/policy/policy_startup_browsertest.cc",
 
         # chromeos does not support profile list avatar menu
         "../browser/profiles/profile_list_desktop_browsertest.cc",
         "../browser/service_process/service_process_control_browsertest.cc",
 
         # inline login UI is disabled on chromeos
         "../browser/ui/webui/signin/inline_login_ui_browsertest.cc",
 
@@ skipping 2760 matching lines @@
 }
 
 if (is_win) {
   loadable_module("conflicts_dll") {
     testonly = true
     sources = [
       "conflicts/conflicts_dll.cc",
     ]
   }
 }