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

Side by Side Diff: chrome/test/ppapi/ppapi_browsertest.cc

Issue 2017113002: [Extensions] DCHECK that ExtensionFunctions respond (and only once) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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 (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 <stddef.h> 5 #include <stddef.h>
6 6
7 #include "base/macros.h" 7 #include "base/macros.h"
8 #include "base/path_service.h" 8 #include "base/path_service.h"
9 #include "base/test/test_timeouts.h" 9 #include "base/test/test_timeouts.h"
10 #include "base/win/windows_version.h" 10 #include "base/win/windows_version.h"
(...skipping 1241 matching lines...) Expand 10 before | Expand all | Expand 10 after
1252 ASSERT_TRUE(extension); 1252 ASSERT_TRUE(extension);
1253 1253
1254 AppLaunchParams params(browser()->profile(), extension, 1254 AppLaunchParams params(browser()->profile(), extension,
1255 extensions::LAUNCH_CONTAINER_NONE, NEW_WINDOW, 1255 extensions::LAUNCH_CONTAINER_NONE, NEW_WINDOW,
1256 extensions::SOURCE_TEST); 1256 extensions::SOURCE_TEST);
1257 params.command_line = *base::CommandLine::ForCurrentProcess(); 1257 params.command_line = *base::CommandLine::ForCurrentProcess();
1258 OpenApplication(params); 1258 OpenApplication(params);
1259 } 1259 }
1260 1260
1261 void RunTests(const std::string& extension_dirname) { 1261 void RunTests(const std::string& extension_dirname) {
1262 ExtensionTestMessageListener listener("PASS", true); 1262 ExtensionTestMessageListener listener("PASS", false);
1263 LaunchTestingApp(extension_dirname); 1263 LaunchTestingApp(extension_dirname);
1264 EXPECT_TRUE(listener.WaitUntilSatisfied()); 1264 EXPECT_TRUE(listener.WaitUntilSatisfied());
1265 } 1265 }
1266 1266
1267 protected: 1267 protected:
1268 std::string toolchain_; 1268 std::string toolchain_;
1269 }; 1269 };
1270 1270
1271 class NewlibPackagedAppTest : public PackagedAppTest { 1271 class NewlibPackagedAppTest : public PackagedAppTest {
1272 public: 1272 public:
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
1309 MAYBE_PPAPI_NACL(NoSocketPermissions)) { 1309 MAYBE_PPAPI_NACL(NoSocketPermissions)) {
1310 RunTests("no_socket_permissions"); 1310 RunTests("no_socket_permissions");
1311 } 1311 }
1312 1312
1313 IN_PROC_BROWSER_TEST_F(NewlibPackagedAppTest, 1313 IN_PROC_BROWSER_TEST_F(NewlibPackagedAppTest,
1314 MAYBE_PPAPI_NACL(SocketPermissions)) { 1314 MAYBE_PPAPI_NACL(SocketPermissions)) {
1315 RunTests("socket_permissions"); 1315 RunTests("socket_permissions");
1316 } 1316 }
1317 1317
1318 #endif 1318 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698