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

Side by Side Diff: content/browser/renderer_host/media/video_capture_host.cc

Issue 2686763002: [Mojo Video Capture] Split OnIncomingCapturedVideoFrame() to OnNewBuffer() and OnFrameReadyInBuffer( (Closed)
Patch Set: Remove unused bool accidentally introduced in PS2 Created 3 years, 10 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 "content/browser/renderer_host/media/video_capture_host.h" 5 #include "content/browser/renderer_host/media/video_capture_host.h"
6 6
7 #include <memory> 7 #include <memory>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/bind_helpers.h" 10 #include "base/bind_helpers.h"
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
79 if (controllers_.find(controller_id) == controllers_.end()) 79 if (controllers_.find(controller_id) == controllers_.end())
80 return; 80 return;
81 81
82 if (base::ContainsKey(device_id_to_observer_map_, controller_id)) 82 if (base::ContainsKey(device_id_to_observer_map_, controller_id))
83 device_id_to_observer_map_[controller_id]->OnBufferDestroyed(buffer_id); 83 device_id_to_observer_map_[controller_id]->OnBufferDestroyed(buffer_id);
84 } 84 }
85 85
86 void VideoCaptureHost::OnBufferReady( 86 void VideoCaptureHost::OnBufferReady(
87 VideoCaptureControllerID controller_id, 87 VideoCaptureControllerID controller_id,
88 int buffer_id, 88 int buffer_id,
89 const scoped_refptr<media::VideoFrame>& video_frame) { 89 const media::mojom::VideoFrameInfoPtr& frame_info) {
90 DCHECK_CURRENTLY_ON(BrowserThread::IO); 90 DCHECK_CURRENTLY_ON(BrowserThread::IO);
91 if (controllers_.find(controller_id) == controllers_.end()) 91 if (controllers_.find(controller_id) == controllers_.end())
92 return; 92 return;
93 93
94 if (!base::ContainsKey(device_id_to_observer_map_, controller_id)) 94 if (!base::ContainsKey(device_id_to_observer_map_, controller_id))
95 return; 95 return;
96 96
97 media::mojom::VideoFrameInfoPtr info = media::mojom::VideoFrameInfo::New();
98 info->timestamp = video_frame->timestamp();
99 info->metadata = video_frame->metadata()->CopyInternalValues();
100
101 DCHECK(media::PIXEL_FORMAT_I420 == video_frame->format() ||
102 media::PIXEL_FORMAT_Y16 == video_frame->format());
103 info->pixel_format = video_frame->format();
104 info->storage_type = media::PIXEL_STORAGE_CPU;
105 info->coded_size = video_frame->coded_size();
106 info->visible_rect = video_frame->visible_rect();
107
108 device_id_to_observer_map_[controller_id]->OnBufferReady(buffer_id, 97 device_id_to_observer_map_[controller_id]->OnBufferReady(buffer_id,
109 std::move(info)); 98 frame_info.Clone());
110 } 99 }
111 100
112 void VideoCaptureHost::OnEnded(VideoCaptureControllerID controller_id) { 101 void VideoCaptureHost::OnEnded(VideoCaptureControllerID controller_id) {
113 DVLOG(1) << __func__; 102 DVLOG(1) << __func__;
114 DCHECK_CURRENTLY_ON(BrowserThread::IO); 103 DCHECK_CURRENTLY_ON(BrowserThread::IO);
115 BrowserThread::PostTask( 104 BrowserThread::PostTask(
116 BrowserThread::IO, FROM_HERE, 105 BrowserThread::IO, FROM_HERE,
117 base::Bind(&VideoCaptureHost::DoEnded, weak_factory_.GetWeakPtr(), 106 base::Bind(&VideoCaptureHost::DoEnded, weak_factory_.GetWeakPtr(),
118 controller_id)); 107 controller_id));
119 } 108 }
(...skipping 206 matching lines...) Expand 10 before | Expand all | Expand 10 after
326 const base::WeakPtr<VideoCaptureController> controller = it->second; 315 const base::WeakPtr<VideoCaptureController> controller = it->second;
327 controllers_.erase(it); 316 controllers_.erase(it);
328 if (!controller) 317 if (!controller)
329 return; 318 return;
330 319
331 media_stream_manager_->video_capture_manager()->StopCaptureForClient( 320 media_stream_manager_->video_capture_manager()->StopCaptureForClient(
332 controller.get(), controller_id, this, on_error); 321 controller.get(), controller_id, this, on_error);
333 } 322 }
334 323
335 } // namespace content 324 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698