summaryrefslogtreecommitdiffstats
path: root/remoting
Commit message (Collapse)AuthorAgeFilesLines
* [Chromoting] Add a HostStatusObserver to the ChromotingHost unit tests.simonmorris@chromium.org2012-06-213-9/+78
| | | | | | | | | This CL depends on codereview.chromium.org/10583022. Review URL: https://chromiumcodereview.appspot.com/10592004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143337 0039d316-1c4b-4281-b951-d872f2087c98
* Removing incorrect usage of TO_L_STRING() macros.alexeypa@chromium.org2012-06-2010-58/+51
| | | | | | | | BUG=133003 Review URL: https://chromiumcodereview.appspot.com/10579034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143194 0039d316-1c4b-4281-b951-d872f2087c98
* Fix up-scaling case to early-exit if no updates are within clip area.wez@chromium.org2012-06-201-0/+6
| | | | | | | | | | | The RectangleUpdateDecoder expects Decoders to report an empty update region if no updates lay within the clip area, to provide rendering flow-control. The VP8 Decoder was always reporting the entire frame as having been rendered when up-scaling, resulting in continuous re-rendering of the frame. BUG=116835,133609 TEST=Run Chromoting client with page-zoom >100% to a host with little or no changes happening on-screen and verify that the client's CPU usage is not >50%. Review URL: https://chromiumcodereview.appspot.com/10576017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143132 0039d316-1c4b-4281-b951-d872f2087c98
* [Chromoting] Make the sequence expectations in the ChromotingHost unit testssimonmorris@chromium.org2012-06-201-102/+81
| | | | | | | | | | | more accurate. This is a follow-up to r143005. Review URL: https://chromiumcodereview.appspot.com/10583022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143098 0039d316-1c4b-4281-b951-d872f2087c98
* Fix crash on it2me disconnect on the mac.dcaiafa@chromium.org2012-06-201-3/+11
| | | | | | | | | | | | | | | | | | | In it2me, the destructor of EventExecutor does not get called in the same thread from where it's used. This is causing the Timer object used in EventExecutorMac to CHECK in the destructor because it requires that it be destroyed in the same thread where it was first started - even if it has already been stopped. So instead of reusing the Timer and starting and stopping it on demand, EventExecutorMac will now create and destroy the timer on session start and stop, respectively. BUG=133017 TEST=Connect and disconnect with it2me. Repeat with me2me. Review URL: https://chromiumcodereview.appspot.com/10557012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143089 0039d316-1c4b-4281-b951-d872f2087c98
* Pull Mac Chromoting Host constants into a header filelambroslambrou@chromium.org2012-06-196-43/+63
| | | | | | | | | | | | | | | Many definitions were shared between the Mac pref-pane and the DaemonController (and also the uninstaller app). This CL consolidates them into one place. BUG=none TEST=compiles Review URL: https://chromiumcodereview.appspot.com/10557026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143056 0039d316-1c4b-4281-b951-d872f2087c98
* [Chromoting] Set expectations for MockEventExecutor::OnSessionStarted.simonmorris@chromium.org2012-06-193-2/+29
| | | | | | | | | | | This compensates for a regression in r139854. A follow-up CL will set sequence expectations more accurately. Review URL: https://chromiumcodereview.appspot.com/10566033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143005 0039d316-1c4b-4281-b951-d872f2087c98
* Allow the Chromoting client plugin to support up-scaling, albeit slowly.wez@chromium.org2012-06-183-10/+48
| | | | | | | | | BUG=116138 Review URL: https://chromiumcodereview.appspot.com/9568048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142853 0039d316-1c4b-4281-b951-d872f2087c98
* [Chromoting] Fix 10.5 bots by not running unittests on 10.5.garykac@chromium.org2012-06-181-0/+13
| | | | | | | Chromoting doesn't support 10.5 or earlier. Review URL: https://chromiumcodereview.appspot.com/10580007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142838 0039d316-1c4b-4281-b951-d872f2087c98
* Ignore errors when installing the firewall exception for Chromoting Host.alexeypa@chromium.org2012-06-181-1/+1
| | | | | | | | | | | | WiX Firewall Extension fails if the Windows Firewall service is not started, see http://sourceforge.net/tracker/index.php?func=detail&aid=2077095&group_id=105970&atid=642714 BUG=132887 Review URL: https://chromiumcodereview.appspot.com/10566012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142828 0039d316-1c4b-4281-b951-d872f2087c98
* [Chromoting] Add cursor shape support for Windowsgarykac@chromium.org2012-06-181-4/+195
| | | | | | | | BUG=116299 TEST=none Review URL: https://chromiumcodereview.appspot.com/10562031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142823 0039d316-1c4b-4281-b951-d872f2087c98
* [Chromoting] Add Mac host support for sending cursor shape to client.garykac@chromium.org2012-06-182-1/+110
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10539035 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142818 0039d316-1c4b-4281-b951-d872f2087c98
* C++ readability review fixes for r141239: Make Chromoting Host report ↵alexeypa@chromium.org2012-06-1614-166/+177
| | | | | | | | | | | crashes to Breakpad (Windows only). BUG=130678 TEST=remoting_unittests.BreakpadWinDeathTest Review URL: https://chromiumcodereview.appspot.com/10535082 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142581 0039d316-1c4b-4281-b951-d872f2087c98
* Replace GetIOMessageLoopProxy() with GetNetworkTaskRunner() in ↵sergeyu@chromium.org2012-06-163-16/+16
| | | | | | | | | | | | UrlRequestContextGetter. Also updated all code that depends on that methods. TBR=mnissler@chromium.org Review URL: https://chromiumcodereview.appspot.com/10539148 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142560 0039d316-1c4b-4281-b951-d872f2087c98
* Detect if user closes the pref-pane without disabling Chromotinglambroslambrou@chromium.org2012-06-162-5/+3
| | | | | | | | | | BUG=129616 TEST=manual Review URL: https://chromiumcodereview.appspot.com/10566013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142544 0039d316-1c4b-4281-b951-d872f2087c98
* [Chromoting] Re-enable ChromotingHost unit tests.simonmorris@chromium.org2012-06-161-48/+77
| | | | | | | | | | | vp8-related valgrind errors are all of the type described in crbug.com/115251. BUG=86546 Review URL: https://chromiumcodereview.appspot.com/10560017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142542 0039d316-1c4b-4281-b951-d872f2087c98
* Cleaned up the reader and writer create methods to return scoped_ptr.kxing@chromium.org2012-06-158-16/+20
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10556020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142498 0039d316-1c4b-4281-b951-d872f2087c98
* Added files for audio readers.kxing@chromium.org2012-06-153-0/+137
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10562007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142497 0039d316-1c4b-4281-b951-d872f2087c98
* [Chromoting] Make ChromotingHost's dependency on libjingle injected, instead ↵simonmorris@chromium.org2012-06-1511-36/+134
| | | | | | | | | | | | | of hard-coded. This will help improve ChromotingHost's unit tests. BUG=86546 Review URL: https://chromiumcodereview.appspot.com/10538091 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142473 0039d316-1c4b-4281-b951-d872f2087c98
* Convert use of TimeDeltas in remoting/host/it2me_host_user_interface.tedvessenes@gmail.com2012-06-151-2/+2
| | | | | | | | | | R=ajwong@chromium.org BUG=108171 Review URL: https://chromiumcodereview.appspot.com/10543172 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142434 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 142056 - Adding "Help us to improve..." message to be presented next ↵jamiewalch@google.com2012-06-151-4/+0
| | | | | | | | | | | | | | to the checkbox enabling crash dump collection. BUG=130678 Review URL: https://chromiumcodereview.appspot.com/10537159 TBR=alexeypa@chromium.org Review URL: https://chromiumcodereview.appspot.com/10544169 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142305 0039d316-1c4b-4281-b951-d872f2087c98
* Added files for audio writers.kxing@chromium.org2012-06-155-0/+151
| | | | | | | | | | The structure of AudioWriter and ProtobufAudioWriter are similar to their video counterparts. Review URL: https://chromiumcodereview.appspot.com/10535153 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142297 0039d316-1c4b-4281-b951-d872f2087c98
* [Chromoting] Use productsign to sign installer package.garykac@chromium.org2012-06-141-1/+4
| | | | | | | | | | | Old code used codesign which doesn't sign the package the correct way. BUG=127267 TEST=none Review URL: https://chromiumcodereview.appspot.com/10539167 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142262 0039d316-1c4b-4281-b951-d872f2087c98
* Build pref-pane as universal binary on Mac OS Xlambroslambrou@chromium.org2012-06-142-43/+185
| | | | | | | | | | BUG=125116 TEST=manual Review URL: https://chromiumcodereview.appspot.com/10383204 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142214 0039d316-1c4b-4281-b951-d872f2087c98
* Roll Breakpad to @970.alexeypa@chromium.org2012-06-141-2/+3
| | | | | | | | | | | | Add two NULL parameters to CrashGenerationServer ctor to match the new prototype (changed in r952) Add the files appeared in r945 to breakpad.gyp BUG=127776,132164 TEST=remoting_unittests.BreakpadWinDeathTest.* Review URL: https://chromiumcodereview.appspot.com/10544121 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142200 0039d316-1c4b-4281-b951-d872f2087c98
* Shutdown It2Me host UI only after the host is stopped.sergeyu@chromium.org2012-06-141-4/+7
| | | | | | | | | | | | | | | | | Currently user interface object is destroyed before host is stopped. The destructor for UserInterface object doesn't unregister that object from list of host status observers, and it cannot do that because the destructor is executed on the UI thread. In the same time it's no longer necessary to destroy UI before the host is stopped. Changed plugin destruction sequence so that UI is destroyed only after the host is stopped. BUG=130596 Review URL: https://chromiumcodereview.appspot.com/10533138 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142071 0039d316-1c4b-4281-b951-d872f2087c98
* Adding "Help us to improve..." message to be presented next to the checkbox ↵alexeypa@chromium.org2012-06-141-0/+4
| | | | | | | | | | | enabling crash dump collection. BUG=130678 Review URL: https://chromiumcodereview.appspot.com/10537159 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142056 0039d316-1c4b-4281-b951-d872f2087c98
* [Chromoting] Using IDispatch::Invoke() to call Omaha inetrfaces.alexeypa@chromium.org2012-06-134-44/+772
| | | | | | | | | | Omaha uses non-standard scheme for versioning its COM interfaces. Instead of letting the callers to know a fixed UUID and prototype of an interface, Omaha relies on the callers to use late binding via IDispatch::Invoke. This way it can extend an interface after it has been released by adding new members and changing its UUID. The limitation is that vtable can no longer be used. BUG=131498 Review URL: https://chromiumcodereview.appspot.com/10532099 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141960 0039d316-1c4b-4281-b951-d872f2087c98
* Add an explicit function to init NSS for SSL server socketsrsleevi@chromium.org2012-06-138-1/+40
| | | | | | | | | | BUG=131622 TEST=tsan goes green for existing tests Review URL: https://chromiumcodereview.appspot.com/10543106 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141955 0039d316-1c4b-4281-b951-d872f2087c98
* Fix SSLSocketAdapter to handle asynchronous writes appropriately.sergeyu@chromium.org2012-06-132-63/+142
| | | | | | | | | | | | Also fixed some other bugs, particularly asynchronous reads were not handled properly in some cases. BUG=129658 Review URL: https://chromiumcodereview.appspot.com/10543069 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141946 0039d316-1c4b-4281-b951-d872f2087c98
* Remvoe usb_keycode_map.h include from remoting.gypi to fix the treejam@chromium.org2012-06-131-1/+0
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141940 0039d316-1c4b-4281-b951-d872f2087c98
* Don't use MessageLoop or MessageLoopProxy in EventExecutor.sergeyu@chromium.org2012-06-136-75/+80
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10539056 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141936 0039d316-1c4b-4281-b951-d872f2087c98
* Move usb_keycode_map.h into ui/base/keycodes.garykac@chromium.org2012-06-135-395/+4
| | | | | | | | | | | | | | | | | | This file contains the mapping table to convert to/from USB scan codes and native Mac, Win and Linux key codes. Currently, 2 exact copies of this file exist: * webkit/plugins/ppapi * remoting/host This cl combines these files into one super-powerful USB key mapping table! BUG=118432 TEST=none Review URL: https://chromiumcodereview.appspot.com/10544106 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141930 0039d316-1c4b-4281-b951-d872f2087c98
* Update libjingle to 153.ronghuawu@chromium.org2012-06-132-3/+3
| | | | | | | | Remove the overrides for thread and messagequeue. Review URL: https://chromiumcodereview.appspot.com/10534122 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141908 0039d316-1c4b-4281-b951-d872f2087c98
* Don't replace $1 with an empty string when localizing string for host plugin.sergeyu@chromium.org2012-06-131-1/+10
| | | | | | | | | | | | | | Previously chrome.l18n.getString() wouldn't touch $1 within the resource when substitutions are not specified. This was fixed in crrev.com/141280, and now it replaced $1 with an empty strings, which broke chromoting host localization that relied on the previous incorrect behaviour. Fixed the webapp to correctly handle strings substitutions. BUG=132146 Review URL: https://chromiumcodereview.appspot.com/10533101 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141804 0039d316-1c4b-4281-b951-d872f2087c98
* Fix bump-scroll on Windows.jamiewalch@google.com2012-06-121-1/+4
| | | | | | | | | BUG=132104 TEST=Manual Review URL: https://chromiumcodereview.appspot.com/10537113 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141765 0039d316-1c4b-4281-b951-d872f2087c98
* Remove duplicate CPU detection code; use base::CPU instead.dalecurtis@chromium.org2012-06-122-9/+7
| | | | | | | | | | BUG=none TEST=media_unittests. Review URL: https://chromiumcodereview.appspot.com/10537082 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141627 0039d316-1c4b-4281-b951-d872f2087c98
* MessageLoopProxy cleanups in remoting client.sergeyu@chromium.org2012-06-1213-129/+115
| | | | | | | | | | | Replaced all usage of MessageLoopProxy with SingleThreadTaskRunner. Also includes some other minor cleanups, e.g. removed ClientContext::network_message_loop(). Review URL: https://chromiumcodereview.appspot.com/10454018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141589 0039d316-1c4b-4281-b951-d872f2087c98
* Prompt user before restarting System Preferences.lambroslambrou@chromium.org2012-06-122-44/+143
| | | | | | | | | | | | | | | | | | | | | | When Chromoting Host components have been upgraded on Mac whilst the pref-pane is still loaded, prompt the user before restarting System Preferences. The user has the opportunity to cancel, in case they were doing something in a different pref-pane. In that case, the Chromoting pref-pane should be completely disabled, and shouldn't process any new config data until it is restarted. This is a followup CL to http://codereview.chromium.org/10413065/ BUG=129226 TEST=manual Review URL: https://chromiumcodereview.appspot.com/10446091 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141583 0039d316-1c4b-4281-b951-d872f2087c98
* [Chromoting] Sign the prefpane and installer package.garykac@chromium.org2012-06-122-6/+18
| | | | | | | | | | | Now that the Chromoting installer is built using Packages, we can build a "flat" installer and have it signed. A signed installer is required for OS X 10.8 (Mt Lion). This cl also signs the Chromoting prefpane. Review URL: https://chromiumcodereview.appspot.com/10538092 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141581 0039d316-1c4b-4281-b951-d872f2087c98
* Make NetLog take in callbacks that return Values rathermmenke@chromium.org2012-06-102-13/+4
| | | | | | | | | | | | than refcounted objects. Avoids the need to create classes and copy data. Also no longer get time whenever an event is logged. BUG=126243 Review URL: https://chromiumcodereview.appspot.com/10399083 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141377 0039d316-1c4b-4281-b951-d872f2087c98
* Make Chromoting Host report crashes to Breakpad (Windows only). The user ↵alexeypa@chromium.org2012-06-0816-11/+578
| | | | | | | | | | | | must enable crash dumps collection by setting the "usagestats" value in the ClientState or ClientStateMedium key. BUG=130678 TEST=remoting_unittests.BreakpadWinDeathTest Review URL: https://chromiumcodereview.appspot.com/10495003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141239 0039d316-1c4b-4281-b951-d872f2087c98
* Don't deregister the host if it wasn't stopped.jamiewalch@google.com2012-06-081-1/+2
| | | | | | | | | BUG=131270 TEST=Manual Review URL: https://chromiumcodereview.appspot.com/10535070 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141152 0039d316-1c4b-4281-b951-d872f2087c98
* Fix jscompiler error.jamiewalch@google.com2012-06-081-1/+3
| | | | | | | | | BUG=None TEST=Manuel Review URL: https://chromiumcodereview.appspot.com/10546066 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141150 0039d316-1c4b-4281-b951-d872f2087c98
* Create UserInterface object on the UI thread.sergeyu@chromium.org2012-06-082-4/+6
| | | | | | | | | | Now UserInterface objects weak pointer to itself that is created in the constructor. That weak pointer is used on the UI thread, so it needs to be created on UI thread too. Review URL: https://chromiumcodereview.appspot.com/10543068 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141136 0039d316-1c4b-4281-b951-d872f2087c98
* fixed issue 128383 - replace GetPeerAddress(AddressList* address) with ↵zhaoqin@chromium.org2012-06-084-12/+8
| | | | | | | | | | | | | GetPeerAddress(IPEndPoint* address) R=szym@chromium.org BUG=128383 TEST=try bot Review URL: https://chromiumcodereview.appspot.com/10491007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141125 0039d316-1c4b-4281-b951-d872f2087c98
* Replace ScopedThreadProxy with MessageLoopProxy & WeakPtrs.wez@chromium.org2012-06-0711-250/+92
| | | | | | | | | | | | | | | | | | | This affects the following classes: * ChromotingClient * ChromotingInstance * HostUserInterface * It2MeHostUserInterface The MessageLoopProxy/WeakPtr combination requires that the WeakPtr is created on the thread referred to by the proxy; code in which that is hard to arrange usually has subtle race-conditions. This is a re-land of CL 1045404, replacing some CR_DEFINE_STATIC_LOCAL() instances with base::LazyInstance to avoid adding global initializers or finalizers. TEST=Existing unit-tests, and manual testing. Review URL: https://chromiumcodereview.appspot.com/10440107 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141028 0039d316-1c4b-4281-b951-d872f2087c98
* Created an AudioStub for Chromoting audio.kxing@chromium.org2012-06-072-0/+35
| | | | | | | | | | BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/10533014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141009 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 140846 - (relanding r140697)rsleevi@chromium.org2012-06-063-26/+72
| | | | | | | | | | | | | | | | | | | | | | | | | | | Move the core state machine of SSLClientSocketNSS into a thread-safe Core NSS SSL functions may block on the underlying PKCS#11 modules or on user input. On ChromeOS, which has a hardware TPM, calls may take upwards of several seconds, preventing any IPC due to the I/O thread being blocked. To avoid blocking the I/O thread on ChromeOS, move the core SSL implementation to a dedicated worker thread, so that only SSL sockets are blocked. BUG=122355 TEST=existing net_unittests + see bug. Review URL: https://chromiumcodereview.appspot.com/10454066 TBR=rsleevi@chromium.org Review URL: https://chromiumcodereview.appspot.com/10546033 TBR=rsleevi@chromium.org Review URL: https://chromiumcodereview.appspot.com/10543036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@140856 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 140697 - Maybe caused sizes regression (which would be acceptablersleevi@chromium.org2012-06-063-72/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | and this will be relanded), but revert to be sure it wasn't something else. Move the core state machine of SSLClientSocketNSS into a thread-safe Core NSS SSL functions may block on the underlying PKCS#11 modules or on user input. On ChromeOS, which has a hardware TPM, calls may take upwards of several seconds, preventing any IPC due to the I/O thread being blocked. To avoid blocking the I/O thread on ChromeOS, move the core SSL implementation to a dedicated worker thread, so that only SSL sockets are blocked. BUG=122355 TEST=existing net_unittests + see bug. Review URL: https://chromiumcodereview.appspot.com/10454066 TBR=rsleevi@chromium.org Review URL: https://chromiumcodereview.appspot.com/10546033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@140846 0039d316-1c4b-4281-b951-d872f2087c98