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

Side by Side Diff: win8/viewer/metro_viewer_process_host.cc

Issue 1120203004: win: Ref-counted classes should have non-public destructors. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 7 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) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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 "win8/viewer/metro_viewer_process_host.h" 5 #include "win8/viewer/metro_viewer_process_host.h"
6 6
7 #include <shlobj.h> 7 #include <shlobj.h>
8 8
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/files/file_path.h" 10 #include "base/files/file_path.h"
(...skipping 27 matching lines...) Expand all
38 MetroViewerProcessHost::InternalMessageFilter::InternalMessageFilter( 38 MetroViewerProcessHost::InternalMessageFilter::InternalMessageFilter(
39 MetroViewerProcessHost* owner) 39 MetroViewerProcessHost* owner)
40 : owner_(owner) { 40 : owner_(owner) {
41 } 41 }
42 42
43 void MetroViewerProcessHost::InternalMessageFilter::OnChannelConnected( 43 void MetroViewerProcessHost::InternalMessageFilter::OnChannelConnected(
44 int32 peer_pid) { 44 int32 peer_pid) {
45 owner_->NotifyChannelConnected(); 45 owner_->NotifyChannelConnected();
46 } 46 }
47 47
48 MetroViewerProcessHost::InternalMessageFilter::~InternalMessageFilter() {
49 }
50
48 MetroViewerProcessHost::MetroViewerProcessHost( 51 MetroViewerProcessHost::MetroViewerProcessHost(
49 const scoped_refptr<base::SingleThreadTaskRunner>& ipc_task_runner) { 52 const scoped_refptr<base::SingleThreadTaskRunner>& ipc_task_runner) {
50 DCHECK(!instance_); 53 DCHECK(!instance_);
51 instance_ = this; 54 instance_ = this;
52 55
53 channel_ = IPC::ChannelProxy::Create(kMetroViewerIPCChannelName, 56 channel_ = IPC::ChannelProxy::Create(kMetroViewerIPCChannelName,
54 IPC::Channel::MODE_NAMED_SERVER, 57 IPC::Channel::MODE_NAMED_SERVER,
55 this, 58 this,
56 ipc_task_runner); 59 ipc_task_runner);
57 } 60 }
(...skipping 275 matching lines...) Expand 10 before | Expand all | Expand 10 after
333 const base::FilePath& folder) { 336 const base::FilePath& folder) {
334 if (success) 337 if (success)
335 select_folder_completion_callback_.Run(base::FilePath(folder), 0, NULL); 338 select_folder_completion_callback_.Run(base::FilePath(folder), 0, NULL);
336 else 339 else
337 failure_callback_.Run(NULL); 340 failure_callback_.Run(NULL);
338 select_folder_completion_callback_.Reset(); 341 select_folder_completion_callback_.Reset();
339 failure_callback_.Reset(); 342 failure_callback_.Reset();
340 } 343 }
341 344
342 } // namespace win8 345 } // namespace win8
OLDNEW
« win8/viewer/metro_viewer_process_host.h ('K') | « win8/viewer/metro_viewer_process_host.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698