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

Side by Side Diff: content/test/ppapi/ppapi_browsertest.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: more tweaking 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "build/build_config.h" 5 #include "build/build_config.h"
6 #include "content/test/ppapi/ppapi_test.h" 6 #include "content/test/ppapi/ppapi_test.h"
7 #include "ppapi/shared_impl/test_utils.h" 7 #include "ppapi/shared_impl/test_utils.h"
8 8
9 // This file lists tests for Pepper APIs (without NaCl) against content_shell. 9 // This file lists tests for Pepper APIs (without NaCl) against content_shell.
10 // TODO(teravest): Move more tests here. http://crbug.com/371873 10 // TODO(teravest): Move more tests here. http://crbug.com/371873
(...skipping 18 matching lines...) Expand all
29 RunTest(STRIP_PREFIXES(test_name)); \ 29 RunTest(STRIP_PREFIXES(test_name)); \
30 } 30 }
31 31
32 // OutOfProcessPPAPITest tests time out under ThreadSanitizer, 32 // OutOfProcessPPAPITest tests time out under ThreadSanitizer,
33 // see https://crbug.com/448323. 33 // see https://crbug.com/448323.
34 #define TEST_PPAPI_OUT_OF_PROCESS(test_name) \ 34 #define TEST_PPAPI_OUT_OF_PROCESS(test_name) \
35 IN_PROC_BROWSER_TEST_F(OutOfProcessPPAPITest, DISABLE_IF_TSAN(test_name)) { \ 35 IN_PROC_BROWSER_TEST_F(OutOfProcessPPAPITest, DISABLE_IF_TSAN(test_name)) { \
36 RunTest(STRIP_PREFIXES(test_name)); \ 36 RunTest(STRIP_PREFIXES(test_name)); \
37 } 37 }
38 38
39 TEST_PPAPI_OUT_OF_PROCESS(BrowserFont) 39 // Doesn't work in GN CrOS builds yet, http://crbug.com/619765
40 #if defined(OS_CHROMEOS)
41 #define MAYBE_BrowserFont DISABLED_BrowserFont
stevenjb 2016/06/16 00:32:41 These only fail if defined(USE_OZONE), so maybe on
Dirk Pranke 2016/06/16 00:37:13 ok.
42 #else
43 #define MAYBE_BrowserFont BrowserFont
44 #endif
45 TTEST_PPAPI_OUT_OF_PROCESS(BrowserFont)
40 46
41 TEST_PPAPI_IN_PROCESS(Buffer) 47 TEST_PPAPI_IN_PROCESS(Buffer)
42 TEST_PPAPI_OUT_OF_PROCESS(Buffer) 48 TEST_PPAPI_OUT_OF_PROCESS(Buffer)
43 49
44 TEST_PPAPI_IN_PROCESS(CharSet) 50 TEST_PPAPI_IN_PROCESS(CharSet)
45 TEST_PPAPI_OUT_OF_PROCESS(CharSet) 51 TEST_PPAPI_OUT_OF_PROCESS(CharSet)
46 52
47 TEST_PPAPI_IN_PROCESS(Console) 53 TEST_PPAPI_IN_PROCESS(Console)
48 TEST_PPAPI_OUT_OF_PROCESS(Console) 54 TEST_PPAPI_OUT_OF_PROCESS(Console)
49 55
(...skipping 74 matching lines...) Expand 10 before | Expand all | Expand 10 after
124 TEST_PPAPI_OUT_OF_PROCESS(NetworkProxy) 130 TEST_PPAPI_OUT_OF_PROCESS(NetworkProxy)
125 131
126 // TODO(danakj): http://crbug.com/115286 132 // TODO(danakj): http://crbug.com/115286
127 TEST_PPAPI_IN_PROCESS(DISABLED_Scrollbar) 133 TEST_PPAPI_IN_PROCESS(DISABLED_Scrollbar)
128 // http://crbug.com/89961 134 // http://crbug.com/89961
129 TEST_PPAPI_OUT_OF_PROCESS(DISABLED_Scrollbar) 135 TEST_PPAPI_OUT_OF_PROCESS(DISABLED_Scrollbar)
130 136
131 TEST_PPAPI_IN_PROCESS(TraceEvent) 137 TEST_PPAPI_IN_PROCESS(TraceEvent)
132 TEST_PPAPI_OUT_OF_PROCESS(TraceEvent) 138 TEST_PPAPI_OUT_OF_PROCESS(TraceEvent)
133 139
134 TEST_PPAPI_OUT_OF_PROCESS(TrueTypeFont) 140 // Doesn't work in GN CrOS builds yet, http://crbug.com/619765
141 #if defined(OS_CHROMEOS)
142 #define MAYBE_TrueTypeFont DISABLED_TrueTypeFont
143 #else
144 #define MAYBE_TrueTypeFont TrueTypeFont
145 #endif
146 TEST_PPAPI_OUT_OF_PROCESS(MAYBE_TrueTypeFont)
135 147
136 TEST_PPAPI_IN_PROCESS(URLUtil) 148 TEST_PPAPI_IN_PROCESS(URLUtil)
137 TEST_PPAPI_OUT_OF_PROCESS(URLUtil) 149 TEST_PPAPI_OUT_OF_PROCESS(URLUtil)
138 150
139 TEST_PPAPI_IN_PROCESS(Var) 151 TEST_PPAPI_IN_PROCESS(Var)
140 TEST_PPAPI_OUT_OF_PROCESS(Var) 152 TEST_PPAPI_OUT_OF_PROCESS(Var)
141 153
142 // Flaky on mac, http://crbug.com/121107 154 // Flaky on mac, http://crbug.com/121107
143 #if defined(OS_MACOSX) 155 #if defined(OS_MACOSX)
144 #define MAYBE_VarDeprecated DISABLED_VarDeprecated 156 #define MAYBE_VarDeprecated DISABLED_VarDeprecated
(...skipping 14 matching lines...) Expand all
159 #endif 171 #endif
160 TEST_PPAPI_OUT_OF_PROCESS(MAYBE_VideoDecoder) 172 TEST_PPAPI_OUT_OF_PROCESS(MAYBE_VideoDecoder)
161 173
162 TEST_PPAPI_IN_PROCESS(VideoDecoderDev) 174 TEST_PPAPI_IN_PROCESS(VideoDecoderDev)
163 TEST_PPAPI_OUT_OF_PROCESS(VideoDecoderDev) 175 TEST_PPAPI_OUT_OF_PROCESS(VideoDecoderDev)
164 176
165 TEST_PPAPI_OUT_OF_PROCESS(VideoEncoder) 177 TEST_PPAPI_OUT_OF_PROCESS(VideoEncoder)
166 178
167 } // namespace 179 } // namespace
168 } // namespace content 180 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698