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

Side by Side Diff: chrome/test/nacl/nacl_browsertest.cc

Issue 23326003: Enable PNaCl by default (revert disabling CL) (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixes for Jan's comments. Created 7 years, 4 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 <stdio.h> 5 #include <stdio.h>
6 #if defined(OS_POSIX) 6 #if defined(OS_POSIX)
7 #include <unistd.h> 7 #include <unistd.h>
8 #elif defined(OS_WIN) 8 #elif defined(OS_WIN)
9 #include <windows.h> 9 #include <windows.h>
10 #endif 10 #endif
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
51 }) 51 })
52 52
53 NACL_BROWSER_TEST_F(NaClBrowserTest, PPAPICore, { 53 NACL_BROWSER_TEST_F(NaClBrowserTest, PPAPICore, {
54 RunNaClIntegrationTest(FILE_PATH_LITERAL("ppapi_ppb_core.html")); 54 RunNaClIntegrationTest(FILE_PATH_LITERAL("ppapi_ppb_core.html"));
55 }) 55 })
56 56
57 NACL_BROWSER_TEST_F(NaClBrowserTest, ProgressEvents, { 57 NACL_BROWSER_TEST_F(NaClBrowserTest, ProgressEvents, {
58 RunNaClIntegrationTest(FILE_PATH_LITERAL("ppapi_progress_events.html")); 58 RunNaClIntegrationTest(FILE_PATH_LITERAL("ppapi_progress_events.html"));
59 }) 59 })
60 60
61 NACL_BROWSER_TEST_F(NaClBrowserTest, PnaclMimeType, {
62 RunLoadTest(FILE_PATH_LITERAL("pnacl_mime_type.html"));
63 })
64
65 // Some versions of Visual Studio does not like preprocessor 61 // Some versions of Visual Studio does not like preprocessor
66 // conditionals inside the argument of a macro, so we put the 62 // conditionals inside the argument of a macro, so we put the
67 // conditionals on a helper function. We are already in an anonymous 63 // conditionals on a helper function. We are already in an anonymous
68 // namespace, so the name of the helper is not visible in external 64 // namespace, so the name of the helper is not visible in external
69 // scope. 65 // scope.
70 #if defined(OS_POSIX) 66 #if defined(OS_POSIX)
71 base::FilePath::StringType NumberOfCoresAsFilePathString() { 67 base::FilePath::StringType NumberOfCoresAsFilePathString() {
72 char string_rep[23]; 68 char string_rep[23];
73 long nprocessors = sysconf(_SC_NPROCESSORS_ONLN); 69 long nprocessors = sysconf(_SC_NPROCESSORS_ONLN);
74 #if TELEMETRY 70 #if TELEMETRY
(...skipping 83 matching lines...) Expand 10 before | Expand all | Expand 10 after
158 RunNaClIntegrationTest(FILE_PATH_LITERAL( 154 RunNaClIntegrationTest(FILE_PATH_LITERAL(
159 "pnacl_dyncode_syscall_disabled.html")); 155 "pnacl_dyncode_syscall_disabled.html"));
160 } 156 }
161 157
162 IN_PROC_BROWSER_TEST_F(NaClBrowserTestPnacl, 158 IN_PROC_BROWSER_TEST_F(NaClBrowserTestPnacl,
163 MAYBE_PNACL(PnaclExceptionHandlingDisabled)) { 159 MAYBE_PNACL(PnaclExceptionHandlingDisabled)) {
164 RunNaClIntegrationTest(FILE_PATH_LITERAL( 160 RunNaClIntegrationTest(FILE_PATH_LITERAL(
165 "pnacl_exception_handling_disabled.html")); 161 "pnacl_exception_handling_disabled.html"));
166 } 162 }
167 163
164 IN_PROC_BROWSER_TEST_F(NaClBrowserTestPnacl, PnaclMimeType) {
165 RunLoadTest(FILE_PATH_LITERAL("pnacl_mime_type.html"));
166 }
167
168 IN_PROC_BROWSER_TEST_F(NaClBrowserTestPnaclDisabled, PnaclMimeType) {
169 RunLoadTest(FILE_PATH_LITERAL("pnacl_mime_type.html"));
170 }
171
168 } // namespace 172 } // namespace
OLDNEW
« no previous file with comments | « chrome/test/data/nacl/pnacl_mime_type/pnacl_mime_type.html ('k') | chrome/test/nacl/nacl_browsertest_uma.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698