summaryrefslogtreecommitdiffstats
path: root/remoting/host
Commit message (Collapse)AuthorAgeFilesLines
* Send empty video frames when screen is not changing.sergeyu@chromium.org2014-08-151-7/+11
| | | | | | | | | | | | | | | Before r272790 the host was sending empty packets 30 times a second when nothing is changing on the screen. After that change and r281677 the frequency was decreased to 5 frame per second. It looks like those changes degraded performance for some users. This change partially restores previous behavior. Empty packets are send for each captured frame even when nothing is changing on the screen. Keep-alive packets are sent every 200ms when there are no other video packets, e.g. when the video stream is paused. Review URL: https://codereview.chromium.org/477103004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@290028 0039d316-1c4b-4281-b951-d872f2087c98
* I've created a new desktop environment, Me2MeWindowDesktopEnvironment. This ↵ronakvora@google.com2014-08-158-2/+415
| | | | | | | | | | | | | environment has creates a WindowCapturerScreenWrapper instead of a ScreenCapturer with its CreateVideoCapturer() method. This wrapper wraps a WindowCapturer and delegates messages sent to it to the wrapped WindowCapturer. When the command line flag window-Id is specified the remoting_me2me_host.cc choses Me2MeWindowDesktopEnvironment instead of the regular desktop environment. Review URL: https://codereview.chromium.org/422503004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@289825 0039d316-1c4b-4281-b951-d872f2087c98
* Switch DesktopEnvironment to return a DesktopCapturer.wez@chromium.org2014-08-1238-373/+279
| | | | | | | | This removes the need for a lot of additional boilerplate in proxying capturer implementations, and makes it possible to implement DesktopEnvironment using any kind of DesktopCapturer, e.g. ScreenCapturer or WindowCapturer, for example. Review URL: https://codereview.chromium.org/455073004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@289020 0039d316-1c4b-4281-b951-d872f2087c98
* CastVideoCapturerAdapter for WebRTC support in Chromotingaiguha@chromium.org2014-08-122-0/+282
| | | | | | | | | | | | | | | | | | | | | | This class is an implementation of cricket::VideoCapturer, the core object used for video streaming in the WebRTC PeerConnection architecture. It acts as an adapter between conventional Chromoting screen capturing and the cricket::VideoCapturer interface. The CastVideoCapturerAdapter wraps a webrtc::ScreenCapturer object and captures frames at a pre-defined frame rate. Upon capture, it converts the received DesktopFrame to a cricket::CapturedFrame and fires SignalFrameCaptured() with it, at which point the base implementation takes over. This object can be used to construct one of type webrtc::VideoSourceInterface, which can be used to construct one of type webrtc::VideoTrackInterface, which can be added to a webrtc::MediaStreamInterface, which can be added to a webrtc::PeerConnection object. With that in place, we would be able to stream video to the other peer in the PeerConnection. Review URL: https://codereview.chromium.org/404443002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@288873 0039d316-1c4b-4281-b951-d872f2087c98
* Fix VideoFrameRecorderHostExtension to return full first frame.wez@chromium.org2014-08-111-1/+12
| | | | | | | | | | VideoFrameRecorder returns frames with the original updated-regions, which the video encoder will use to avoid re-encoding un-changed pixel data. To ensure that the first returned frame has full content (i.e. is a key frame), the extension overrides the updated region to have the encoder output the whole frame's content. Review URL: https://codereview.chromium.org/462503002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@288771 0039d316-1c4b-4281-b951-d872f2087c98
* Supports DevTools socket access authentication based on Android permissions.serya@chromium.org2014-08-081-3/+3
| | | | | | | | BUG=399567 Review URL: https://codereview.chromium.org/382143005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@288273 0039d316-1c4b-4281-b951-d872f2087c98
* Use webrtc::MouseCursorMonitor for cursor shapesdcaiafa@chromium.org2014-08-0830-91/+615
| | | | | | | | | | | | | | | | | | | | Use webrtc::MouseCursorMonitor for cursor shapes instead of webrtc::VideoFrameCapturer, in preparation for deprecating cursor shape functionality in the latter. Fix memory corruption in VideoSchedulerTests_StartAndStop, where a lingering capture task could trigger a expectation action declared on the stack during tear down. My changes to the test somehow made the race condition more likely. BUG=324033 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=247689 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=248045 Review URL: https://codereview.chromium.org/92473002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@288226 0039d316-1c4b-4281-b951-d872f2087c98
* Move StringToLowerASCII to base namespacebrettw@chromium.org2014-08-071-1/+1
| | | | | | | | TBR=sky Review URL: https://codereview.chromium.org/448853002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@288085 0039d316-1c4b-4281-b951-d872f2087c98
* Update webrtc&libjingle 6774:6825.ronghuawu@chromium.org2014-08-062-5/+5
| | | | | | | | | | | | | BUG=N/A R=hellner@chromium.org TBR=darin@chromium.org, hclam@chromium.org, jochen@chromium.org, palmer@chromium.org, wez@chromium.org Commit on behalf of hellner@. Original CL tried and approved in https://codereview.chromium.org/429113002/ Review URL: https://codereview.chromium.org/450463002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@287788 0039d316-1c4b-4281-b951-d872f2087c98
* Add video frame recording capability to Chromoting hosts.wez@chromium.org2014-08-028-5/+266
| | | | | | | | | | | | | | This will be used to record sequences of video frames for real-world sessions, for performance-evaluation purposes. If the host is run with a video-recording buffer size specified then it will advertise a videoRecorder capability to clients. Clients seeing this capability can send "video-recorder" extension messages to the host to start and stop recording, and to fetch the next recorded frame. Frames are encoded to VideoPackets using the "verbatim" encoder, and must then be serialized and Base-64 encoded for transmission via the extension message protocol. See crrev.com/386853002 for the client part of this. Review URL: https://codereview.chromium.org/372943002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@287183 0039d316-1c4b-4281-b951-d872f2087c98
* Some tiny cleanups.wez@chromium.org2014-08-021-17/+13
| | | | | | | | | - Fix EXPECT_EQ() ordering in Capabilities tests. - Move MATCHERs definitions in ClientSession tests. Review URL: https://codereview.chromium.org/428403003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@287173 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Update webrtc&libjingle 6774:6799."erikchen@chromium.org2014-07-312-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit bf45dff0ca703e5ad3bb6c6754ebdcb0c1b1296a. (https://codereview.chromium.org/429253003) Broke sizes on Linux http://build.chromium.org/p/chromium/builders/Linux/builds/51786 https://code.google.com/p/webrtc/source/browse/trunk/webrtc/base/linux.cc defines static std::string lsb_release_string; among other things It's guarded in WEBRTC_CHROMIUM_BUILDs. Maybe it should be WEBRTC_CHROMIUM_BUILD. BUG=NONE TBR=darin@chromium.org, hclam@chromium.org, jochen@chromium.org, palmer@chromium.org, sergeyu@chromium.org, wez@chromium.org, ronghuawu@chromium.org NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/435823002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286914 0039d316-1c4b-4281-b951-d872f2087c98
* Update webrtc&libjingle 6774:6799.ronghuawu@chromium.org2014-07-312-5/+5
| | | | | | | | | | | | | BUG=N/A R=hellner@chromium.org TBR=darin@chromium.org, hclam@chromium.org, jochen@chromium.org, palmer@chromium.org, sergeyu@chromium.org, wez@chromium.org Commit on behalf of hellner@. Original CL tried and approved in https://codereview.chromium.org/429113002/ Review URL: https://codereview.chromium.org/429253003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286902 0039d316-1c4b-4281-b951-d872f2087c98
* Add HostExtensionSessionManager to help ClientSession handle extensions.wez@chromium.org2014-07-3119-273/+907
| | | | | | | | | | | | | | | | | This CL changes the HostExtension[Session] interface: 1. Each HostExtension may optionally specify one required capability; if the client does not offer the capability then the HostExtension is not instantiated for the session. 2. Each HostExtensionSession may optionally wrap or replace the video encoder or capturer for the session. 3. HostExtension[Session]s are passed the ClientSession's ClientSessionControl and ClientStub interfaces, rather than the concrete ClientSession class itself. 4. HostExtensionSessions may force the video pipeline to be re-created, if necessary. This will be used by the VideoFrameRecorder extension to allow it to wrap the VideoEncoder when it is first created. This CL also changes the host implementation and tests: 1. ChromotingHost passes the set of configured HostExtensions to each ClientSession to work with, rather than creating & applying HostExtensionSession instances directly. 2. All logic for managing HostExtension[Session]s is moved out of ClientSession into the new HostExtensionSessionManager sub-component. 3. The FakeExtension test implementation of HostExtension is moved out of the ClientSession unittests implementation, and updated to allow testing of e.g. encoder wrapping capabilities. Review URL: https://codereview.chromium.org/402233003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286773 0039d316-1c4b-4281-b951-d872f2087c98
* Add basic perf tests for chromoting protocol.sergeyu@chromium.org2014-07-257-96/+329
| | | | | | | | | | | | | | This CL doesn't yet include any emulation of network characteristics (latency, bandwidth, etc.). I'm going to implement it in a separate CL. BUG=394067 R=rmsousa@chromium.org, ttuttle@chromium.org TBR=ttuttle@chromium.org (DEPS on net in remoting/test) Review URL: https://codereview.chromium.org/394883008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285429 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor unix domain socket.byungchul@chromium.org2014-07-241-2/+2
| | | | | | | | | | | | | | | | This is a pre-requisite for http server refactoring, https://codereview.chromium.org/296053012/. 1) Define UnixDomainClientSocket and UnixDomainServerSocket utilizing SocketLibevent. 2) Rename UnixDomainSocket to UnixDomainListenSocket to reduce confusion. 3) unittests BUG=371906 Review URL: https://codereview.chromium.org/376323002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285148 0039d316-1c4b-4281-b951-d872f2087c98
* Fix member pointer callbacks in an ATL sink maprnk@chromium.org2014-07-241-6/+10
| | | | | | | | | | | | | | | | | Clang won't let you form a pointer to member without an explicit '&ClassName::' before the method name. In fact, neither will MSVC unless you explicitly disable the error, which the ATL does. This has been documented on MSDN since VS 2005: http://msdn.microsoft.com/en-us/library/ms174271.aspx Clang doesn't implement this extension, so just fix the code. R=wez@chromium.org BUG=82385 Review URL: https://codereview.chromium.org/410853002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285118 0039d316-1c4b-4281-b951-d872f2087c98
* Switch to BoringSSL.davidben@chromium.org2014-07-221-0/+3
| | | | | | | | | | | | | | | | | This is a reland of r284079 which was reverted in r284248 for components build issues. That, in turn, was a reland of r283813 which was reverted in r283845 because it broke WebRTC tests on Android. That, in turn, was a reland of r283542 which was reverted in r283591 because it broke the WebView build. This is a much larger change than its diff suggests. If it breaks something, please revert first and ask questions later. BUG=393317 R=agl@chromium.org, jam@chromium.org Review URL: https://codereview.chromium.org/401153002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@284729 0039d316-1c4b-4281-b951-d872f2087c98
* Cast Port Allocator Factory for PeerConnectionaiguha@chromium.org2014-07-222-0/+104
| | | | | | | | | | This CL contains the implementation of PortAllocatorFactoryInterface in peerconnectioninterface.h. It bridges the gap in port allocation objects between WebRTC and Chromium. The factory produces a ChromiumPortAllocator that can successfully be used by the PeerConnection API. Review URL: https://codereview.chromium.org/398813003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@284589 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Switch to BoringSSL."agl@chromium.org2014-07-181-3/+0
| | | | | | | | | | This reverts commit r284079. BUG=395271 Review URL: https://codereview.chromium.org/406693004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@284248 0039d316-1c4b-4281-b951-d872f2087c98
* Switch to BoringSSL.agl@chromium.org2014-07-181-0/+3
| | | | | | | | | | | | | | | (This is a reland of r283813 which was reverted in r283845 because it broke WebRTC tests on Android. That, in turn, was a reland of of r283542 which was reverted in r283591 because it broke the WebView build.) This is a much larger change than its diff suggests. If it breaks something, please revert first and ask questions later. BUG=none Review URL: https://codereview.chromium.org/401613008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@284079 0039d316-1c4b-4281-b951-d872f2087c98
* Enable x64 Chromoting host and installerdcaiafa@chromium.org2014-07-181-17/+25
| | | | | | | | | | | | | | N.B. I had to move the sawbuck_provider in the .wxs out of the installation directory. Even though it only contains registry changes, candle will complain if it is not inside a directory, but the application directory is platform specific, whereas sawbuck_provider is always 32-bit. BUG=394467 Review URL: https://codereview.chromium.org/398173002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@284007 0039d316-1c4b-4281-b951-d872f2087c98
* Rename *Fake to Fake*sergeyu@chromium.org2014-07-1710-75/+44
| | | | | | | | | | | | Normally fake interface implementations use Fake prefix, but there were some with Fake suffix, namely SignalStrategyFake and HostStatusMonitorFake. Renamed them to use Fake prefix. R=rmsousa@chromium.org Review URL: https://codereview.chromium.org/393703002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283925 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 283813 "Switch to BoringSSL."vitalybuka@chromium.org2014-07-171-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | Failed WebRtcBrowserTest on android_dbg_triggered_tests. > Switch to BoringSSL. > > (This is a reland of r283542 which was reverted in r283591 because it > broke the WebView build. The android_aosp trybots are broken[1] so this > based on hope.) > > This is a much larger change than its diff suggests. If it breaks > something, please revert first and ask questions later. > > [1] http://code.google.com/p/chromium/issues/detail?id=394597 > > BUG=none > > Review URL: https://codereview.chromium.org/399993002 TBR=agl@chromium.org Review URL: https://codereview.chromium.org/405503002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283845 0039d316-1c4b-4281-b951-d872f2087c98
* Switch to BoringSSL.agl@chromium.org2014-07-171-0/+3
| | | | | | | | | | | | | | | | | (This is a reland of r283542 which was reverted in r283591 because it broke the WebView build. The android_aosp trybots are broken[1] so this based on hope.) This is a much larger change than its diff suggests. If it breaks something, please revert first and ask questions later. [1] http://code.google.com/p/chromium/issues/detail?id=394597 BUG=none Review URL: https://codereview.chromium.org/399993002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283813 0039d316-1c4b-4281-b951-d872f2087c98
* Remove remoting/jingle_gluesergeyu@chromium.org2014-07-1725-47/+48
| | | | | | | | | | | | | Moved all files from remoting/jingle_glue to remoting/signaling and remoting/protocol. The new remoting/signaling directory contains signaling layer implementation for remoting that can be used independently of remoting/protocol. TBR=rch@chromium.org Review URL: https://codereview.chromium.org/390983003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283654 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup list of source files in remoting targetssergeyu@chromium.org2014-07-173-10/+6
| | | | | | | | | | Some files were compiled for both remoting_host and remoting_core. Also moved some files to remoting_host so that they don't need to be listed multiple times for different platforms Review URL: https://codereview.chromium.org/393073002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283597 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Switch to BoringSSL."agl@chromium.org2014-07-171-3/+0
| | | | | | | | This reverts commit 283542. This broke the WebView Android build. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283591 0039d316-1c4b-4281-b951-d872f2087c98
* Switch to BoringSSL.agl@chromium.org2014-07-161-0/+3
| | | | | | | | | | (This is a much larger change than its diff suggests. If it breaks something, please revert first and ask questions later.) R=davidben@chromium.org, eroman@chromium.org, rsleevi@chromium.org Review URL: https://codereview.chromium.org/345063006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283542 0039d316-1c4b-4281-b951-d872f2087c98
* Add a unit test that filenames aren't unintentionally converted to URLs.dcheng@chromium.org2014-07-151-1/+1
| | | | | | | | | | | | | | | Also fixes two issues in OSExchangeDataProviderWin: - It used a disjoint set of clipboard formats when handling GetUrl(..., true /* filename conversion */) vs GetFilenames(...), so the actual returned results would vary depending on which one was called. - It incorrectly used ::DragFinish() instead of ::ReleaseStgMedium(). ::DragFinish() is only meant to be used in conjunction with WM_DROPFILES. BUG=346135 Review URL: https://codereview.chromium.org/380553002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283226 0039d316-1c4b-4281-b951-d872f2087c98
* clang/win: static_cast cannot cast void* to function pointer types.thakis@chromium.org2014-07-152-2/+2
| | | | | | | | | | | | (Even reinterpret_cast'ing this isn't well-defined apparently, but it should compile and work in practice, at least on win/x86.) BUG=82385 NOTRY=true Review URL: https://codereview.chromium.org/392813002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283188 0039d316-1c4b-4281-b951-d872f2087c98
* Remove ClientSession::EventHandler::OnSessionSequenceNumber()sergeyu@chromium.org2014-07-125-16/+0
| | | | | | | | The callback was not used Review URL: https://codereview.chromium.org/387123003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@282782 0039d316-1c4b-4281-b951-d872f2087c98
* Pull out common code from client and host versions of LogToServerlambroslambrou@chromium.org2014-07-109-269/+200
| | | | | | | | | | Also added unittests for client code. TEST=unittests Review URL: https://codereview.chromium.org/320403002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@282193 0039d316-1c4b-4281-b951-d872f2087c98
* Fix typo in comments added in r281677sergeyu@chromium.org2014-07-081-1/+1
| | | | | | | | | TBR=wez@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/373213002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@281881 0039d316-1c4b-4281-b951-d872f2087c98
* Decrease keep-alive period for the video channel to 200ms.sergeyu@chromium.org2014-07-081-2/+4
| | | | | | | | | | | Keep-alive above 250ms causes PseudoTCP to reset congestion window between messages, making connection much less responsive. BUG=391991 Review URL: https://codereview.chromium.org/376763002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@281677 0039d316-1c4b-4281-b951-d872f2087c98
* Switch to using URLRequestContextBuilder to create some contexts.mmenke@chromium.org2014-07-088-5/+18
| | | | | | | | | | | | | Also add missing functionality to URLRequestContextBuilder to fit their needs. This is intended to simplify code to initialize the network stack, and make it easier to refactor network stack initialization. BUG=388836 Review URL: https://codereview.chromium.org/354693004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@281638 0039d316-1c4b-4281-b951-d872f2087c98
* Destroy ScreenCapturer and VideoEncoder on their respective threads.wez@chromium.org2014-07-082-25/+86
| | | | | | | | | | This allows ScreenCapturer and VideoEncoder implementations to assume that instances will always be deleted on the same thread from which they are used (though they may still be created on a different thread). Review URL: https://codereview.chromium.org/362063004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@281632 0039d316-1c4b-4281-b951-d872f2087c98
* Add missing Pass() calls for passing scoped_ptrs on Windows.thakis@chromium.org2014-07-043-4/+7
| | | | | | | | | | | BUG=82385 TBR=awong@chromium.org R=joaodasilva@chromium.org,sergeyu@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/367403002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@281366 0039d316-1c4b-4281-b951-d872f2087c98
* Move ServiceUrls to remoting/base.sergeyu@chromium.org2014-07-015-139/+3
| | | | | | | | | | ServiceUrls is also usable on the client side, particularly to get bot jid for logs. Moved to remoting/base where it can be reused by the client. Review URL: https://codereview.chromium.org/357503003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280712 0039d316-1c4b-4281-b951-d872f2087c98
* Remove palmer from some OWNERS files for IPC security review.palmer@chromium.org2014-06-271-1/+0
| | | | | | Review URL: https://codereview.chromium.org/361513002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280382 0039d316-1c4b-4281-b951-d872f2087c98
* Add VideoFrameRecorder for use recording test frame sequences.wez@chromium.org2014-06-273-0/+589
| | | | | | | | | | | | This will be used to record representative frame sequences for performance evaluation of the new VP9 encoder. BUG=260879 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=279795 Review URL: https://codereview.chromium.org/339073002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280255 0039d316-1c4b-4281-b951-d872f2087c98
* Fix remoting_me2me_host to ignore relay policy when nat traversal is disabled.sergeyu@chromium.org2014-06-261-8/+7
| | | | | | | | | | BUG=389223 R=dcaiafa@chromium.org TBR=lambroslambrou@chromium.org Review URL: https://codereview.chromium.org/358013002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280078 0039d316-1c4b-4281-b951-d872f2087c98
* Revert of Add VideoFrameRecorder for use recording test frame sequences. ↵avi@chromium.org2014-06-253-586/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (https://codereview.chromium.org/339073002/) Reason for revert: Brand new test fails in ASAN: Direct leak of 112 byte(s) in 1 object(s) allocated from: #0 0x462ccb in operator new(unsigned long) /usr/local/google/work/chromium/src/third_party/llvm/projects/compiler-rt/lib/asan/asan_new_delete.cc:55 #1 0x7ee8f9 in CreateNextFrame remoting/host/video_frame_recorder_unittest.cc:96 #2 0x7ee8f9 in remoting::VideoFrameRecorderTest::CreateTestFrames() remoting/host/video_frame_recorder_unittest.cc:112 #3 0x7f1184 in remoting::VideoFrameRecorderTest_EncodeButDontRecord_Test::TestBody() remoting/host/video_frame_recorder_unittest.cc:269 #4 0xb3bf38 in HandleExceptionsInMethodIfSupported<testing::Test, void> testing/gtest/src/gtest.cc:2045 #5 0xb3bf38 in testing::Test::Run() testing/gtest/src/gtest.cc:2061 #6 0xb3e1c9 in testing::TestInfo::Run() testing/gtest/src/gtest.cc:2237 #7 0xb3ef56 in testing::TestCase::Run() testing/gtest/src/gtest.cc:2344 #8 0xb5206a in testing::internal::UnitTestImpl::RunAllTests() testing/gtest/src/gtest.cc:4065 #9 0xb516a0 in HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool> testing/gtest/src/gtest.cc:2045 #10 0xb516a0 in testing::UnitTest::Run() testing/gtest/src/gtest.cc:3697 #11 0xab66bc in RUN_ALL_TESTS testing/gtest/include/gtest/gtest.h:2231 #12 0xab66bc in base::TestSuite::Run() base/test/test_suite.cc:227 #13 0xaada32 in Run base/callback.h:401 #14 0xaada32 in base::(anonymous namespace)::LaunchUnitTestsInternal(base::Callback<int ()> const&, int, bool, base::Callback<void ()> const&) base/test/launcher/unit_test_launcher.cc:498 #15 0xaad3ad in base::LaunchUnitTests(int, char**, base::Callback<int ()> const&) base/test/launcher/unit_test_launcher.cc:553 #16 0x4e8067 in main chrome/test/base/run_all_remoting_unittests.cc:31 #17 0x7f1f660df76c in __libc_start_main /build/buildd/eglibc-2.15/csu/libc-start.c:226 Original issue's description: > Add VideoFrameRecorder for use recording test frame sequences. > > This will be used to record representative frame sequences for performance evaluation of the new VP9 encoder. > > BUG=260879 > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=279795 TBR=sergeyu@chromium.org,wez@chromium.org NOTREECHECKS=true NOTRY=true BUG=260879 Review URL: https://codereview.chromium.org/350903003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279805 0039d316-1c4b-4281-b951-d872f2087c98
* Add VideoFrameRecorder for use recording test frame sequences.wez@chromium.org2014-06-253-0/+586
| | | | | | | | | | This will be used to record representative frame sequences for performance evaluation of the new VP9 encoder. BUG=260879 Review URL: https://codereview.chromium.org/339073002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279795 0039d316-1c4b-4281-b951-d872f2087c98
* Fixing #define.weitaosu@chromium.org2014-06-241-3/+3
| | | | | | | | | | NOTRY=true BUG= Review URL: https://codereview.chromium.org/346843009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279498 0039d316-1c4b-4281-b951-d872f2087c98
* Wait for CRD daemon process to be killed before returning.lambroslambrou@chromium.org2014-06-191-19/+25
| | | | | | | | | | | | | | | | | | | | This should mean that "/etc/init.d/chrome-remote-desktop restart" will stop the service and won't try to start it again until the daemon is killed. A timeout is used to make sure that system shutdown is not paused indefinitely if the daemon is unkillable. Note that, since the daemon is not a child process, os.waitpid() cannot be used for this. The psutil Python module is used instead. Some other parts of the script have been updated as well to use psutil methods. This is slightly safer, as psutil has some PID-reuse detection built in. BUG=334437 Review URL: https://codereview.chromium.org/335303002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278267 0039d316-1c4b-4281-b951-d872f2087c98
* Remove NPAPI plugin from chromoting webapp.sergeyu@chromium.org2014-06-1813-2375/+3
| | | | | | | | | BUG=134215 R=weitaosu@chromium.org, wez@chromium.org Review URL: https://codereview.chromium.org/342583002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278198 0039d316-1c4b-4281-b951-d872f2087c98
* Update list of apps in chromoting NM manifestssergeyu@chromium.org2014-06-182-4/+4
| | | | | | | | | | | Added new V2 QA app. Removed old canary that we no longer use. BUG=385726 R=jamiewalch@chromium.org Review URL: https://codereview.chromium.org/344643003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278185 0039d316-1c4b-4281-b951-d872f2087c98
* Increase size limit of Me2Me Linux virtual desktoplambroslambrou@chromium.org2014-06-181-1/+1
| | | | | | | | | | | People are using bigger and bigger monitors these days! This change only makes a difference when RANDR-enabled Xvfb is used. NOTRY=true Review URL: https://codereview.chromium.org/341653004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277967 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup: Remove more dead GTK code.thestig@chromium.org2014-06-171-8/+4
| | | | | | | | TBR=sky@chromium.org Review URL: https://codereview.chromium.org/326043002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277881 0039d316-1c4b-4281-b951-d872f2087c98