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

chrome/browser/ui/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/chromecast_build.gni")
 import("//build/config/crypto.gni")
 import("//build/config/features.gni")
 import("//build/config/ui.gni")
 import("//build/split_static_library.gni")
 import("//chrome/common/features.gni")
@@ skipping 1528 matching lines @@
       deps += [ "//extensions/components/native_app_window" ]
     }
 
     if (is_chromeos) {
       sources += [
         "views/apps/app_info_dialog/arc_app_info_links_panel.cc",
         "views/apps/app_info_dialog/arc_app_info_links_panel.h",
       ]
     } else if (!is_mac || mac_views_browser) {
       sources += [
-        "external_protocol_dialog_delegate.cc",
-        "external_protocol_dialog_delegate.h",
-        "views/external_protocol_dialog.cc",
-        "views/external_protocol_dialog.h",
         "views/frame/opaque_browser_frame_view.cc",
         "views/frame/opaque_browser_frame_view.h",
         "views/frame/opaque_browser_frame_view_layout.cc",
         "views/frame/opaque_browser_frame_view_layout.h",
         "views/frame/opaque_browser_frame_view_layout_delegate.h",
         "views/frame/opaque_browser_frame_view_linux.cc",
         "views/frame/opaque_browser_frame_view_linux.h",
         "views/frame/opaque_browser_frame_view_platform_specific.cc",
         "views/frame/opaque_browser_frame_view_platform_specific.h",
         "views/profiles/profile_chooser_view.cc",
         "views/profiles/profile_chooser_view.h",
         "views/screen_capture_notification_ui_views.cc",
         "views/sync/one_click_signin_dialog_view.cc",
         "views/sync/one_click_signin_dialog_view.h",
         "views/sync/profile_signin_confirmation_dialog_views.cc",
         "views/sync/profile_signin_confirmation_dialog_views.h",
       ]
     }
 
+    if (!is_chromeos) {
+      sources += [
+        "external_protocol_dialog_delegate.cc",
+        "external_protocol_dialog_delegate.h",
+        "views/external_protocol_dialog.cc",
+        "views/external_protocol_dialog.h",
+      ]
+    }
+
     # These files do Gtk+-based theming, but Gtk+ is not available on
     # Ozone builds, or on common environments where Ozone will be used.
     #
     # TODO(tonikitoo): It seems sensible to make it possible to opt-in
     # the use of these files in case Gtk is available with its Wayland.
     if (use_aura && !use_ozone && is_desktop_linux) {
       sources += [
         "views/chrome_browser_main_extra_parts_views_linux.cc",
         "views/chrome_browser_main_extra_parts_views_linux.h",
       ]
@@ skipping 366 matching lines @@
       ]
     }
   }
   if (use_gio) {
     deps += [ "//build/linux/libgio" ]
   }
   if (use_aura && !use_ozone && is_desktop_linux) {
     # This is the only component that can interact with gtk.
     deps += [ "//chrome/browser/ui/libgtkui" ]
   }
   if (is_win || is_mac || is_desktop_linux) {

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

I think this is the right place for the files. (the whole thing could do with a
cleanup, but we should try to re-use existing blocks if possible)

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

Done. Toolkit views is defined only on all desktop platforms currently right?

     sources += [
       "avatar_button_error_controller.cc",
       "avatar_button_error_controller.h",
       "avatar_button_error_controller_delegate.h",
       "startup/default_browser_infobar_delegate.cc",
       "startup/default_browser_infobar_delegate.h",
       "startup/default_browser_prompt.cc",
       "startup/default_browser_prompt.h",
       "startup/default_browser_prompt_win.cc",
       "sync/one_click_signin_links_delegate.h",
@@ skipping 711 matching lines @@
         "cocoa/extensions/extension_view_mac.mm",
         "cocoa/extensions/media_galleries_dialog_cocoa.h",
         "cocoa/extensions/media_galleries_dialog_cocoa.mm",
         "cocoa/extensions/media_gallery_list_entry_view.h",
         "cocoa/extensions/media_gallery_list_entry_view.mm",
         "cocoa/extensions/toolbar_actions_bar_bubble_mac.h",
         "cocoa/extensions/toolbar_actions_bar_bubble_mac.mm",
         "cocoa/extensions/windowed_install_dialog_controller.h",
         "cocoa/extensions/windowed_install_dialog_controller.mm",
         "cocoa/external_protocol_dialog.h",
-        "cocoa/external_protocol_dialog.mm",
+        "cocoa/external_protocol_dialog_cocoa.mm",
+        "cocoa/external_protocol_dialog_views_mac.mm",
         "cocoa/fast_resize_view.h",
         "cocoa/fast_resize_view.mm",
         "cocoa/find_bar/find_bar_bridge.h",
         "cocoa/find_bar/find_bar_bridge.mm",
         "cocoa/find_bar/find_bar_cocoa_controller.h",
         "cocoa/find_bar/find_bar_cocoa_controller.mm",
         "cocoa/find_bar/find_bar_text_field.h",
         "cocoa/find_bar/find_bar_text_field.mm",
         "cocoa/find_bar/find_bar_text_field_cell.h",
         "cocoa/find_bar/find_bar_text_field_cell.mm",
@@ skipping 796 matching lines @@
       "test/test_confirm_bubble_model.cc",
       "test/test_confirm_bubble_model.h",
     ]
     deps += [ "//chrome/test:test_support_ui" ]
   }
 
   if (enable_extensions) {
     deps += [ "//extensions/browser" ]
   }
 }