Pass handle of the native view window to the native messaging host.

chrome/browser/extensions/api/messaging/native_message_process_host.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/extensions/api/messaging/native_message_process_host.h"
 
 #include "base/bind.h"
 #include "base/files/file_path.h"
 #include "base/logging.h"
 #include "base/platform_file.h"
 #include "base/values.h"
 #include "chrome/browser/extensions/api/messaging/native_messaging_host_manifest.h"
 #include "chrome/browser/extensions/api/messaging/native_process_launcher.h"
 #include "chrome/common/chrome_version_info.h"
 #include "chrome/common/extensions/features/feature.h"
+#include "content/public/browser/render_widget_host.h"
+#include "content/public/browser/render_widget_host_view.h"
 #include "extensions/common/constants.h"
 #include "net/base/file_stream.h"
 #include "net/base/io_buffer.h"
 #include "net/base/net_errors.h"
 #include "net/base/net_util.h"
 #include "url/gurl.h"
 
 namespace {
 
 // Maximum message size in bytes for messages received from Native Messaging
@@ skipping 44 matching lines @@
                  base::Unretained(this)));
 }
 
 NativeMessageProcessHost::~NativeMessageProcessHost() {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::IO));
   Close(std::string());
 }
 
 // static
 scoped_ptr<NativeMessageProcessHost> NativeMessageProcessHost::Create(
+    int source_process_id,

[Sergey Ulanov, 2013/08/07 22:51:20]

instead of passing process_id and routing_id pair, can the caller provide
native_view pointer instead?

[alexeypa (please no reviews), 2013/08/08 16:57:32]

Done.

+    int source_routing_id,
     base::WeakPtr<Client> weak_client_ui,
     const std::string& source_extension_id,
     const std::string& native_host_name,
     int destination_port) {
+  // Get handle of the native view and pass it to the launcher. The launcher
+  // will later pass the handle to the native messaging host process.
+  gfx::NativeWindow native_view =
+      content::RenderWidgetHost::FromID(source_process_id, source_routing_id)->
+          GetView()->GetNativeView();
   return CreateWithLauncher(weak_client_ui, source_extension_id,
                             native_host_name, destination_port,
-                            NativeProcessLauncher::CreateDefault());
+                            NativeProcessLauncher::CreateDefault(native_view));
 }
 
 // static
 scoped_ptr<NativeMessageProcessHost>
 NativeMessageProcessHost::CreateWithLauncher(
     base::WeakPtr<Client> weak_client_ui,
     const std::string& source_extension_id,
     const std::string& native_host_name,
     int destination_port,
     scoped_ptr<NativeProcessLauncher> launcher) {
@@ skipping 234 matching lines @@
     closed_ = true;
     read_stream_.reset();
     write_stream_.reset();
     content::BrowserThread::PostTask(content::BrowserThread::UI, FROM_HERE,
         base::Bind(&Client::CloseChannel, weak_client_ui_,
                    destination_port_, error_message));
   }
 }
 
 }  // namespace extensions