Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2)

Unified Diff: chrome/browser/extensions/extension_commands_global_registry_apitest.cc

Issue 2071703002: Disable remaining tests that fail on the Linux desktop CrOS GN builds. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix compile failure, disable tests instead of commenting them out Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | content/test/ppapi/ppapi_browsertest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/extension_commands_global_registry_apitest.cc
diff --git a/chrome/browser/extensions/extension_commands_global_registry_apitest.cc b/chrome/browser/extensions/extension_commands_global_registry_apitest.cc
index 34ad9c66637ef7158c2ac7e1ba42aa47b5fe3eb1..bc589a451214e05cd75498cd14c2ed302385f7c6 100644
--- a/chrome/browser/extensions/extension_commands_global_registry_apitest.cc
+++ b/chrome/browser/extensions/extension_commands_global_registry_apitest.cc
@@ -119,6 +119,13 @@ void SendNativeCommandShift(int key_code) {
// doesn't have focus. Also test that non-global commands are not treated as
// global and that keys beyond Ctrl+Shift+[0..9] cannot be auto-assigned by an
// extension.
+//
+// Doesn't work in GN CrOS ozone builds yet, http://crbug.com/619784
+#if defined(OS_CHROMEOS) && defined(USE_OZONE)
+#define MAYBE_GlobalCommand DISABLED_GlobalCommand
+#else
+#define MAYBE_GlobalCommand GlobalCommand
+#endif
IN_PROC_BROWSER_TEST_F(GlobalCommandsApiTest, GlobalCommand) {
// Load the extension in the non-incognito browser.
ResultCatcher catcher;
« no previous file with comments | « no previous file | content/test/ppapi/ppapi_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698