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

Side by Side Diff: remoting/host/desktop_session_win.cc

Issue 367403002: Add missing Pass() calls for passing scoped_ptrs on Windows. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: :-( Created 6 years, 5 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 "remoting/host/desktop_session_win.h" 5 #include "remoting/host/desktop_session_win.h"
6 6
7 #include <limits> 7 #include <limits>
8 #include <sddl.h> 8 #include <sddl.h>
9 9
10 #include "base/base_switches.h" 10 #include "base/base_switches.h"
(...skipping 534 matching lines...) Expand 10 before | Expand all | Expand 10 after
545 target.Pass(), 545 target.Pass(),
546 launch_elevated, 546 launch_elevated,
547 base::WideToUTF8( 547 base::WideToUTF8(
548 kDaemonIpcSecurityDescriptor))); 548 kDaemonIpcSecurityDescriptor)));
549 if (!delegate->Initialize(session_id)) { 549 if (!delegate->Initialize(session_id)) {
550 TerminateSession(); 550 TerminateSession();
551 return; 551 return;
552 } 552 }
553 553
554 // Create a launcher for the desktop process, using the per-session delegate. 554 // Create a launcher for the desktop process, using the per-session delegate.
555 launcher_.reset(new WorkerProcessLauncher(delegate.Pass(), this)); 555 launcher_.reset(new WorkerProcessLauncher(
556 delegate.PassAs<WorkerProcessLauncher::Delegate>(), this));
556 } 557 }
557 558
558 void DesktopSessionWin::OnSessionDetached() { 559 void DesktopSessionWin::OnSessionDetached() {
559 DCHECK(caller_task_runner_->BelongsToCurrentThread()); 560 DCHECK(caller_task_runner_->BelongsToCurrentThread());
560 561
561 launcher_.reset(); 562 launcher_.reset();
562 563
563 if (monitoring_notifications_) { 564 if (monitoring_notifications_) {
564 ReportElapsedTime("detached"); 565 ReportElapsedTime("detached");
565 566
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
602 exploded.hour, 603 exploded.hour,
603 exploded.minute, 604 exploded.minute,
604 exploded.second, 605 exploded.second,
605 exploded.millisecond, 606 exploded.millisecond,
606 passed.c_str()); 607 passed.c_str());
607 608
608 last_timestamp_ = now; 609 last_timestamp_ = now;
609 } 610 }
610 611
611 } // namespace remoting 612 } // namespace remoting
OLDNEW
« no previous file with comments | « remoting/host/daemon_process_win.cc ('k') | remoting/host/win/worker_process_launcher_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698