Chromium Code Reviews| Index: chrome/test/BUILD.gn |
| diff --git a/chrome/test/BUILD.gn b/chrome/test/BUILD.gn |
| index e0f20cfafdfc1cb440cdd793036a36745f8f20cc..7334db97a8b75056e9790dae5cb01bbdca9848c1 100644 |
| --- a/chrome/test/BUILD.gn |
| +++ b/chrome/test/BUILD.gn |
| @@ -1155,8 +1155,6 @@ if (!is_android) { |
| sources -= [ |
| "../../apps/load_and_launch_browsertest.cc", |
| "../browser/policy/policy_startup_browsertest.cc", |
| - "../browser/printing/cloud_print/test/cloud_print_policy_browsertest.cc", |
| - "../browser/printing/cloud_print/test/cloud_print_proxy_process_browsertest.cc", |
| # chromeos does not support profile list avatar menu |
| "../browser/profiles/profile_list_desktop_browsertest.cc", |
| @@ -1310,10 +1308,6 @@ if (!is_android) { |
| # 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", |
| ] |
| if (safe_browsing_mode == 1) { |
| @@ -1428,8 +1422,6 @@ if (!is_android) { |
| if (!enable_print_preview) { |
| sources -= [ |
| "../browser/extensions/api/cloud_print_private/cloud_print_private_apitest.cc", |
| - "../browser/printing/cloud_print/test/cloud_print_policy_browsertest.cc", |
| - "../browser/printing/cloud_print/test/cloud_print_proxy_process_browsertest.cc", |
| "../browser/printing/print_preview_dialog_controller_browsertest.cc", |
| "../browser/printing/print_preview_pdf_generated_browsertest.cc", |
| "../browser/service_process/service_process_control_browsertest.cc", |
| @@ -1437,6 +1429,19 @@ if (!is_android) { |
| "data/webui/print_preview.cc", |
| "data/webui/print_preview.h", |
| ] |
| + if (is_mac || is_chromeos) { |
| + sources -= [ |
| + # This test depends on GetCommandLineForRelaunch, which is not |
| + # available on Mac. It is also not intended to run on ChromeOS. |
| + "../browser/printing/cloud_print/test/cloud_print_policy_browsertest.cc", |
|
Robert Sesek
2016/06/17 21:46:55
I think this has to be excluded even if enable_pri
Lei Zhang
2016/06/17 21:57:56
Right.
|
| + ] |
| + } |
| + if (is_chromeos) { |
| + sources -= [ |
| + # Not intended to run on ChromeOS. |
| + "../browser/printing/cloud_print/test/cloud_print_proxy_process_browsertest.cc", |
| + ] |
| + } |
| } |
| if (enable_mdns) { |
| sources += [ |