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

Unified Diff: chrome/test/ppapi/ppapi_test.cc

Issue 1062163004: Plugins: Move Prerender tests to PPAPI and some refactoring. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 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
Index: chrome/test/ppapi/ppapi_test.cc
diff --git a/chrome/test/ppapi/ppapi_test.cc b/chrome/test/ppapi/ppapi_test.cc
index 346398d33854bfc1fcb5aa349b7abc8bc77b43b0..34d5cf9e22fbd8ae9f2955a650d120b3191c49db 100644
--- a/chrome/test/ppapi/ppapi_test.cc
+++ b/chrome/test/ppapi/ppapi_test.cc
@@ -26,11 +26,11 @@
#include "content/public/browser/notification_service.h"
#include "content/public/browser/web_contents.h"
#include "content/public/common/content_switches.h"
+#include "content/public/test/ppapi_test_utils.h"
#include "media/base/media_switches.h"
#include "net/base/filename_util.h"
#include "net/base/test_data_directory.h"
#include "ppapi/shared_impl/ppapi_switches.h"
-#include "ppapi/shared_impl/test_harness_utils.h"
#include "ui/gl/gl_switches.h"
using content::DomOperationNotificationDetails;
@@ -280,24 +280,13 @@ PPAPITest::PPAPITest() : in_process_(true) {
void PPAPITest::SetUpCommandLine(base::CommandLine* command_line) {
PPAPITestBase::SetUpCommandLine(command_line);
- base::FilePath plugin_dir;
- EXPECT_TRUE(PathService::Get(base::DIR_MODULE, &plugin_dir));
-
- base::FilePath plugin_lib = plugin_dir.Append(ppapi::GetTestLibraryName());
- EXPECT_TRUE(base::PathExists(plugin_lib));
-
- // Append the switch to register the pepper plugin.
- // library path = <out dir>/<test_name>.<library_extension>
// library name = "PPAPI Tests"
// version = "1.2.3"
- // MIME type = application/x-ppapi-<test_name>
- base::FilePath::StringType pepper_plugin = plugin_lib.value();
- pepper_plugin.append(FILE_PATH_LITERAL("#PPAPI Tests"));
- pepper_plugin.append(FILE_PATH_LITERAL("#")); // No description.
- pepper_plugin.append(FILE_PATH_LITERAL("#1.2.3"));
- pepper_plugin.append(FILE_PATH_LITERAL(";application/x-ppapi-tests"));
- command_line->AppendSwitchNative(switches::kRegisterPepperPlugins,
- pepper_plugin);
+ base::FilePath::StringType parameters;
+ parameters.append(FILE_PATH_LITERAL("#PPAPI Tests"));
+ parameters.append(FILE_PATH_LITERAL("#")); // No description.
+ parameters.append(FILE_PATH_LITERAL("#1.2.3"));
+ ppapi::RegisterTestLibraryWithExtraParameters(command_line, parameters);
command_line->AppendSwitchASCII(switches::kAllowNaClSocketAPI, "127.0.0.1");
if (in_process_)

Powered by Google App Engine
This is Rietveld 408576698