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

Unified Diff: win8/viewer/metro_viewer_process_host.cc

Issue 740653005: Remove implicit conversions from scoped_refptr to T* in win8/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address comments Created 6 years, 1 month 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
« no previous file with comments | « win8/test/ui_automation_client.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: win8/viewer/metro_viewer_process_host.cc
diff --git a/win8/viewer/metro_viewer_process_host.cc b/win8/viewer/metro_viewer_process_host.cc
index 276aa2c0b2f9265c6f6fea07f6f5a34d36ca9267..9f35ec352d3b88fb2aa4fcd9e01eb69d4a8618f7 100644
--- a/win8/viewer/metro_viewer_process_host.cc
+++ b/win8/viewer/metro_viewer_process_host.cc
@@ -65,7 +65,7 @@ MetroViewerProcessHost::~MetroViewerProcessHost() {
base::ProcessId viewer_process_id = GetViewerProcessId();
channel_->Close();
- if (message_filter_) {
+ if (message_filter_.get()) {
// Wait for the viewer process to go away.
if (viewer_process_id != base::kNullProcessId) {
base::ProcessHandle viewer_process = NULL;
@@ -78,7 +78,7 @@ MetroViewerProcessHost::~MetroViewerProcessHost() {
::CloseHandle(viewer_process);
}
}
- channel_->RemoveFilter(message_filter_);
+ channel_->RemoveFilter(message_filter_.get());
}
instance_ = NULL;
}
@@ -96,7 +96,7 @@ bool MetroViewerProcessHost::LaunchViewerAndWaitForConnection(
channel_connected_event_.reset(new base::WaitableEvent(false, false));
message_filter_ = new InternalMessageFilter(this);
- channel_->AddFilter(message_filter_);
+ channel_->AddFilter(message_filter_.get());
if (base::win::GetVersion() >= base::win::VERSION_WIN8) {
base::win::ScopedComPtr<IApplicationActivationManager> activator;
« no previous file with comments | « win8/test/ui_automation_client.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698