summaryrefslogtreecommitdiffstats
path: root/remoting
Commit message (Collapse)AuthorAgeFilesLines
* Post StopOnEncodeThread to the correct thread.jamiewalch@chromium.org2012-12-161-1/+1
| | | | | | | | | BUG=165303 Review URL: https://chromiumcodereview.appspot.com/11569042 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@173347 0039d316-1c4b-4281-b951-d872f2087c98
* Fix chromoting host to shutdown cleanly when config file is missing.sergeyu@chromium.org2012-12-151-1/+1
| | | | | | | | | BUG=165716 Review URL: https://chromiumcodereview.appspot.com/11575046 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@173339 0039d316-1c4b-4281-b951-d872f2087c98
* Chromoting Host: Initialize clipboard_atom_ before using it.lambroslambrou@chromium.org2012-12-151-5/+5
| | | | | | | | | | | | We weren't seeing XFixes CLIPBOARD selection changes, because we requested notification before initializing |clipboard_atom_| ! BUG=166203 Review URL: https://chromiumcodereview.appspot.com/11575047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@173338 0039d316-1c4b-4281-b951-d872f2087c98
* [sync] Componentize sync: Part 4: Fix includes in third_party/protobufrsimha@chromium.org2012-12-141-2/+1
| | | | | | | | | | | | Various chrome components depend on third_party/protobuf:protobuf_lite, and in order to move to component builds, third_party/protobuf:protobuf_lite must support component builds. As of today, there are a few incorrect includes and a few missing exports in third_party that block this. This patch corrects instances of incorrect includes of headers from third_party/protobuf.gyp:protobuf_full_do_not_use, and replaces them with correct includes from third_party/protobuf.gyp:protobuf_lite BUG=136928 Review URL: https://codereview.chromium.org/11316370 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@173228 0039d316-1c4b-4281-b951-d872f2087c98
* Increase default maximum desktop size and make it configurable.jamiewalch@google.com2012-12-141-6/+17
| | | | | | | | BUG=166099 Review URL: https://codereview.chromium.org/11566037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@173214 0039d316-1c4b-4281-b951-d872f2087c98
* Enable translations for unbranded chromoting builds.sergeyu@chromium.org2012-12-141-51/+44
| | | | | | | | | | Previously when building chromoting webapp translations were included only in official builds. Review URL: https://chromiumcodereview.appspot.com/11578013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@173109 0039d316-1c4b-4281-b951-d872f2087c98
* Fix crash on disconnect.jamiewalch@chromium.org2012-12-142-9/+14
| | | | | | | | | BUG=163684 Review URL: https://chromiumcodereview.appspot.com/11576023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@173096 0039d316-1c4b-4281-b951-d872f2087c98
* Add trapKey() API to ClientPlugin wrapper.wez@chromium.org2012-12-142-1/+24
| | | | | | | | | | BUG=144758 TEST=None; this will be manually testable via a follow-up CL that makes use of the API. Review URL: https://chromiumcodereview.appspot.com/11416378 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@173046 0039d316-1c4b-4281-b951-d872f2087c98
* Don't run host for managed accounts on OS X.skonig@chromium.org2012-12-131-0/+6
| | | | | | | | | | | | | | | | Previously if you set up the host under a normal user account, then logged out and logged back in under another account that was managed (i.e. had parental controls enabled), we would attempt to launch the host, triggering an OS notification that it can't be run. We didn't trap this as a permanent error so we'd try to re-launch again, triggering another notification, then another, etc. This change looks for the specific error code returned when a process can't be run by a managed account, then treats it as a premanent error. BUG=144685 TEST=Manual Review URL: https://chromiumcodereview.appspot.com/11553037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172793 0039d316-1c4b-4281-b951-d872f2087c98
* Add work-around for Mac OS X CapsLock key event behaviour.wez@chromium.org2012-12-131-1/+18
| | | | | | | | | BUG=144758 Review URL: https://chromiumcodereview.appspot.com/11558030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172792 0039d316-1c4b-4281-b951-d872f2087c98
* Fix translation IDs in Chromoting strings.sergeyu@chromium.org2012-12-1342-672/+672
| | | | | | | | | | | | | | Chromoting translations were not generated correctly when transitioning to using Grit for Chromoting resources in r166127. Branded strings were translated as if they are not branded. Upading string IDs in chromoting translation files so that resources generated correctly for both Chromoting and Chrome Remote Desktop builds. BUG=165845 Review URL: https://codereview.chromium.org/11555039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172784 0039d316-1c4b-4281-b951-d872f2087c98
* Close the client session in case if the disconnect window could not be shown ↵alexeypa@chromium.org2012-12-137-148/+229
| | | | | | | | | | | for any reason. BUG=156659 Review URL: https://chromiumcodereview.appspot.com/11531007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172713 0039d316-1c4b-4281-b951-d872f2087c98
* Ensure Chromoting crash-reporting is opt-inlambroslambrou@chromium.org2012-12-122-10/+7
| | | | | | | | | | | Update all default or error code-paths to set the flag to false. BUG=165373 Review URL: https://chromiumcodereview.appspot.com/11547008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172689 0039d316-1c4b-4281-b951-d872f2087c98
* Clearing a local reference to the I/O thread's task runner to makre sure the ↵alexeypa@chromium.org2012-12-122-52/+40
| | | | | | | | daemon can be stopped. This CL also switches the daemon to use base::RunLoop instead of using the message loop directly. Review URL: https://chromiumcodereview.appspot.com/11512008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172492 0039d316-1c4b-4281-b951-d872f2087c98
* Fix AudioPipeReader to calculate stream possition correclty.sergeyu@chromium.org2012-12-122-20/+35
| | | | | | | | | | | | | | Previously AudioPipeReader wasn't always calculating stream position correctly. In cases when PulseAudio doesn't fill the pipe buffer for some reason (e.g. it didn't get scheduled) the host would skip that data and will end up reading less bytes per second than necessary. BUG=164308 Review URL: https://chromiumcodereview.appspot.com/11446054 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172464 0039d316-1c4b-4281-b951-d872f2087c98
* Remove Linux host setup wizard and libwebkitgtk dependency.sergeyu@chromium.org2012-12-113-337/+0
| | | | | | Review URL: https://codereview.chromium.org/11530020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172460 0039d316-1c4b-4281-b951-d872f2087c98
* Fix me2me_virtual_host.py to handle blocked user accounts properly.sergeyu@chromium.org2012-12-111-2/+6
| | | | | | | | | Previously the host daemon wasn't handling correctly error codes 103 and 105. This was a caused by merge conflict between crrev.com/10830225 and crrev.com/10823083. Review URL: https://codereview.chromium.org/11535010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172430 0039d316-1c4b-4281-b951-d872f2087c98
* Wait for encode thread during VideoScheduler teardown.wez@chromium.org2012-12-112-3/+18
| | | | | | | | | | | | | VideoFrameCapturer currently owns the buffers underlying CaptureData instances, so VideoScheduler::Stop() mustn't notify completion until it is done accessing them. BUG=163641 Review URL: https://chromiumcodereview.appspot.com/11474043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172280 0039d316-1c4b-4281-b951-d872f2087c98
* Build Mac Breakpad symbols for Chromoting Hostlambroslambrou@chromium.org2012-12-112-1/+60
| | | | | | | | | | BUG=136579 TEST=Official build logs Review URL: https://chromiumcodereview.appspot.com/11340058 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172270 0039d316-1c4b-4281-b951-d872f2087c98
* Launch the host under LocalService account at low integrity level with all ↵alexeypa@chromium.org2012-12-1113-94/+731
| | | | | | | | | | | privileges removed. It receives it's own window station and desktop only accessible by SYSTEM and logon SID the host process is running under. The security descriptor assigned to th eprocess gives access to SYSTEM and limited (query and terminate) access to built-in administrators group. BUG=134694 Review URL: https://chromiumcodereview.appspot.com/11428172 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172268 0039d316-1c4b-4281-b951-d872f2087c98
* Reorder tab title to put the important information first.jamiewalch@google.com2012-12-111-2/+2
| | | | | | | | | | I've also changed the format to use a dash instead of a colon for consistency with Google properties like Drive and Gmail. BUG=164319 Review URL: https://codereview.chromium.org/11507014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172239 0039d316-1c4b-4281-b951-d872f2087c98
* Enable crash-reporting for Mac host, for users who opt in.lambroslambrou@chromium.org2012-12-105-11/+44
| | | | | | | | | | | | | This enables the Web UI for opting in, and sets an option in the Host config accordingly. The host, during startup, reads the config option and enables crash-reporting if set. BUG=136579 Review URL: https://chromiumcodereview.appspot.com/11447003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172183 0039d316-1c4b-4281-b951-d872f2087c98
* Fix AudioScheduler and VideoScheduler creation.wez@chromium.org2012-12-086-46/+95
| | | | | | | | | | | | | The schedulers post startup tasks to a secondary thread, running the risk that the secondary thread will them down before the caller has had a chance to take a reference to them. BUG=164795 Review URL: https://chromiumcodereview.appspot.com/11474015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171925 0039d316-1c4b-4281-b951-d872f2087c98
* Remove WebSocket server and HostStatusService from remoting host.sergeyu@chromium.org2012-12-089-1442/+0
| | | | | | | | | We are not planning to use WebSockets for host configuration. Removing websocket-related code from remoting host. Review URL: https://codereview.chromium.org/11475037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171915 0039d316-1c4b-4281-b951-d872f2087c98
* Don't relaunch the host process too often.jamiewalch@google.com2012-12-081-8/+45
| | | | | | | | BUG=162329 Review URL: https://codereview.chromium.org/11468027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171914 0039d316-1c4b-4281-b951-d872f2087c98
* Set screen DPI to 96 on screen resize.lambroslambrou@chromium.org2012-12-071-1/+22
| | | | | | | | | BUG=160275 Review URL: https://chromiumcodereview.appspot.com/11467012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171832 0039d316-1c4b-4281-b951-d872f2087c98
* remoting: Remove bogus entry from host DEPS list.tfarina@chromium.org2012-12-071-8/+1
| | | | | | | | | | TEST=checkdeps.py succeed. R=sergeyu@chromium.org Review URL: https://chromiumcodereview.appspot.com/11448010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171821 0039d316-1c4b-4281-b951-d872f2087c98
* Fix ClientSession to not access ConnectionToClient stubs after Disconnect().wez@chromium.org2012-12-074-30/+44
| | | | | | | | | | | | | | | | | | This CL Stop()s the audio and video scheduler components, and invalidates the client clipboard stub WeakPtr factory to ensure that the corresponding stubs will not continue to be accessed between Disconnect() and the ClientSession being Stop()ed. The ChromotingHost unit-tests are also fixed to correctly notify ClientSession::OnConnectedClosed() when the ClientSession disconnects the mock ConnectionToClient, to properly exercise the ChromotingHost's client tear-down. BUG=163157 Review URL: https://chromiumcodereview.appspot.com/11419292 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171743 0039d316-1c4b-4281-b951-d872f2087c98
* Make sure that DisconnectWindow class does not depend on ChromotingHost so ↵alexeypa@chromium.org2012-12-077-63/+56
| | | | | | | | | | | it could be used in the desktop process. BUG=134694 Review URL: https://chromiumcodereview.appspot.com/11471008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171742 0039d316-1c4b-4281-b951-d872f2087c98
* Use raw input to detect local mouse events.alexeypa@chromium.org2012-12-076-193/+257
| | | | | | | | | | | | | | | This CL switches the local input monitor to use raw input instead of hooking low-level mouse input events (WH_MOUSE_LL). The OS can silently remove WH_MOUSE_LL hooks if it is running longer than the timeout specified in [HKCU\Control Panel\Desktop]\LowLevelHooksTimeout. This can result in the local user not being able to take control of the mouse. Other advantades are: - raw input is considered more efficient than low-level hooks; - raw input is received on the UI thread meaning that the host creates one less thread. BUG=148690 Review URL: https://chromiumcodereview.appspot.com/11419216 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171732 0039d316-1c4b-4281-b951-d872f2087c98
* Remove unused APIs from the AutoThreadTaskRunner interface.wez@chromium.org2012-12-074-122/+31
| | | | | | | | | | | This CL also re-instates CHECKs for the success of PostTask() calls to the underlying task-runner, to catch cases where the underlying message loop is being stopped prematurely. BUG=145856 Review URL: https://chromiumcodereview.appspot.com/11308254 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171713 0039d316-1c4b-4281-b951-d872f2087c98
* Disable Chromoting host compilation on ChromeOSsergeyu@chromium.org2012-12-072-576/+610
| | | | | | | | | | | Chromoting host is not supported on ChromeOS and many targets in remoting.gyp fail to compile for ChromeOS. Disable these targets. BUG=164591 Review URL: https://codereview.chromium.org/11471009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171680 0039d316-1c4b-4281-b951-d872f2087c98
* Add HostState enum to track host process status.sergeyu@chromium.org2012-12-071-120/+135
| | | | | | | | | | Previously state of the host was tracked via shutting_down_ and restarting_ flags which made state transitions hard to understand in some case. Review URL: https://codereview.chromium.org/11416093 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171677 0039d316-1c4b-4281-b951-d872f2087c98
* Added support of Secure Attention Sequence in multiprocess mode.alexeypa@chromium.org2012-12-079-19/+117
| | | | | | | | | BUG=134694 Review URL: https://chromiumcodereview.appspot.com/11447021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171664 0039d316-1c4b-4281-b951-d872f2087c98
* Dead/duplicate code cleanup.jamiewalch@google.com2012-12-063-79/+42
| | | | | | | | | | | | | * Removed unused access token parameters. * Removed email address parameters. * Removed unnecessary NULL checks for tokens. * Removed duplicate enum. BUG=None Review URL: https://codereview.chromium.org/11446049 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171629 0039d316-1c4b-4281-b951-d872f2087c98
* disable unittest XServerClipboardTest.CopyPastelambroslambrou@chromium.org2012-12-061-1/+2
| | | | | | | | | BUG=163428 Review URL: https://chromiumcodereview.appspot.com/11414275 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171433 0039d316-1c4b-4281-b951-d872f2087c98
* Reset retry flag on successful connection.jamiewalch@google.com2012-12-061-0/+1
| | | | | | | | BUG=163321 Review URL: https://codereview.chromium.org/11445035 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171374 0039d316-1c4b-4281-b951-d872f2087c98
* Use AutoThread in ChromotingHostContext & NPAPI plugin.wez@chromium.org2012-12-0511-214/+148
| | | | | | | | | | | | | | | | | | Callers now create ChromotingHostContext to create a set of threads for host tasks to run on, and pass the threads' TaskRunners to each host component explicitly. The ChromotingHostContext can then be torn down as soon as the caller no longer needs to create new components using the threads, and the threads themselves will exit only when the created components no longer require them. This is a re-re-land of 11094056; the re-land failed due to a rebasing typo, and some over-zealous thread checking in ChromotingHostContext. BUG=145856 TEST=Chromoting IT2Me & Me2Me hosts start correctly, can be connected to, and shutdown cleanly. Unit-tests pass. Review URL: https://chromiumcodereview.appspot.com/11412305 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171358 0039d316-1c4b-4281-b951-d872f2087c98
* Clear the usagestats flag created by the host versions that mishandled the flag.alexeypa@chromium.org2012-12-051-0/+20
| | | | | | | | BUG=164296 Review URL: https://codereview.chromium.org/11439015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171318 0039d316-1c4b-4281-b951-d872f2087c98
* Serialize shutdown of the service and main threads to make sure that all ↵alexeypa@chromium.org2012-12-052-55/+70
| | | | | | | | | | | local objects are destroyed before base::AtExitManager kicks in. BUG=163629 Review URL: https://chromiumcodereview.appspot.com/11434109 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171138 0039d316-1c4b-4281-b951-d872f2087c98
* Fixed consent checkbox test.jamiewalch@google.com2012-12-051-2/+2
| | | | | | | | BUG=164296 Review URL: https://codereview.chromium.org/11446003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171104 0039d316-1c4b-4281-b951-d872f2087c98
* (1) Added a recursive boolean param to FilePathWatcher::Watch() function to ↵kmadhusu@chromium.org2012-12-052-3/+3
| | | | | | | | | | | | | | | watch for sub directory tree changes. Fixed all the calling sites. (2) Added support to watch sub trees on Windows. (3) Added FilePathWatcherTest.RecursiveWatch browser test. BUG=144491 TEST=none Review URL: https://chromiumcodereview.appspot.com/11415066 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171097 0039d316-1c4b-4281-b951-d872f2087c98
* Add rmsousa@ and remove simonmorris@ from remoting/OWNERS.wez@chromium.org2012-12-031-1/+1
| | | | | | Review URL: https://chromiumcodereview.appspot.com/11299256 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170760 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 170360jamiewalch@google.com2012-11-3011-137/+191
| | | | | | | | | | | | | | | | | | | | | | > Use AutoThread in ChromotingHostContext & NPAPI plugin. > > Callers now create ChromotingHostContext to create a set of threads > for host tasks to run on, and pass the threads' TaskRunners to each > host component explicitly. The ChromotingHostContext can then be > torn down as soon as the caller no longer needs to create new > components using the threads, and the threads themselves will exit > only when the created components no longer require them. > > This is a re-land of 11094056, which failed on the valgrind & Windows TSan bots due to ChromotingHostContext.StartAndStop not running its message loop to allow the context's threads join tasks to execute. > > TBR=alexeypa > > Review URL: https://chromiumcodereview.appspot.com/11316247 TBR=wez@chromium.org Review URL: https://codereview.chromium.org/11419265 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170591 0039d316-1c4b-4281-b951-d872f2087c98
* Avoid posting tasks from a constructor of a ref-counted object because doing ↵alexeypa@chromium.org2012-11-301-4/+5
| | | | | | | | | | | so might cause premature deletion of the object. BUG=163559 Review URL: https://chromiumcodereview.appspot.com/11316267 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170468 0039d316-1c4b-4281-b951-d872f2087c98
* Correctly detect Windows XP Pro x64 and Windows 2003 as not supporting ↵alexeypa@chromium.org2012-11-302-3/+3
| | | | | | | | | | | | elevation. TBR=wez@chromium.org BUG=163542 Review URL: https://codereview.chromium.org/11411279 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170416 0039d316-1c4b-4281-b951-d872f2087c98
* Improve curtain mode error message.jamiewalch@chromium.org2012-11-302-8/+19
| | | | | | | | | BUG=161158 Review URL: https://chromiumcodereview.appspot.com/11316231 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170397 0039d316-1c4b-4281-b951-d872f2087c98
* Don't log 'connection failed' for session errors that occur after connection ↵jamiewalch@chromium.org2012-11-303-5/+13
| | | | | | | | | | | succeeds. BUG=112150 NOTRY=true Review URL: https://chromiumcodereview.appspot.com/11412229 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170395 0039d316-1c4b-4281-b951-d872f2087c98
* Added support of keyboard, mouse and clipboard events to DesktopSessionAgent.alexeypa@chromium.org2012-11-3015-5/+441
| | | | | | | | | BUG=134694 Review URL: https://chromiumcodereview.appspot.com/11417094 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170393 0039d316-1c4b-4281-b951-d872f2087c98
* Mac Breakpad integration for Chromoting Host (disabled for now)lambroslambrou@chromium.org2012-11-305-5/+123
| | | | | | | | | | | | | | | This adds Breakpad initialization, and the necessary Breakpad components to the Me2Me host bundle. Tested by tweaking the code to enable Breakpad, and to upload to a test server instead of the production URL. BUG=136579 Review URL: https://chromiumcodereview.appspot.com/11308267 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170369 0039d316-1c4b-4281-b951-d872f2087c98