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

Side by Side Diff: content/renderer/pepper/host_dispatcher_wrapper.cc

Issue 605393002: Revert of PPAPI: Never re-enter JavaScript for PostMessage. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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 "content/renderer/pepper/host_dispatcher_wrapper.h" 5 #include "content/renderer/pepper/host_dispatcher_wrapper.h"
6 6
7 #include "content/common/view_messages.h" 7 #include "content/common/view_messages.h"
8 #include "content/renderer/pepper/pepper_hung_plugin_filter.h" 8 #include "content/renderer/pepper/pepper_hung_plugin_filter.h"
9 #include "content/renderer/pepper/pepper_plugin_instance_impl.h" 9 #include "content/renderer/pepper/pepper_plugin_instance_impl.h"
10 #include "content/renderer/pepper/pepper_proxy_channel_delegate_impl.h" 10 #include "content/renderer/pepper/pepper_proxy_channel_delegate_impl.h"
(...skipping 14 matching lines...) Expand all
25 peer_pid_(peer_pid), 25 peer_pid_(peer_pid),
26 plugin_child_id_(plugin_child_id), 26 plugin_child_id_(plugin_child_id),
27 permissions_(perms), 27 permissions_(perms),
28 is_external_(is_external) {} 28 is_external_(is_external) {}
29 29
30 HostDispatcherWrapper::~HostDispatcherWrapper() {} 30 HostDispatcherWrapper::~HostDispatcherWrapper() {}
31 31
32 bool HostDispatcherWrapper::Init(const IPC::ChannelHandle& channel_handle, 32 bool HostDispatcherWrapper::Init(const IPC::ChannelHandle& channel_handle,
33 PP_GetInterface_Func local_get_interface, 33 PP_GetInterface_Func local_get_interface,
34 const ppapi::Preferences& preferences, 34 const ppapi::Preferences& preferences,
35 scoped_refptr<PepperHungPluginFilter> filter) { 35 PepperHungPluginFilter* filter) {
36 if (channel_handle.name.empty()) 36 if (channel_handle.name.empty())
37 return false; 37 return false;
38 38
39 #if defined(OS_POSIX) 39 #if defined(OS_POSIX)
40 DCHECK_NE(-1, channel_handle.socket.fd); 40 DCHECK_NE(-1, channel_handle.socket.fd);
41 if (channel_handle.socket.fd == -1) 41 if (channel_handle.socket.fd == -1)
42 return false; 42 return false;
43 #endif 43 #endif
44 44
45 dispatcher_delegate_.reset(new PepperProxyChannelDelegateImpl); 45 dispatcher_delegate_.reset(new PepperProxyChannelDelegateImpl);
46 dispatcher_.reset(new ppapi::proxy::HostDispatcher( 46 dispatcher_.reset(new ppapi::proxy::HostDispatcher(
47 module_->pp_module(), local_get_interface, permissions_)); 47 module_->pp_module(), local_get_interface, filter, permissions_));
48 // The HungPluginFilter needs to know when we are blocked on a sync message
49 // to the plugin. Note the filter outlives the dispatcher, so there is no
50 // need to remove it as an observer.
51 dispatcher_->AddSyncMessageStatusObserver(filter.get());
52 48
53 if (!dispatcher_->InitHostWithChannel(dispatcher_delegate_.get(), 49 if (!dispatcher_->InitHostWithChannel(dispatcher_delegate_.get(),
54 peer_pid_, 50 peer_pid_,
55 channel_handle, 51 channel_handle,
56 true, // Client. 52 true, // Client.
57 preferences)) { 53 preferences)) {
58 dispatcher_.reset(); 54 dispatcher_.reset();
59 dispatcher_delegate_.reset(); 55 dispatcher_delegate_.reset();
60 return false; 56 return false;
61 } 57 }
62 // HungPluginFilter needs to listen for some messages on the IO thread.
63 dispatcher_->AddIOThreadMessageFilter(filter);
64
65 dispatcher_->channel()->SetRestrictDispatchChannelGroup( 58 dispatcher_->channel()->SetRestrictDispatchChannelGroup(
66 kRendererRestrictDispatchGroup_Pepper); 59 kRendererRestrictDispatchGroup_Pepper);
67 return true; 60 return true;
68 } 61 }
69 62
70 const void* HostDispatcherWrapper::GetProxiedInterface(const char* name) { 63 const void* HostDispatcherWrapper::GetProxiedInterface(const char* name) {
71 return dispatcher_->GetProxiedInterface(name); 64 return dispatcher_->GetProxiedInterface(name);
72 } 65 }
73 66
74 void HostDispatcherWrapper::AddInstance(PP_Instance instance) { 67 void HostDispatcherWrapper::AddInstance(PP_Instance instance) {
(...skipping 29 matching lines...) Expand all
104 if (host) { 97 if (host) {
105 RenderFrame* render_frame = host->GetRenderFrameForInstance(instance); 98 RenderFrame* render_frame = host->GetRenderFrameForInstance(instance);
106 if (render_frame) { 99 if (render_frame) {
107 render_frame->Send(new ViewHostMsg_DidDeleteOutOfProcessPepperInstance( 100 render_frame->Send(new ViewHostMsg_DidDeleteOutOfProcessPepperInstance(
108 plugin_child_id_, instance, is_external_)); 101 plugin_child_id_, instance, is_external_));
109 } 102 }
110 } 103 }
111 } 104 }
112 105
113 } // namespace content 106 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/pepper/host_dispatcher_wrapper.h ('k') | content/renderer/pepper/message_channel.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698