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

Side by Side Diff: chrome/test/nacl/nacl_browsertest_util.h

Issue 1005173006: Add a switch for using PNaCl Subzero and use it for -O0 translation. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: stuff Created 5 years, 9 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 (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 #ifndef CHROME_TEST_NACL_NACL_BROWSERTEST_UTIL_H_ 5 #ifndef CHROME_TEST_NACL_NACL_BROWSERTEST_UTIL_H_
6 #define CHROME_TEST_NACL_NACL_BROWSERTEST_UTIL_H_ 6 #define CHROME_TEST_NACL_NACL_BROWSERTEST_UTIL_H_
7 7
8 #include "base/files/file_path.h" 8 #include "base/files/file_path.h"
9 #include "base/memory/scoped_ptr.h" 9 #include "base/memory/scoped_ptr.h"
10 #include "chrome/test/base/in_process_browser_test.h" 10 #include "chrome/test/base/in_process_browser_test.h"
(...skipping 105 matching lines...) Expand 10 before | Expand all | Expand 10 after
116 base::FilePath::StringType Variant() override; 116 base::FilePath::StringType Variant() override;
117 }; 117 };
118 118
119 class NaClBrowserTestPnacl : public NaClBrowserTestBase { 119 class NaClBrowserTestPnacl : public NaClBrowserTestBase {
120 public: 120 public:
121 base::FilePath::StringType Variant() override; 121 base::FilePath::StringType Variant() override;
122 122
123 bool IsAPnaclTest() override; 123 bool IsAPnaclTest() override;
124 }; 124 };
125 125
126 class NaClBrowserTestPnaclSubzero : public NaClBrowserTestPnacl {
Nick Bray (chromium) 2015/03/27 20:30:42 Comment about this being temporary (I assume) and
jvoung (off chromium) 2015/03/28 01:18:34 Done.
127 public:
128 void SetUpCommandLine(base::CommandLine* command_line) override;
129 };
130
126 class NaClBrowserTestPnaclNonSfi : public NaClBrowserTestBase { 131 class NaClBrowserTestPnaclNonSfi : public NaClBrowserTestBase {
127 public: 132 public:
128 void SetUpCommandLine(base::CommandLine* command_line) override; 133 void SetUpCommandLine(base::CommandLine* command_line) override;
129 base::FilePath::StringType Variant() override; 134 base::FilePath::StringType Variant() override;
130 }; 135 };
131 136
132 // "Transitional" here means that this uses nacl_helper_nonsfi which has 137 // "Transitional" here means that this uses nacl_helper_nonsfi which has
133 // nacl_helper feature for Non-SFI mode, but statically linked to newlib 138 // nacl_helper feature for Non-SFI mode, but statically linked to newlib
134 // instead of using host glibc. It is still under development. 139 // instead of using host glibc. It is still under development.
135 // TODO(hidehiko): Switch NonSfi tests to use nacl_helper_nonsfi, when 140 // TODO(hidehiko): Switch NonSfi tests to use nacl_helper_nonsfi, when
(...skipping 95 matching lines...) Expand 10 before | Expand all | Expand 10 after
231 236
232 #define NACL_BROWSER_TEST_F(suite, name, body) \ 237 #define NACL_BROWSER_TEST_F(suite, name, body) \
233 IN_PROC_BROWSER_TEST_F(suite##Newlib, name) \ 238 IN_PROC_BROWSER_TEST_F(suite##Newlib, name) \
234 body \ 239 body \
235 IN_PROC_BROWSER_TEST_F(suite##GLibc, MAYBE_GLIBC(name)) \ 240 IN_PROC_BROWSER_TEST_F(suite##GLibc, MAYBE_GLIBC(name)) \
236 body \ 241 body \
237 IN_PROC_BROWSER_TEST_F(suite##Pnacl, MAYBE_PNACL(name)) \ 242 IN_PROC_BROWSER_TEST_F(suite##Pnacl, MAYBE_PNACL(name)) \
238 body 243 body
239 244
240 #endif // CHROME_TEST_NACL_NACL_BROWSERTEST_UTIL_H_ 245 #endif // CHROME_TEST_NACL_NACL_BROWSERTEST_UTIL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698