[Mojo Video Capture] Replace const scoped_refptr<T>& with scoped_refptr<T> and use std::move

[Mojo Video Capture] Replace const scoped_refptr<T>& with scoped_refptr<T> and use std::move

Replace method signatures of touched code where const scoped_refptr<T>& is
passed into a method with passing the smart pointer by value. This is to
conform with updated Chromium style guidelines.

Note, this also required reviewing all call sites and deciding whether or not
the pointers should be copied or moved.

This CL is part of the Mojo Video Capture work. For the bigger picture,
see [1] CL1.9.2

BUG=584797
TEST=video_capture_unittests, content_unittests, capture_unittests

[1] https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxsxWslJ_X-zpOhijvI8/edit?usp=sharing

Committed: https://crrev.com/2f830966c63bc055be0f17b10190ba2f69adf0f3
Cr-Commit-Position: refs/heads/master@{#431715}

[chfremer, 2016-11-10 17:25:14]

Description was changed from

==========
Presubmit


Merge branch 'Part_4' into 1.9.1


Merge branch 'Part_4' into 1.9.1


Merge branch 'Part_4' into 1.9.1


Merge branch 'Part_4' into 1.9.1


Merge branch 'Part_4' into 1.9.1


Merge branch 'Part_4' into 1.9.1


Merge branch 'Part_4' into 1.9.1


Merge branch 'Part_4' into 1.9.1


Merge branch 'Part_4' into 1.9.1


Merge branch 'Part_4' into 1.9.1


Merge branch 'Part_4' into 1.9.1


Merge branch 'Part_4' into 1.9.1


const scoped_refptr<T>& -> scoped_refptr<T>

BUG=
==========

to

==========
[Mojo Video Capture] Use std:move instead of const scoped_refptr<T>&

Replace method signatures of touched code where const scoped_refptr<T>& is 
passed into a method with passing the smart pointer by value. This is to 
conform with updated Chromium style guidelines. 

Note, this also required reviewing all call sites and deciding whether or not 
the pointers should be copied or moved.

This CL is part of the Mojo Video Capture work. For the bigger picture, 
see [1] CL1.9.2

BUG=584797
TEST=video_capture_unittests, content_unittests, capture_unittests

[1]
https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxs...
==========

[chfremer, 2016-11-10 17:29:58]

Description was changed from

==========
[Mojo Video Capture] Use std:move instead of const scoped_refptr<T>&

Replace method signatures of touched code where const scoped_refptr<T>& is 
passed into a method with passing the smart pointer by value. This is to 
conform with updated Chromium style guidelines. 

Note, this also required reviewing all call sites and deciding whether or not 
the pointers should be copied or moved.

This CL is part of the Mojo Video Capture work. For the bigger picture, 
see [1] CL1.9.2

BUG=584797
TEST=video_capture_unittests, content_unittests, capture_unittests

[1]
https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxs...
==========

to

==========
[Mojo Video Capture] Replace const scoped_refptr<T>& with scoped_refptr<T> and
use std::move

Replace method signatures of touched code where const scoped_refptr<T>& is 
passed into a method with passing the smart pointer by value. This is to 
conform with updated Chromium style guidelines. 

Note, this also required reviewing all call sites and deciding whether or not 
the pointers should be copied or moved.

This CL is part of the Mojo Video Capture work. For the bigger picture, 
see [1] CL1.9.2

BUG=584797
TEST=video_capture_unittests, content_unittests, capture_unittests

[1]
https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxs...
==========

[chfremer, 2016-11-10 17:33:58]

chfremer@chromium.org changed reviewers:
+ mcasas@chromium.org, miu@chromium.org

[chfremer, 2016-11-10 17:33:59]

miu@: PTAL content/browser/media/* and media/capture/content/*
mcasas@: PTAL content/browser/renderer_host/* and services/*

[chfremer, 2016-11-10 17:34:19]

The CQ bit was checked by chfremer@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-10 17:34:58]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-10 17:51:22]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-10 17:51:23]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[chfremer, 2016-11-10 18:00:30]

The CQ bit was checked by chfremer@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-10 18:00:53]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-10 20:10:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-10 20:10:09]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[chfremer, 2016-11-11 16:58:06]

chfremer@chromium.org changed reviewers:
+ sergeyu@chromium.org, xhwang@chromium.org

[chfremer, 2016-11-11 16:58:07]

Since miu@ is OOO,
sergeyu@: Please RS content/browser/media/*
xhwang@: Please RS media/capture/content/*

[Sergey Ulanov, 2016-11-11 18:17:09]

lgtm

[xhwang, 2016-11-11 18:17:36]

media/capture/content rs lgtm

[mcasas, 2016-11-11 23:46:15]

lgtm

https://codereview.chromium.org/2490153003/diff/20001/media/capture/content/t...
File media/capture/content/thread_safe_capture_oracle.h (right):

https://codereview.chromium.org/2490153003/diff/20001/media/capture/content/t...
media/capture/content/thread_safe_capture_oracle.h:44: CaptureFrameCallback;
nit: maybe update to 'using ... ?

[chfremer, 2016-11-11 23:47:09]

The CQ bit was checked by chfremer@chromium.org

[commit-bot: I haz the power, 2016-11-11 23:47:50]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-12 00:51:59]

Description was changed from

==========
[Mojo Video Capture] Replace const scoped_refptr<T>& with scoped_refptr<T> and
use std::move

Replace method signatures of touched code where const scoped_refptr<T>& is 
passed into a method with passing the smart pointer by value. This is to 
conform with updated Chromium style guidelines. 

Note, this also required reviewing all call sites and deciding whether or not 
the pointers should be copied or moved.

This CL is part of the Mojo Video Capture work. For the bigger picture, 
see [1] CL1.9.2

BUG=584797
TEST=video_capture_unittests, content_unittests, capture_unittests

[1]
https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxs...
==========

to

==========
[Mojo Video Capture] Replace const scoped_refptr<T>& with scoped_refptr<T> and
use std::move

Replace method signatures of touched code where const scoped_refptr<T>& is 
passed into a method with passing the smart pointer by value. This is to 
conform with updated Chromium style guidelines. 

Note, this also required reviewing all call sites and deciding whether or not 
the pointers should be copied or moved.

This CL is part of the Mojo Video Capture work. For the bigger picture, 
see [1] CL1.9.2

BUG=584797
TEST=video_capture_unittests, content_unittests, capture_unittests

[1]
https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxs...
==========

[commit-bot: I haz the power, 2016-11-12 00:52:00]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-12 00:58:47]

Description was changed from

==========
[Mojo Video Capture] Replace const scoped_refptr<T>& with scoped_refptr<T> and
use std::move

Replace method signatures of touched code where const scoped_refptr<T>& is 
passed into a method with passing the smart pointer by value. This is to 
conform with updated Chromium style guidelines. 

Note, this also required reviewing all call sites and deciding whether or not 
the pointers should be copied or moved.

This CL is part of the Mojo Video Capture work. For the bigger picture, 
see [1] CL1.9.2

BUG=584797
TEST=video_capture_unittests, content_unittests, capture_unittests

[1]
https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxs...
==========

to

==========
[Mojo Video Capture] Replace const scoped_refptr<T>& with scoped_refptr<T> and
use std::move

Replace method signatures of touched code where const scoped_refptr<T>& is
passed into a method with passing the smart pointer by value. This is to
conform with updated Chromium style guidelines.

Note, this also required reviewing all call sites and deciding whether or not
the pointers should be copied or moved.

This CL is part of the Mojo Video Capture work. For the bigger picture,
see [1] CL1.9.2

BUG=584797
TEST=video_capture_unittests, content_unittests, capture_unittests

[1]
https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxs...

Committed: https://crrev.com/2f830966c63bc055be0f17b10190ba2f69adf0f3
Cr-Commit-Position: refs/heads/master@{#431715}
==========

[commit-bot: I haz the power, 2016-11-12 00:58:48]

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