Cleanup usage of scoped_ptr<> in remoting for C++11

Cleanup usage of scoped_ptr<> in remoting for C++11

 - Use nullptr to initialize null scoped_ptr().
 - use Pass(), PassAs() is no longer required.

Committed: https://crrev.com/afce97826070950bca497ce147a30312ce780e74
Cr-Commit-Position: refs/heads/master@{#297236}

[Sergey Ulanov, 2014-09-27 01:41:14]

sergeyu@chromium.org changed reviewers:
+ rmsousa@chromium.org

[Sergey Ulanov, 2014-09-27 01:42:37]

I'll send another CL for make_scoped_ptr().

[rmsousa, 2014-09-27 01:49:20]

lgtm with a comment (more of a C++ question)

https://codereview.chromium.org/609923004/diff/1/remoting/host/client_session.cc
File remoting/host/client_session.cc (right):

https://codereview.chromium.org/609923004/diff/1/remoting/host/client_session...
remoting/host/client_session.cc:501: return make_scoped_ptr(new
remoting::VideoEncoderVerbatim());
Isn't this returning a different scoped_ptr type
(scoped_ptr<VideoEncoderVerbatim> instead of scoped_ptr<VideoEncoder>)?

Or does the C++ template system automatically cast the parameter to
VideoEncoder* to match the return type?

[Sergey Ulanov, 2014-09-29 17:33:24]

https://codereview.chromium.org/609923004/diff/1/remoting/host/client_session.cc
File remoting/host/client_session.cc (right):

https://codereview.chromium.org/609923004/diff/1/remoting/host/client_session...
remoting/host/client_session.cc:501: return make_scoped_ptr(new
remoting::VideoEncoderVerbatim());
On 2014/09/27 01:49:20, rmsousa wrote:
> Isn't this returning a different scoped_ptr type
> (scoped_ptr<VideoEncoderVerbatim> instead of scoped_ptr<VideoEncoder>)?
> 
> Or does the C++ template system automatically cast the parameter to
> VideoEncoder* to match the return type?

Yes, upcast is automatic now (that's why PassAs() is no longer necessary)

[Sergey Ulanov, 2014-09-29 18:35:27]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2014-09-29 18:36:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/609923004/1

[commit-bot: I haz the power, 2014-09-29 19:38:39]

Committed patchset #1 (id:1) as 6b71a8aade80cb38603d7b049c5039b3a15731ab

[commit-bot: I haz the power, 2014-09-29 19:39:17]

Patchset 1 (id:??) landed as
https://crrev.com/afce97826070950bca497ce147a30312ce780e74
Cr-Commit-Position: refs/heads/master@{#297236}