summaryrefslogtreecommitdiffstats
path: root/remoting
Commit message (Collapse)AuthorAgeFilesLines
* Fixed warning generated by verify_webapp.pyjamiewalch@google.com2011-11-151-2/+2
| | | | | | | | | BUG=None TEST=Manual Review URL: http://codereview.chromium.org/8568023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110214 0039d316-1c4b-4281-b951-d872f2087c98
* Stop heartbeat timer when signalling is disconnected.sergeyu@chromium.org2011-11-151-0/+1
| | | | | | Review URL: http://codereview.chromium.org/8520021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110194 0039d316-1c4b-4281-b951-d872f2087c98
* Move home-screen functionality into separate file.jamiewalch@google.com2011-11-155-168/+216
| | | | | | | | | BUG=None TEST=Manual Review URL: http://codereview.chromium.org/8564037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110133 0039d316-1c4b-4281-b951-d872f2087c98
* Allow linker initialization of lazy instancejoth@chromium.org2011-11-154-4/+4
| | | | | | | | | | | | | | Using the initializer list construct = {0} allows the object to be linker initialized. Modify the LazyInstance class design to make it a pod aggregate type that can be linker initialized this way. Also combines the instance and state members, in line with the Singleton<> class design. Introduces a new LAZY_INSTANCE_INITIALIZER macro specifically for using to init all lazy instances + modify all existing callsites to use it. (Old code would no longer compile) BUG=94925 TEST=existing tests pass. http://build.chromium.org/f/chromium/perf/linux-release/sizes/report.html?history=150&header=chrome-si&graph=chrome-si&rev=-1 should step downward. TBR=jam@chromium.org,rvargas@chromium.org,darin@chromium.org,ben@chromium.org,apatrick@chromium.org,akalin@chromium.org Review URL: http://codereview.chromium.org/8491043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110076 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor ChannelAuthenticator so that it can be used with Authenticator.sergeyu@chromium.org2011-11-156-53/+51
| | | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/8527018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110051 0039d316-1c4b-4281-b951-d872f2087c98
* Send correct termination reason when session is terminated by the host.sergeyu@chromium.org2011-11-154-11/+33
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/8537020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110036 0039d316-1c4b-4281-b951-d872f2087c98
* Initial Authenticator interface.sergeyu@chromium.org2011-11-152-0/+93
| | | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/8508036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110008 0039d316-1c4b-4281-b951-d872f2087c98
* Fixed css to make host list prettier.jamiewalch@google.com2011-11-146-17/+86
| | | | | | | | | BUG=None TEST=Manual Review URL: http://codereview.chromium.org/8511077 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109968 0039d316-1c4b-4281-b951-d872f2087c98
* Report correct error when connection cannot be established.sergeyu@chromium.org2011-11-121-0/+9
| | | | | | | | | | BUG=103937 TEST=see the bug Review URL: http://codereview.chromium.org/8536032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109756 0039d316-1c4b-4281-b951-d872f2087c98
* Initialize clip_rect_ member to be empty until explicitly set.wez@chromium.org2011-11-111-0/+1
| | | | | | | | | BUG=98751 Review URL: http://codereview.chromium.org/8511068 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109731 0039d316-1c4b-4281-b951-d872f2087c98
* The chromoting client sends simple logging to the server.simonmorris@chromium.org2011-11-115-1/+349
| | | | | | Review URL: http://codereview.chromium.org/8527011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109712 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup: Remove unneeded forward declarations from remoting.thestig@chromium.org2011-11-1137-135/+41
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/8536002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109593 0039d316-1c4b-4281-b951-d872f2087c98
* Make ConnectionToClient and ClientSession not ref-counted.sergeyu@chromium.org2011-11-1114-119/+157
| | | | | | | | | | | | Also made then NonThreadSafe. BUG=96325 TEST=Chromoting works. Review URL: http://codereview.chromium.org/8495035 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109574 0039d316-1c4b-4281-b951-d872f2087c98
* Access ChromotingHost::clients_ only on network thread.sergeyu@chromium.org2011-11-1116-314/+256
| | | | | | | | | | | | | | | Previously ChromotingHost was doing some work on the main thread and some on the network thread. |clients_| and some other members were accessed without lock on both of these threads. Moved most of the ChromotingHost activity to the network thread to avoid possible race conditions. BUG=96325 TEST=Chromoting works Review URL: http://codereview.chromium.org/8495024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109556 0039d316-1c4b-4281-b951-d872f2087c98
* Removed Google from product name in fr and pt_BR translations.jamiewalch@google.com2011-11-102-2/+2
| | | | | | | | | BUG=103813 TEST=Manual Review URL: http://codereview.chromium.org/8524013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109538 0039d316-1c4b-4281-b951-d872f2087c98
* Fix the web-app not to fail if the user is not Me2Me enabled.wez@chromium.org2011-11-101-2/+4
| | | | | | | | | TEST=Run Chromoting web-app under an account that is not enabled for Me2Me. Web-app should still allow IT2Me functions to be accessed. Review URL: http://codereview.chromium.org/8509028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109474 0039d316-1c4b-4281-b951-d872f2087c98
* Move ConnectionToClient::EventHandler from ChromotingHost to ClientSessionsergeyu@chromium.org2011-11-1013-208/+193
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/8476018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109457 0039d316-1c4b-4281-b951-d872f2087c98
* -Wexit-time-destructors for remoting.thakis@chromium.org2011-11-104-12/+29
| | | | | | | | | | | Also remove 1 exit time destructor and 2 static initializers. BUG=101600,94925 TEST=none Review URL: http://codereview.chromium.org/8509026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109376 0039d316-1c4b-4281-b951-d872f2087c98
* Move code in src/remoting to the new callbacks.sergeyu@chromium.org2011-11-1058-357/+254
| | | | | | | | | BUG=None TEST=Remoting still works. Review URL: http://codereview.chromium.org/8493020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109367 0039d316-1c4b-4281-b951-d872f2087c98
* Fix intermittent CHECK in remoting client when shutting down.sergeyu@chromium.org2011-11-101-0/+6
| | | | | | | | | | | | MessageReader::~MessageReader() may CHECK because ChromotingClient may be destroyed without destroying all pending video packets. BUG=None TEST=None Review URL: http://codereview.chromium.org/8507025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109355 0039d316-1c4b-4281-b951-d872f2087c98
* Fixed capitalization that broke jscompile.jamiewalch@google.com2011-11-091-1/+1
| | | | | | | | | BUG=None TEST=jscompile runs without errors. Review URL: http://codereview.chromium.org/8497047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109275 0039d316-1c4b-4281-b951-d872f2087c98
* Correct errant references to remoting.ClientError to remoting.Error.wez@chromium.org2011-11-081-2/+2
| | | | | | | | | TEST=Remoting JavaScript should "compile" cleanly with jscompile. Review URL: http://codereview.chromium.org/8499001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108968 0039d316-1c4b-4281-b951-d872f2087c98
* Fix typo that prevents client displaying received frame rate.wez@chromium.org2011-11-082-10/+8
| | | | | | | | | TEST=Run CRD and press D to enable debug logging, then connect to host. Frame Rate should display a value, rather than 'n/a'. Review URL: http://codereview.chromium.org/8492020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108964 0039d316-1c4b-4281-b951-d872f2087c98
* Add missing return in access-token refresh case of refreshHostList.wez@chromium.org2011-11-081-0/+1
| | | | | | | | | TEST=No "Uncaught Access Token expired" errors logged in JavaScript console if CRD app is opened with a refresh token stored, but no access token cached. Review URL: http://codereview.chromium.org/8493021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108956 0039d316-1c4b-4281-b951-d872f2087c98
* Pretty-print IQ packets in Chromoting debug log.garykac@chromium.org2011-11-073-4/+579
| | | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/8330004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108939 0039d316-1c4b-4281-b951-d872f2087c98
* Added official translations for M16.jamiewalch@google.com2011-11-0741-41/+41
| | | | | | | | | BUG=100887 TEST=Manual Review URL: http://codereview.chromium.org/8455003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108891 0039d316-1c4b-4281-b951-d872f2087c98
* Gather history of capture and encode time determine next recoring delayhclam@chromium.org2011-11-076-100/+139
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The current logic of chromoting is to capture as fast as possible with 20fps as a hard limit. This creates a heavy load on the slower systems with only one processor. I measured 100% CPU load running on these systems, resulting in very jerky performance. In addition to speed up capture and encode we should use system load and configurations as an information to determine recording frequency. In this patch I used the number of processors and history of capture and encode time to determine the next recording delay. This essentially lower recoding frequency when system is under load or is incapable to catch up with the current 20fps frequency. I have tested this patch on a Linux system limited to single core and intentionally added extra 70ms of while(1) loop in capture and encode to simulate slow processing. The results are as follows: CPU load Latency Before 100% 440ms After ~50% 360ms So as a result both CPU load and latency improved. BUG=100314 TEST=remoting_unittests and manually test with with heavily loaded system Review URL: http://codereview.chromium.org/8342040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108871 0039d316-1c4b-4281-b951-d872f2087c98
* Remove SecureP2PSocket, since we'll be using DTLS in future.wez@chromium.org2011-11-054-525/+0
| | | | | | | | | BUG=90802 Review URL: http://codereview.chromium.org/8477020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108792 0039d316-1c4b-4281-b951-d872f2087c98
* Add verify-translations.py script.wez@chromium.org2011-11-051-0/+49
| | | | | | Review URL: http://codereview.chromium.org/8475019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108791 0039d316-1c4b-4281-b951-d872f2087c98
* Fix memory leak in ClientSessionTestsergeyu@chromium.org2011-11-051-2/+0
| | | | | | | | | | | TEST=valgrind is happy BUG=None TBR=wez@chromium.org Review URL: http://codereview.chromium.org/8478023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108787 0039d316-1c4b-4281-b951-d872f2087c98
* Remove old Authentication code that we don't use or need.sergeyu@chromium.org2011-11-0536-479/+80
| | | | | | | | | | | | Also removed some auth-related protobufs that we don't currenly use. BUG=None TEST=IT2Me still works. Review URL: http://codereview.chromium.org/8351084 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108759 0039d316-1c4b-4281-b951-d872f2087c98
* React to error messages received in response to transport-info.sergeyu@chromium.org2011-11-052-3/+23
| | | | | | | | | | BUG=100822 TEST=client disconnects when host dies. Review URL: http://codereview.chromium.org/8437017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108754 0039d316-1c4b-4281-b951-d872f2087c98
* Fix leak in IqSenderTest.sergeyu@chromium.org2011-11-042-4/+6
| | | | | | | | | BUG=102983 TEST=No leak Review URL: http://codereview.chromium.org/8481011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108737 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor IqRequest.sergeyu@chromium.org2011-11-0429-456/+485
| | | | | | | | | | | | | | Remove CreateIqRequest from SignalStrategy interface. Intead to send an Iq stanza the new IqSender now need to be used. IqSender creats of IqRequest objects and handling iq responses. BUG=None TEST=Unittests. Review URL: http://codereview.chromium.org/8432009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108606 0039d316-1c4b-4281-b951-d872f2087c98
* Centre the disconnect dialog at the bottom of the primary monitor.jamiewalch@chromium.org2011-11-031-24/+39
| | | | | | | | | BUG=101178 TEST=Manual Review URL: http://codereview.chromium.org/8438056 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108540 0039d316-1c4b-4281-b951-d872f2087c98
* Fixed UX on ChromeOS.jamiewalch@google.com2011-11-034-25/+24
| | | | | | | | | BUG=None TEST=Check that Share Now is disabled on ChromeOS, with accompanying text explaining why. Review URL: http://codereview.chromium.org/8439063 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108527 0039d316-1c4b-4281-b951-d872f2087c98
* Add missing closing brace to img tag.wez@chromium.org2011-11-031-1/+1
| | | | | | Review URL: http://codereview.chromium.org/8450003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108518 0039d316-1c4b-4281-b951-d872f2087c98
* Add client_screen.js to I18N verification.wez@chromium.org2011-11-031-0/+2
| | | | | | Review URL: http://codereview.chromium.org/8439066 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108489 0039d316-1c4b-4281-b951-d872f2087c98
* Small web-app clean-ups.jamiewalch@chromium.org2011-11-035-35/+34
| | | | | | | | | | BUG=None TEST=Manual Review URL: http://codereview.chromium.org/8351080 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108381 0039d316-1c4b-4281-b951-d872f2087c98
* Fix the Chromoting Host plugin's MIME type on Linux.wez@chromium.org2011-11-031-1/+1
| | | | | | | | | The MIME type had acquired double-quotes in a recent CL, preventing the plugin from being detected and loaded. Review URL: http://codereview.chromium.org/8387055 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108376 0039d316-1c4b-4281-b951-d872f2087c98
* An ugly but working host-list implementation for the Chromoting app.wez@chromium.org2011-11-034-0/+225
| | | | | | | | | | | | | | | Hosts will have a Connect button if online, and the text Offline otherwise. The host list will display only once one or more hosts are registered, for now. Connection setup code still needs to be rationalized between the two modes. The necessary host-side changes will arrive in a separate CL. Review URL: http://codereview.chromium.org/8432021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108375 0039d316-1c4b-4281-b951-d872f2087c98
* Make Mac Chromoting Disconnect Window look good for R2L systems.dmaclach@chromium.org2011-11-024-35/+132
| | | | | | | | | | BUG=NONE TEST=Try chromoting in arabic. Review URL: http://codereview.chromium.org/8364025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108360 0039d316-1c4b-4281-b951-d872f2087c98
* removing unnecessary includeamit@chromium.org2011-11-021-1/+0
| | | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/8386018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108343 0039d316-1c4b-4281-b951-d872f2087c98
* Make sure curtain mode doesn't lock up MacOS 10.7dmaclach@chromium.org2011-11-012-12/+24
| | | | | | | | | | | | Also adds some extra error logging. BUG=None TEST=Go into curtain mode on 10.7 and then disconnect the session. Review URL: http://codereview.chromium.org/8384011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108184 0039d316-1c4b-4281-b951-d872f2087c98
* Make ContinueWindow on Mac terminate after timeoutdmaclach@chromium.org2011-11-011-64/+91
| | | | | | | | | | BUG=100928 TEST=see bug Review URL: http://codereview.chromium.org/8423018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108182 0039d316-1c4b-4281-b951-d872f2087c98
* Change linux NAT traversal default for Chromoting.dmaclach@chromium.org2011-11-011-12/+0
| | | | | | | | | | | | | The NAT traversal default for all platforms is now true. BUG=97689 TEST=None Review URL: http://codereview.chromium.org/8423020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108181 0039d316-1c4b-4281-b951-d872f2087c98
* Make LazyInstance managed Locks leaky.thestig@chromium.org2011-11-012-2/+6
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/8430019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108159 0039d316-1c4b-4281-b951-d872f2087c98
* Make X509Certificate::GetDEREncoded a static function taking an OSCertHandlersleevi@chromium.org2011-11-011-3/+5
| | | | | | | | | | | | | | | | Rather than require an X509Certificate*, which has additional processing overhead, make X509Certificate::GetDEREncoded a static function which takes an OSCertHandle. Callers which already have an X509Certificate* can easily use ->os_cert_handle(), while those that have an OSCertHandle, such as by way of GetIntermediateCertificates(), can use the OSCertHandle directly. BUG=91464 TEST=none Review URL: http://codereview.chromium.org/8414047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108067 0039d316-1c4b-4281-b951-d872f2087c98
* Roll libjingle 88:92sergeyu@chromium.org2011-11-013-3/+3
| | | | | | | | | | | | | | Beside other changes the new version of libjingle has thread-safe QName class and has all std::string and QName statics removed. BUG=94993,102451 TEST=Compiles Review URL: http://codereview.chromium.org/8413059 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108065 0039d316-1c4b-4281-b951-d872f2087c98
* Don't send a session-terminate in disconnect() if WCS not initialised.wez@chromium.org2011-11-011-14/+14
| | | | | | | | | sendIq_() requires that WCS is initialized when it is called, otherwise it will switch the session into the "failed" state. (The state change causes a disconnect() call, which triggers a sendIq_() and the tab locks up.) Review URL: http://codereview.chromium.org/8427020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108061 0039d316-1c4b-4281-b951-d872f2087c98