summaryrefslogtreecommitdiffstats
path: root/remoting
Commit message (Collapse)AuthorAgeFilesLines
* Add a flag field to PP_CompletionCallback to control if the callback shouldpolina@google.com2011-06-301-2/+2
| | | | | | | | | | | | | | | always be invoked asynchronously on success or error or skipped if the operation can complete synchronously without blocking. Keep the default behavior as-is until clients update their code. Bump revisions of all interfaces that take callbacks as args. Update browser interface function implementations and C++ layer to force callbacks if sync option is not set. Change ppapi/tests to run tests involving callbacks with both flag options. BUG=79376 TEST=ppapi_tests + bots Review URL: http://codereview.chromium.org/6899055 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91205 0039d316-1c4b-4281-b951-d872f2087c98
* Fix memory leak in remoting_unittests.sergeyu@chromium.org2011-06-302-17/+30
| | | | | | | | | BUG=None TEST=valgrind is happy. Review URL: http://codereview.chromium.org/7283033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91077 0039d316-1c4b-4281-b951-d872f2087c98
* Fix test for disabling curtain mode (and hiding DisconnectWindow, etc).wez@chromium.org2011-06-301-2/+2
| | | | | | | | | | BUG=87467 TEST= Review URL: http://codereview.chromium.org/7218064 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91075 0039d316-1c4b-4281-b951-d872f2087c98
* Don't use HttpPortAllocator when it is not neccessary.sergeyu@chromium.org2011-06-303-21/+25
| | | | | | | | | | | | HttpPortAllocator was making HTTP requests that we don't need when NAT traversing is disabled. BUG=None TEST=None Review URL: http://codereview.chromium.org/7273074 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91063 0039d316-1c4b-4281-b951-d872f2087c98
* Initial continue window implementation on mac.dmaclach@chromium.org2011-06-307-14/+125
| | | | | | | | | BUG=NONE TEST=Connect and then wait for the "timeout" to make sure dialog appears. Review URL: http://codereview.chromium.org/7260006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91062 0039d316-1c4b-4281-b951-d872f2087c98
* PseudoTcp to expose settings of nagling and ACK delayhclam@chromium.org2011-06-302-16/+101
| | | | | | | | | | | | | | | | Changes are as follows: 1. PseudoTcp to expose whether Nagle's algo is turned on. 2. PseudoTcp to expose setting of ACK delay. 3. JingleSession to disable Nagle's algo. 4. Set ACK delay to 10 milliseconds. 5. Unit tests for testing jingle session latency. BUG=None TEST=None Review URL: http://codereview.chromium.org/7275024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91057 0039d316-1c4b-4281-b951-d872f2087c98
* Remove the dependency on app_base from some targets.tfarina@chromium.org2011-06-291-1/+0
| | | | | | | | | | | BUG=72317 TEST=None R=rsesek@chromium.org Review URL: http://codereview.chromium.org/7283024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91045 0039d316-1c4b-4281-b951-d872f2087c98
* Disable tests.sergeyu@chromium.org2011-06-291-7/+7
| | | | | | | | BUG=None TEST=None TBR=hclam@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91041 0039d316-1c4b-4281-b951-d872f2087c98
* Remove JingleClient.sergeyu@chromium.org2011-06-2921-692/+339
| | | | | | | | | | | Everything that JingleClient was doing has been rolled into JingleSessionManager BUG=None TEST=Remoting still works. Review URL: http://codereview.chromium.org/7277050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91035 0039d316-1c4b-4281-b951-d872f2087c98
* Chromoting: Release all currently pressed keys when disconnecting.garykac@chromium.org2011-06-293-10/+64
| | | | | | | | | BUG=84544 TEST=unit Review URL: http://codereview.chromium.org/7275034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91007 0039d316-1c4b-4281-b951-d872f2087c98
* Use shared UI strings for Chromoting's Disconnect window (in Windows).garykac@google.com2011-06-292-3/+17
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/7277025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90961 0039d316-1c4b-4281-b951-d872f2087c98
* Tweak the TCPChannelTester to be easier to read.wez@chromium.org2011-06-291-9/+4
| | | | | | | | | | BUG= TEST= Review URL: http://codereview.chromium.org/7276043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90918 0039d316-1c4b-4281-b951-d872f2087c98
* Add FakeSignalStrategy.sergeyu@chromium.org2011-06-293-0/+146
| | | | | | | | | | | The new class will be used in tests instead of SessionManagerPair. BUG=None TEST=None Review URL: http://codereview.chromium.org/7276017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90889 0039d316-1c4b-4281-b951-d872f2087c98
* Resubmit "remoting_host_plugin to link with libvpx built from source"hclam@chromium.org2011-06-281-20/+9
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/7253060 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90883 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 90866 - remoting_host_plugin to link with libvpx built from sourcefischman@chromium.org2011-06-281-1/+20
| | | | | | | | | | | | | | | Remove the dependency on libvpx_lib but link to libvpx instead. This gets rid of linking to a pre-built library. BUG=None TEST=None Review URL: http://codereview.chromium.org/7262005 TBR=hclam@chromium.org Review URL: http://codereview.chromium.org/7272041 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90877 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 90867 - Add FakeSignalStrategy.sergeyu@chromium.org2011-06-283-146/+0
| | | | | | | | | | | | | | The new class will be used in tests instead of SessionManagerPair. BUG=None TEST=None Review URL: http://codereview.chromium.org/7276017 TBR=sergeyu@chromium.org Review URL: http://codereview.chromium.org/7276057 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90872 0039d316-1c4b-4281-b951-d872f2087c98
* Add FakeSignalStrategy.sergeyu@chromium.org2011-06-283-0/+146
| | | | | | | | | | | The new class will be used in tests instead of SessionManagerPair. BUG=None TEST=None Review URL: http://codereview.chromium.org/7276017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90867 0039d316-1c4b-4281-b951-d872f2087c98
* remoting_host_plugin to link with libvpx built from sourcehclam@chromium.org2011-06-281-20/+1
| | | | | | | | | | | | Remove the dependency on libvpx_lib but link to libvpx instead. This gets rid of linking to a pre-built library. BUG=None TEST=None Review URL: http://codereview.chromium.org/7262005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90866 0039d316-1c4b-4281-b951-d872f2087c98
* Add IDs to outgoing Iq stanzas.sergeyu@chromium.org2011-06-289-15/+40
| | | | | | | | | BUG=None TEST=Messages sent from client to host have valid IDs. Review URL: http://codereview.chromium.org/7276021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90864 0039d316-1c4b-4281-b951-d872f2087c98
* Move UI strings from Chromoting's Disconnect window into base class.garykac@chromium.org2011-06-284-15/+30
| | | | | | | | | | | | | This allows them to be shared with other platforms. Windows CL will follow. Also, move Linux code into remoting namespace to be consistent with other platforms. BUG=none TEST=none Review URL: http://codereview.chromium.org/7275012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90858 0039d316-1c4b-4281-b951-d872f2087c98
* Remove StartSession/EndSession from SignalStrategy interface.sergeyu@chromium.org2011-06-288-58/+29
| | | | | | | | | BUG=None TEST=remoting still works. Review URL: http://codereview.chromium.org/7277017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90836 0039d316-1c4b-4281-b951-d872f2087c98
* Reland 90618 - Remove check for entry_. From coverity CID=8237gbillock@chromium.org2011-06-281-2/+3
| | | | | | | | | | | Check result of inflateInit Coverity CID=14712 R=fischman@chromium.org Review URL: http://codereview.chromium.org/7253017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90696 0039d316-1c4b-4281-b951-d872f2087c98
* Fix buglet calling out to JS from plugin.wez@chromium.org2011-06-272-19/+12
| | | | | | | | | | BUG= TEST=Host plugin does not crash renderer if run in a Chrome instance set to run plugins in-process. Review URL: http://codereview.chromium.org/7265012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90659 0039d316-1c4b-4281-b951-d872f2087c98
* Move ContentDescription class to a separate file.sergeyu@chromium.org2011-06-275-392/+447
| | | | | | | | | BUG=None TEST=Compiles Review URL: http://codereview.chromium.org/7261011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90650 0039d316-1c4b-4281-b951-d872f2087c98
* Fixup error message shown when Chromoting is not enabled properly.garykac@google.com2011-06-271-9/+7
| | | | | | | | | | | | | | | | | | | Previously, we did a version check first, so if the plugin didn't load we would display an (inappropriate) "version mis-match, please upgrade" message. This was very frustrating since it often happened immediately after installation, when the user was trying to use Remoting for the first time. This change checks the plugin status first and (if there's a problem) directs the user to verify that the 'Remoting' and 'P2P API' flags are enabled. This partially addresses bug 87282, but we still need to make this look prettier. BUG=87282 TEST=manual Review URL: http://codereview.chromium.org/7263008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90626 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 90618 - Add checks for entry_. From coverity CID=8237fischman@chromium.org2011-06-271-2/+2
| | | | | | | | | | | | | | | | Check result of inflateInit Coverity CID=14712 BUG= TEST= Review URL: http://codereview.chromium.org/7222027 TBR=gbillock@chromium.org Review URL: http://codereview.chromium.org/7253014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90625 0039d316-1c4b-4281-b951-d872f2087c98
* Add checks for entry_. From coverity CID=8237gbillock@chromium.org2011-06-271-2/+2
| | | | | | | | | | | | | Check result of inflateInit Coverity CID=14712 BUG= TEST= Review URL: http://codereview.chromium.org/7222027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90618 0039d316-1c4b-4281-b951-d872f2087c98
* Build fix for Clanghclam@chromium.org2011-06-272-0/+4
| | | | | | TBR=fischman git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90610 0039d316-1c4b-4281-b951-d872f2087c98
* Add in storage abstraction object to remoting webapp.ajwong@chromium.org2011-06-272-0/+97
| | | | | | | | | BUG=none TEST=manual in dev console. Review URL: http://codereview.chromium.org/7237010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90608 0039d316-1c4b-4281-b951-d872f2087c98
* Implementation of SecureP2PSockethclam@chromium.org2011-06-274-0/+522
| | | | | | | | | | | Implement encryption on UDP for chromoting. BUG=None TEST=None Review URL: http://codereview.chromium.org/7038053 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90606 0039d316-1c4b-4281-b951-d872f2087c98
* Warmth of a connection (cwnd) is estimated by the amount of data written to ↵gagansingh@google.com2011-06-272-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | the socket. Choosing the warmest connection would mean faster resource load times. idle time is the time a socket has remained idle (no http requests being served on it). Probability of server resetting a connection increases with idle time duration. Using a cost function that takes into account bytes transferred and idle time to pick best connection to schedule http requests on. CODEREVIEW done in http://codereview.chromium.org/6990036/ Contributed by gagansingh@google.com Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=90373 Reverted: http://codereview.chromium.org/7255002 :( Have fixed 2 things since: 1. Removed LOG(ERROR) from http_basic_stream.cc that was causing layout tests to fail. 2. Initialized class variables in http_basic_stream.cc that was causing uninitialized memory bugs in valgrind: http://code.google.com/p/chromium/issues/detail?id=87423 Review URL: http://codereview.chromium.org/7251004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90601 0039d316-1c4b-4281-b951-d872f2087c98
* Added png since trybots can't handle binary patches.dmaclach@chromium.org2011-06-271-0/+0
| | | | | | | | | BUG=NONE TEST=NONE Review URL: http://codereview.chromium.org/7273006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90600 0039d316-1c4b-4281-b951-d872f2087c98
* The design docs say 10 minutes before continue window. So fixed to match ↵dmaclach@chromium.org2011-06-271-1/+1
| | | | | | | | | | | design docs. BUG=NONE TEST=Start up a connection and then sit and wait for 10 minutes. See if you get dialog. Review URL: http://codereview.chromium.org/7235012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90584 0039d316-1c4b-4281-b951-d872f2087c98
* Print round trip latency in chromoting webapphclam@chromium.org2011-06-241-1/+3
| | | | | | | | | | | | Print the latency of sending an event to receiving a recorresponding video frame. BUG=None TEST=None Review URL: http://codereview.chromium.org/7235011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90457 0039d316-1c4b-4281-b951-d872f2087c98
* Fix the problem of linux host mouse not movinghclam@chromium.org2011-06-242-3/+3
| | | | | | | | | | | | | | | LocalMouseMoved is called on a wrong thread and the window to track recent mouse movements was too small. This patch changes the call to happen on the network thread and increase the window to 50. BUG=87455 TEST=None Review URL: http://codereview.chromium.org/7263005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90452 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup client shutdown sequence.sergeyu@chromium.org2011-06-2410-61/+104
| | | | | | | | | BUG=None TEST=Client doesn't crash when reloading the tab. Review URL: http://codereview.chromium.org/7241016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90443 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup usage of JingleThread in the protocol code.sergeyu@chromium.org2011-06-2414-76/+52
| | | | | | | | | | | | JingleThread was used in some places where it is not necessary. Replace it with MessageLoop. BUG=None TEST=None Review URL: http://codereview.chromium.org/7237013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90419 0039d316-1c4b-4281-b951-d872f2087c98
* Remove support for non-sandboxed connections.sergeyu@chromium.org2011-06-2414-221/+31
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/7240011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90393 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 90373 - Warmth of a connection (cwnd) is estimated by the amount of ↵dimich@chromium.org2011-06-242-12/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | data written to the socket. Choosing the warmest connection would mean faster resource load times. Caused layout tests http/tests/loading/redirect-methods.html and http/tests/security/xss-DENIED-iframe-src-alias.html fail on deps WebKit bots, and caused http://code.google.com/p/chromium/issues/detail?id=87423 Deps bots failures: http://test-results.appspot.com/dashboards/flakiness_dashboard.html#group=%40ToT%20-%20chromium.org&tests=http%2Ftests%2Floading%2Fredirect-methods.html%2Chttp%2Ftests%2Fsecurity%2Fxss-DENIED-iframe-src-alias.html idle time is the time a socket has remained idle (no http requests being served on it). Probability of server resetting a connection increases with idle time duration. Using a cost function that takes into account bytes transferred and idle time to pick best connection to schedule http requests on. CODEREVIEW done in http://codereview.chromium.org/6990036/ Contributed by gagansingh@google.com Review URL: http://codereview.chromium.org/7189055 TBR=gagansingh@google.com Review URL: http://codereview.chromium.org/7255002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90389 0039d316-1c4b-4281-b951-d872f2087c98
* Warmth of a connection (cwnd) is estimated by the amount of data written to ↵gagansingh@google.com2011-06-242-0/+12
| | | | | | | | | | | | | | | | | | | the socket. Choosing the warmest connection would mean faster resource load times. idle time is the time a socket has remained idle (no http requests being served on it). Probability of server resetting a connection increases with idle time duration. Using a cost function that takes into account bytes transferred and idle time to pick best connection to schedule http requests on. CODEREVIEW done in http://codereview.chromium.org/6990036/ Contributed by gagansingh@google.com Review URL: http://codereview.chromium.org/7189055 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90373 0039d316-1c4b-4281-b951-d872f2087c98
* Fix JingleSession to respect net::StreamSocket Read/Write semantics.wez@chromium.org2011-06-242-69/+114
| | | | | | | | | | | | Remove some unnecessary state checks in PseudoTcpAdapter. BUG= TEST=Unit tests and Remoting components continue to work. Review URL: http://codereview.chromium.org/7104012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90334 0039d316-1c4b-4281-b951-d872f2087c98
* Fix crash in XmppSignalStrategy.sergeyu@chromium.org2011-06-232-2/+4
| | | | | | | | | BUG=None TEST=Remoting host doesn't crash Review URL: http://codereview.chromium.org/7236020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90295 0039d316-1c4b-4281-b951-d872f2087c98
* Remove remoting_x11_clientsergeyu@chromium.org2011-06-2310-653/+0
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/7243014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90289 0039d316-1c4b-4281-b951-d872f2087c98
* Refactoring around SignalStrategy interface.sergeyu@chromium.org2011-06-2310-112/+159
| | | | | | | | | | | | - Added AddListener/RemoveListener in the SignalStrategy interface. - Remove dependancy on JavascriptIqRequest from JingleSignalingConnector. BUG=None TEST=Chromoting client still works Review URL: http://codereview.chromium.org/7239005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90219 0039d316-1c4b-4281-b951-d872f2087c98
* Chromoting's ContinueWindow implementation for Windows.garykac@google.com2011-06-225-41/+159
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/7219002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90124 0039d316-1c4b-4281-b951-d872f2087c98
* Use the correct Chrome UI thread when creating the chromoting continue window.garykac@google.com2011-06-224-12/+13
| | | | | | | | | | | Also, rename the ChromotingHostContext's |ui_thread_| to |desktop_thread_| to reduce the likelihood that we'll make this mistake in the future. BUG=none TEST=none Review URL: http://codereview.chromium.org/7218018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90121 0039d316-1c4b-4281-b951-d872f2087c98
* Fix crash in ChromotingHost::Shutdown().sergeyu@chromium.org2011-06-221-1/+1
| | | | | | | | | | | The method was calling callback with a lock held which is incorrect. BUG=None TEST=Click disconnect button, the plugin doesn't crash. Review URL: http://codereview.chromium.org/7241002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90117 0039d316-1c4b-4281-b951-d872f2087c98
* Use a trampoline to avoid copy/pasting OAuth2 grant code.ajwong@chromium.org2011-06-228-34/+69
| | | | | | | | | BUG=none TEST=manual Review URL: http://codereview.chromium.org/7230001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90102 0039d316-1c4b-4281-b951-d872f2087c98
* Retrieve e-mail programmatically based on OAuth2 token.ajwong@chromium.org2011-06-224-25/+71
| | | | | | | | | BUG=none TEST=manual Review URL: http://codereview.chromium.org/7202010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90101 0039d316-1c4b-4281-b951-d872f2087c98
* Split iq_request.[h|cc] into multiple files.sergeyu@chromium.org2011-06-2216-510/+660
| | | | | | | | | BUG=None TEST=compiles Review URL: http://codereview.chromium.org/7211033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90073 0039d316-1c4b-4281-b951-d872f2087c98