summaryrefslogtreecommitdiffstats
path: root/remoting
Commit message (Collapse)AuthorAgeFilesLines
* Disable JingleSessionTest.Connect, JingleSessionTest.TestControlChannelphajdan.jr@chromium.org2011-02-021-2/+4
| | | | | | | | | | | | | | Those tests flakily trigger a libjingle assertion: Error(common.cc:67): e:\b\build\slave\win\build\src\third_party\libjingle\source\talk\session\tunnel\pseudotcpchannel.cc(281): ASSERT FAILED: stream_ != NULL && stream_thread_->IsCurrent() @ cricket::PseudoTcpChannel::Close TBR=hclam BUG=70225 TEST=remoting_unittests Review URL: http://codereview.chromium.org/6334046 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73436 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup everything but net/ for our first clang plugins.erg@google.com2011-02-021-1/+1
| | | | | | | | | BUG=none TEST=compiles Review URL: http://codereview.chromium.org/6250088 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73396 0039d316-1c4b-4281-b951-d872f2087c98
* Stop ScreenRecorder before it startedhclam@chromium.org2011-02-022-3/+7
| | | | | | | | | | | Allow ScreenRecorder to be stopped anytime. BUG=None TEST=None Review URL: http://codereview.chromium.org/6246034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73392 0039d316-1c4b-4281-b951-d872f2087c98
* Disable login panel correctly for chromoting client pluginhclam@chromium.org2011-02-011-1/+1
| | | | | | | | | | | TBR=sergeyu BUG=None TEST=None Review URL: http://codereview.chromium.org/6312065 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73325 0039d316-1c4b-4281-b951-d872f2087c98
* Remove obsolete files from 'sources'. This causes needless work on ↵jeanluc@chromium.org2011-01-311-2/+0
| | | | | | | | | | | incremental builds in VS2010. BUG=71130 TEST=Check that Chrome builds. Review URL: http://codereview.chromium.org/6386019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73190 0039d316-1c4b-4281-b951-d872f2087c98
* Chromoting client extension: Properly encode ClientLogin request.sergeyu@chromium.org2011-01-311-2/+4
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/6378015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73178 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 73121 - Remove obsolete files from 'sources'. This causes needless ↵nsylvain@chromium.org2011-01-311-0/+2
| | | | | | | | | | | | | | | | | | work on incremental builds in VS2010. This is breaking clobber builds with this error : e:\b\build\slave\google-chrome-rel-win\build\src\chrome_frame\iids.cc(6) : fatal error C1083: Cannot open include file: 'chrome_tab.h': No such file or directory BUG=71130 TEST=Check that Chrome builds. Review URL: http://codereview.chromium.org/6307018 TBR=jeanluc@chromium.org Review URL: http://codereview.chromium.org/6347032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73124 0039d316-1c4b-4281-b951-d872f2087c98
* Remove obsolete files from 'sources'. This causes needless work on ↵jeanluc@chromium.org2011-01-301-2/+0
| | | | | | | | | | | incremental builds in VS2010. BUG=71130 TEST=Check that Chrome builds. Review URL: http://codereview.chromium.org/6307018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73121 0039d316-1c4b-4281-b951-d872f2087c98
* Handle Pepper ContextMenu event in Chromoting client plugingarykac@chromium.org2011-01-291-0/+5
| | | | | | | | | BUG=none TEST=manual testing Review URL: http://codereview.chromium.org/6265032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73063 0039d316-1c4b-4281-b951-d872f2087c98
* Chromoting: JS code to display to hide login dialoghclam@chromium.org2011-01-293-6/+31
| | | | | | | | | | | | Extension JS code to display and hide login dialog. Also allow user to enter username and password. BUG=None TEST=none Review URL: http://codereview.chromium.org/6133005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73062 0039d316-1c4b-4281-b951-d872f2087c98
* Chromoting host to respond to BeginSessionRequesthclam@chromium.org2011-01-297-35/+146
| | | | | | | | | | | | ChromotingHost to respond to BeginSessionRequest, pretending login was successul. This allows chromoting client to send login command and proceed. BUG=None TEST=None Review URL: http://codereview.chromium.org/6144002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73052 0039d316-1c4b-4281-b951-d872f2087c98
* Resubmit "Fix crashes in ChromotingHost"hclam@chromium.org2011-01-2812-48/+284
| | | | | | | | | | | | Simplified the test to run on the main message loop. Also fixed a possible memory leak caught by valgrind. BUG=70903 TEST=None Review URL: http://codereview.chromium.org/6245016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73023 0039d316-1c4b-4281-b951-d872f2087c98
* Chromoting pepper plugin to respond to BeginSessionResponsehclam@chromium.org2011-01-2813-28/+132
| | | | | | | | | | | | Trigger loginChallenge when the client is connected or the last login attempt has failed. BUG=None TEST=None Review URL: http://codereview.chromium.org/6160001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73022 0039d316-1c4b-4281-b951-d872f2087c98
* Remove aggressive DCHECK in remoting::protocol::MessageReader()hclam@chromium.org2011-01-281-2/+0
| | | | | | | | | | | DCHECK(!read_pending_) actually may happen and causes unnecessary crashes. BUG=None TEST=None Review URL: http://codereview.chromium.org/6359018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72971 0039d316-1c4b-4281-b951-d872f2087c98
* Use a random number to generate a self-signed certiciate for chromotinghclam@chromium.org2011-01-271-1/+5
| | | | | | | | | | | | | | | | Chromoting host to use a random number to generate self-signed certificate. This method is still not optimal, in case of collision SSLServerSocketNSS will fail to start due to NSS rejecting the certificate. Using a random number is a temporary solution until we can assign certificates to the host. BUG=70013 TEST=Connect to the same chromoting host after client is disconnected. Review URL: http://codereview.chromium.org/6315009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72745 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Fix crashes in ChromotingHost"hclam@chromium.org2011-01-2610-255/+46
| | | | | | | | | | | | | | Reverting the patch since it exposed several memory leaks and threading problems. TBR=thakis BUG=70935 TEST=None Review URL: http://codereview.chromium.org/6266023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72679 0039d316-1c4b-4281-b951-d872f2087c98
* Fixed memory leaks in remoting_unittests.sergeyu@chromium.org2011-01-261-6/+11
| | | | | | | | | BUG=70826 TEST=remoting_unittests under valgrind Review URL: http://codereview.chromium.org/6266020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72601 0039d316-1c4b-4281-b951-d872f2087c98
* Add PepperViewProxy to protect PepperView and ChromotingInstance on shutdownhclam@chromium.org2011-01-268-52/+273
| | | | | | | | | | | | | | Adding a refcounted PepperViewProxy so that we can detach PepperView when ChromotingInstance is destroyed. PepperViewProxy also performs the task of thread delegation for PepperView so we can assume PepperView is used only on pepper thread. BUG=65696 TEST=None Review URL: http://codereview.chromium.org/6359010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72568 0039d316-1c4b-4281-b951-d872f2087c98
* Fix crashes in ChromotingHosthclam@chromium.org2011-01-2510-46/+255
| | | | | | | | | | | | Capturer should be owned by ScreenRecoder. This patch also addes test to test reconnection of ChromotingHost. BUG=69969 TEST=ChromotingHostTest.* Review URL: http://codereview.chromium.org/6266010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72543 0039d316-1c4b-4281-b951-d872f2087c98
* Changed MessageReader so that it doesn't read from the socket if there aresergeyu@chromium.org2011-01-2220-152/+430
| | | | | | | | | | | other messages being processed. Added unittests for MessageReader. BUG=None TEST=Unittests Review URL: http://codereview.chromium.org/6271004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72262 0039d316-1c4b-4281-b951-d872f2087c98
* Add a done task to ScreenRecorder::Stop()hclam@chromium.org2011-01-214-71/+226
| | | | | | | | | | | | ScreenRecorder::Stop() need a done task to indicate it is actually paused. This is needed so that we can shutdown threads safely. BUG=69997 TEST=remoting_unittests --gtest_filter=ScreenRecorder.* Review URL: http://codereview.chromium.org/6282006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72188 0039d316-1c4b-4281-b951-d872f2087c98
* Enable session resumption in SSLServerSocketNSS and unit testshclam@chromium.org2011-01-211-1/+0
| | | | | | | | | | | | SSL_ConfigServerSessionIDCache() is being used in SSLServerSocketNSS and enabling session resumption since passes are now passing. BUG=67236 TEST=net_unittests, remoting_unittests Review URL: http://codereview.chromium.org/6370008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72186 0039d316-1c4b-4281-b951-d872f2087c98
* Fix memory leak in JingleSessionTest.Connect and friendshclam@chromium.org2011-01-211-1/+1
| | | | | | | | | | | | Private key is cloned instead of transferred. This eliminates the leak in JingleSessionTest.*. BUG=70291 TEST=valgrind bot is green Review URL: http://codereview.chromium.org/6317009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72164 0039d316-1c4b-4281-b951-d872f2087c98
* Remove obsolete base/lock.h and fix up callers to use the new header file andbrettw@chromium.org2011-01-2116-48/+47
| | | | | | | | | | | the base namespace. Fix several files including lock.h unnecessarily. BUG=none TEST=none Original review=http://codereview.chromium.org/6142009/ Patch by leviw@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72106 0039d316-1c4b-4281-b951-d872f2087c98
* Enable JingleSessionTest on windowshclam@chromium.org2011-01-191-23/+1
| | | | | | | | | | | TBR=wtc BUG=67928 TEST=JingleSessionTest.* should pass on XP bot Review URL: http://codereview.chromium.org/6338010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71875 0039d316-1c4b-4281-b951-d872f2087c98
* Simplified MessageReader and MessageDecoder classes.sergeyu@chromium.org2011-01-1417-164/+110
| | | | | | | | | | | | Now these two classes are not templates, and just handle raw data. New ProtobufMessageReader is used to parse messages. BUG=None TEST=Unittests. Review URL: http://codereview.chromium.org/6277003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71497 0039d316-1c4b-4281-b951-d872f2087c98
* Move app/key* to ui/base/keycodes/*ben@chromium.org2011-01-132-2/+2
| | | | | | | | | | BUG=none TEST=none TBR=brettw Review URL: http://codereview.chromium.org/6246001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71330 0039d316-1c4b-4281-b951-d872f2087c98
* Chromoting plugin script interface to support login challengehclam@chromium.org2011-01-102-5/+50
| | | | | | | | | | | | | Added an attribute to accept loginChallenge callback. This callback will be called with another callback, JS code is supposed to call the received callback to provide username and password. BUG=None TEST=None Review URL: http://codereview.chromium.org/5962008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70939 0039d316-1c4b-4281-b951-d872f2087c98
* Add an instance parameter to var objects, audio, and the 2D API. This ↵brettw@chromium.org2011-01-072-4/+5
| | | | | | | | | | | | | | | | replaces the module in most cases. This will be used in the proxy to multiplex one plugin across multiple renderer processes. We need the instance in the proxy to know which process to send it to. I added a deprecated var object creation function for native client, which depends on the module and this is very difficult to change. Because it doesn't have the multiplexing requirements, this is fine for now. TEST=ppapi ui tests BUG=none Review URL: http://codereview.chromium.org/6085009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70721 0039d316-1c4b-4281-b951-d872f2087c98
* Move CancellationFlag and WaitableEvent to the synchronization subdirectory.brettw@chromium.org2011-01-025-5/+5
| | | | | | | | TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/5977010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70369 0039d316-1c4b-4281-b951-d872f2087c98
* Move base/lock and base/condition_variable to base/synchronization/brettw@chromium.org2011-01-011-8/+8
| | | | | | | | | | | I kept a base/lock.h in place with a using statement to avoid updating all callers in one CL. TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/6018013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70363 0039d316-1c4b-4281-b951-d872f2087c98
* Remove base/platform_thread.h stub and fix up all callers to use the new ↵brettw@google.com2011-01-011-1/+1
| | | | | | | | | location and namespace. TEST=none BUG=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70346 0039d316-1c4b-4281-b951-d872f2087c98
* Move base/thread.h to base/threading, fix up callers to use the new location.brettw@chromium.org2011-01-018-8/+8
| | | | | | | | TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/6028009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70345 0039d316-1c4b-4281-b951-d872f2087c98
* Move platform_thread to base/threading and put in the base namespace. I left abrettw@chromium.org2010-12-311-1/+2
| | | | | | | | | | | stub and "using" declarations in the old location to avoid having to change the entire project at once. TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/6001010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70342 0039d316-1c4b-4281-b951-d872f2087c98
* Move thread local stuff from base to base/threading and consistently use thebrettw@chromium.org2010-12-301-1/+1
| | | | | | | | | | | base namespace. This also fixes references to TLSSlot to base::ThreadLocalStorage::Slot. TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/5986012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70322 0039d316-1c4b-4281-b951-d872f2087c98
* Fix crash in JingleThread caused by non-nestable tasks.sergeyu@chromium.org2010-12-232-7/+37
| | | | | | | | | BUG=None TEST=Unittests Review URL: http://codereview.chromium.org/6003003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70115 0039d316-1c4b-4281-b951-d872f2087c98
* Disable JingleSessionTests on Windowshclam@chromium.org2010-12-231-1/+2
| | | | | | | | | | | | Fixing remoting_unittests crash on XP. TBR=sergeyu BUG=67928 TEST=None Review URL: http://codereview.chromium.org/5963005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70103 0039d316-1c4b-4281-b951-d872f2087c98
* Use certificate and key files instead of generating them each time for unit ↵hclam@chromium.org2010-12-233-10/+58
| | | | | | | | | | | tests. BUG= TEST= Review URL: http://codereview.chromium.org/6018008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70097 0039d316-1c4b-4281-b951-d872f2087c98
* Send certificate in content description in jingle for Chromotinghclam@chromium.org2010-12-235-61/+96
| | | | | | | | | | | | Transmit the server certificate to the client via content description. This allow secure encryption between Chromoting host and client. BUG=None TEST=remoting_unittests Review URL: http://codereview.chromium.org/5804001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70075 0039d316-1c4b-4281-b951-d872f2087c98
* Add an extra SSL layer in JingleSession for Chromotinghclam@chromium.org2010-12-237-68/+400
| | | | | | | | | | | | | | | Wrap the existing StreamSocketAdpaters in JingleSession with an extra of SSLClientSocket and SSLServerSocket. Since the server certificate is self-signed, SSLClientSocket will refuse to connect. An additional patch is needed to transmit the certificate via other channels so client sSL socket can accept this untrusted certificate. BUG=None TEST=remoting_unittests Review URL: http://codereview.chromium.org/5675003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70074 0039d316-1c4b-4281-b951-d872f2087c98
* Chromoting protocol layers to receive and send login messageshclam@chromium.org2010-12-2320-113/+297
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/6030007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70036 0039d316-1c4b-4281-b951-d872f2087c98
* Protocol buffer definition for local loginhclam@chromium.org2010-12-233-2/+47
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/5970008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70035 0039d316-1c4b-4281-b951-d872f2087c98
* Add CertVerifier to SocketStreamAdapterhclam@chromium.org2010-12-232-1/+8
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/6072007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70030 0039d316-1c4b-4281-b951-d872f2087c98
* Remove GetVp8CxAlgoAddress hackhclam@chromium.org2010-12-202-11/+8
| | | | | | | | | | | | libvpx has introduced vpx_codec_vp8_cx() and vpx_codec_vp8_dx() use them instead. BUG=None TEST=None Review URL: http://codereview.chromium.org/5831006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69745 0039d316-1c4b-4281-b951-d872f2087c98
* HostConfig interface updates.sergeyu@chromium.org2010-12-1611-96/+40
| | | | | | | | | | | | | | | - Simplified mutation semantic: Update() replaced with Save(). Set methods can be called at any time. - JsonHostConfig now inherits from InMemoryHostConfig to avoid code duplication. - Added kHostEnabledConfigPath. BUG=None TEST=Unittests. Review URL: http://codereview.chromium.org/5738008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69454 0039d316-1c4b-4281-b951-d872f2087c98
* Fixed memleak in HeartbeatSenderTest_ProcessResponse.sergeyu@chromium.org2010-12-161-2/+2
| | | | | | | | | BUG=66808 TEST=Unittests Review URL: http://codereview.chromium.org/5761003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69448 0039d316-1c4b-4281-b951-d872f2087c98
* Cache certificate verification results in memory.wtc@google.com2010-12-161-1/+2
| | | | | | | | | R=agl BUG=63357 TEST=none Review URL: http://codereview.chromium.org/5386001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69414 0039d316-1c4b-4281-b951-d872f2087c98
* Fix the last scoped_refptr misuses in NewRunnableMethod.willchan@chromium.org2010-12-151-2/+2
| | | | | | | | | BUG=28083 TEST=builds Review URL: http://codereview.chromium.org/5852001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69220 0039d316-1c4b-4281-b951-d872f2087c98
* Pass RefCounted objects as scoped_refptr to NewRunnableMethod.willchan@chromium.org2010-12-141-3/+3
| | | | | | | | | BUG=28083 TEST=none Review URL: http://codereview.chromium.org/5740005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69183 0039d316-1c4b-4281-b951-d872f2087c98
* Make members of Singleton<T> private and only visible to the singleton type. ↵satish@chromium.org2010-12-141-4/+10
| | | | | | | | | | | | | | | This enforces that the Singleton<T> pattern can only be used within classes which want singleton-ness. As part of this CL I have also fixed up files which got missed in my previous CLs to use a GetInstance() method and use Singleton<T> from the source file. There are a small number of places where I have also switched to LazyInstance as that was more appropriate for types used in a single source file. BUG=65298 TEST=all existing tests should continue to pass. Review URL: http://codereview.chromium.org/5682008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69107 0039d316-1c4b-4281-b951-d872f2087c98