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

Side by Side Diff: content/browser/renderer_host/pepper/browser_ppapi_host_impl.cc

Issue 15984016: Call scoped_refptr<T>::get() rather than relying on implicit "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased Created 7 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 | Annotate | Revision Log
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/browser/renderer_host/pepper/browser_ppapi_host_impl.h" 5 #include "content/browser/renderer_host/pepper/browser_ppapi_host_impl.h"
6 6
7 #include "content/browser/renderer_host/pepper/pepper_message_filter.h" 7 #include "content/browser/renderer_host/pepper/pepper_message_filter.h"
8 #include "content/browser/tracing/trace_message_filter.h" 8 #include "content/browser/tracing/trace_message_filter.h"
9 #include "content/common/pepper_renderer_instance_data.h" 9 #include "content/common/pepper_renderer_instance_data.h"
10 #include "content/public/browser/render_view_host.h" 10 #include "content/public/browser/render_view_host.h"
(...skipping 19 matching lines...) Expand all
30 new BrowserPpapiHostImpl(sender, permissions, plugin_name, 30 new BrowserPpapiHostImpl(sender, permissions, plugin_name,
31 profile_directory, 31 profile_directory,
32 true); 32 true);
33 browser_ppapi_host->set_plugin_process_handle(plugin_child_process); 33 browser_ppapi_host->set_plugin_process_handle(plugin_child_process);
34 34
35 channel->AddFilter( 35 channel->AddFilter(
36 new PepperMessageFilter(permissions, 36 new PepperMessageFilter(permissions,
37 host_resolver, 37 host_resolver,
38 render_process_id, 38 render_process_id,
39 render_view_id)); 39 render_view_id));
40 channel->AddFilter(browser_ppapi_host->message_filter()); 40 channel->AddFilter(browser_ppapi_host->message_filter().get());
41 channel->AddFilter(new TraceMessageFilter()); 41 channel->AddFilter(new TraceMessageFilter());
42 42
43 return browser_ppapi_host; 43 return browser_ppapi_host;
44 } 44 }
45 45
46 BrowserPpapiHostImpl::BrowserPpapiHostImpl( 46 BrowserPpapiHostImpl::BrowserPpapiHostImpl(
47 IPC::Sender* sender, 47 IPC::Sender* sender,
48 const ppapi::PpapiPermissions& permissions, 48 const ppapi::PpapiPermissions& permissions,
49 const std::string& plugin_name, 49 const std::string& plugin_name,
50 const base::FilePath& profile_data_directory, 50 const base::FilePath& profile_data_directory,
(...skipping 102 matching lines...) Expand 10 before | Expand all | Expand 10 after
153 */ 153 */
154 return ppapi_host_->OnMessageReceived(msg); 154 return ppapi_host_->OnMessageReceived(msg);
155 } 155 }
156 156
157 void BrowserPpapiHostImpl::HostMessageFilter::OnHostDestroyed() { 157 void BrowserPpapiHostImpl::HostMessageFilter::OnHostDestroyed() {
158 DCHECK(ppapi_host_); 158 DCHECK(ppapi_host_);
159 ppapi_host_ = NULL; 159 ppapi_host_ = NULL;
160 } 160 }
161 161
162 } // namespace content 162 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698