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

Unified Diff: chrome/browser/extensions/api/messaging/native_message_process_host_unittest.cc

Issue 22532011: Pass handle of the native view window to the native messaging host. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/api/messaging/native_message_process_host_unittest.cc
diff --git a/chrome/browser/extensions/api/messaging/native_message_process_host_unittest.cc b/chrome/browser/extensions/api/messaging/native_message_process_host_unittest.cc
index 8653d10646d4395ad08a285bef7a18dcd7ec6f1c..3c0ca5912a3a4a585fb3bbd77c513e06166f63f6 100644
--- a/chrome/browser/extensions/api/messaging/native_message_process_host_unittest.cc
+++ b/chrome/browser/extensions/api/messaging/native_message_process_host_unittest.cc
@@ -217,9 +217,11 @@ TEST_F(NativeMessagingTest, EchoConnect) {
CommandLine::ForCurrentProcess()->AppendSwitchASCII(
switches::kNativeMessagingHosts, hosts_option);
- native_message_process_host_ = NativeMessageProcessHost::Create(
+ scoped_ptr<NativeProcessLauncher> launcher =
+ NativeProcessLauncher::CreateDefault(gfx::NativeView());
+ native_message_process_host_ = NativeMessageProcessHost::CreateWithLauncher(
AsWeakPtr(), kTestNativeMessagingExtensionId,
- kTestNativeMessagingHostName, 0);
+ kTestNativeMessagingHostName, 0, launcher.Pass());
ASSERT_TRUE(native_message_process_host_.get());
native_message_process_host_->Send("{\"text\": \"Hello.\"}");

Powered by Google App Engine
This is Rietveld 408576698