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

Unified Diff: content/browser/renderer_host/media/web_contents_video_capture_device.cc

Issue 14081010: Cleanup: Remove unnecessary ".get()" from scoped_ptrs<>. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix some gtk issues Created 7 years, 8 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/media/web_contents_video_capture_device.cc
diff --git a/content/browser/renderer_host/media/web_contents_video_capture_device.cc b/content/browser/renderer_host/media/web_contents_video_capture_device.cc
index 8530cf44ef686780f94b5e6a16b6e034457b9151..a5095cf126e9515558899ed29771dd791de6f885 100644
--- a/content/browser/renderer_host/media/web_contents_video_capture_device.cc
+++ b/content/browser/renderer_host/media/web_contents_video_capture_device.cc
@@ -1119,17 +1119,17 @@ void WebContentsVideoCaptureDevice::Impl::AssignCaptureMachine(
scoped_ptr<CaptureMachine> capture_machine) {
DCHECK(!impl || impl->thread_checker_.CalledOnValidThread());
- if (!impl.get()) {
+ if (!impl) {
// If WCVD::Impl was destroyed before we got back on it's thread and
// capture_machine is not NULL, then we need to return to the UI thread to
// safely cleanup the CaptureMachine.
- if (capture_machine.get()) {
+ if (capture_machine) {
BrowserThread::PostTask(
BrowserThread::UI, FROM_HERE, base::Bind(
&DeleteCaptureMachineOnUIThread, base::Passed(&capture_machine)));
return;
}
- } else if (!capture_machine.get()) {
+ } else if (!capture_machine) {
impl->Error();
} else {
impl->capture_machine_ = capture_machine.Pass();
@@ -1167,7 +1167,7 @@ WebContentsVideoCaptureDevice::Impl::~Impl() {
// There is still a capture pipeline running that is checking in with the
// oracle, and processing captures that are already started in flight. That
// pipeline must be shut down asynchronously, on the UI thread.
- if (capture_machine_.get()) {
+ if (capture_machine_) {
// The task that is posted to the UI thread might not run if we are shutting
// down, so we transfer ownership of CaptureMachine to the closure so that
// it is still cleaned up when the closure is deleted.

Powered by Google App Engine
This is Rietveld 408576698