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

Unified Diff: remoting/host/desktop_session_agent.cc

Issue 15692018: Remove screen capturers from media/video/capture/screen. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 6 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
« no previous file with comments | « remoting/host/desktop_session_agent.h ('k') | remoting/host/desktop_session_proxy.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/desktop_session_agent.cc
diff --git a/remoting/host/desktop_session_agent.cc b/remoting/host/desktop_session_agent.cc
index 490ef90a1b10b194d8403acec03e4b9e8e2bdc51..581f2c10a7201b1ebc7519e59751bbdd57f9eb5d 100644
--- a/remoting/host/desktop_session_agent.cc
+++ b/remoting/host/desktop_session_agent.cc
@@ -320,9 +320,11 @@ void DesktopSessionAgent::OnCaptureCompleted(webrtc::DesktopFrame* frame) {
}
void DesktopSessionAgent::OnCursorShapeChanged(
- scoped_ptr<media::MouseCursorShape> cursor_shape) {
+ webrtc::MouseCursorShape* cursor_shape) {
DCHECK(video_capture_task_runner_->BelongsToCurrentThread());
+ scoped_ptr<webrtc::MouseCursorShape> owned_cursor(cursor_shape);
+
SendToNetwork(new ChromotingDesktopNetworkMsg_CursorShapeChanged(
*cursor_shape));
}
@@ -412,11 +414,11 @@ void DesktopSessionAgent::OnCaptureFrame() {
return;
}
- // media::ScreenCapturer supports a very few (currently 2) outstanding capture
- // requests. The requests are serialized on |video_capture_task_runner()| task
- // runner. If the client issues more requests, pixel data in captured frames
- // will likely be corrupted but stability of media::ScreenCapturer will not be
- // affected.
+ // webrtc::ScreenCapturer supports a very few (currently 2) outstanding
+ // capture requests. The requests are serialized on
+ // |video_capture_task_runner()| task runner. If the client issues more
+ // requests, pixel data in captured frames will likely be corrupted but
+ // stability of webrtc::ScreenCapturer will not be affected.
video_capturer_->Capture(webrtc::DesktopRegion());
}
« no previous file with comments | « remoting/host/desktop_session_agent.h ('k') | remoting/host/desktop_session_proxy.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698