summaryrefslogtreecommitdiffstats
path: root/remoting
Commit message (Collapse)AuthorAgeFilesLines
* FBTF: Continue fixing nits found by my automated source scanner.erg@google.com2010-10-054-3/+13
| | | | | | | | | | | (~1.3 megs off Debug linux .a files) BUG=none TEST=compiles Review URL: http://codereview.chromium.org/3556013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61587 0039d316-1c4b-4281-b951-d872f2087c98
* Fix races in JingleChromotingConnectionTest.sergeyu@chromium.org2010-10-051-3/+3
| | | | | | | | | BUG=57692 TEST=remoting_unittests under TSan. Review URL: http://codereview.chromium.org/3541010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61469 0039d316-1c4b-4281-b951-d872f2087c98
* Reapply r61237: FBTF: Another big ctor/dtor cleanup found by automated tools.erg@google.com2010-10-049-16/+45
| | | | | | | | | | | | | | | Removes changes to code in webkit/ that broke chrome_frame. Will debug that portion later. (Shaves ~2MB off Linux Debug .a files) BUG=none TEST=compiles First Review URL: http://codereview.chromium.org/3563004 Review URL: http://codereview.chromium.org/3621003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61435 0039d316-1c4b-4281-b951-d872f2087c98
* This is a monster CL.ajwong@chromium.org2010-10-0448-1267/+909
| | | | | | | | | | | | | | | | | | | | It started as an attempt to put the decoder onto another thread. However, this became complicated due to multiple object ownership transfers and coupling between the decode layer and the network layer; the decoder's states were highly coupled with how the network packets were processed. This could probably be broken up slightly, but at this point, it's easier to just commit as a whole The refactor includes: 1) Making the decoder interface unaware of "network packet" types. 2) Making the network layer process packets in order. 3) Threading through asynchronous APIs all over the place. 4) Simplifying the rectangle update protocol. 5) Cleaning up object lifetime and ownership semantics between the decode layer and the renderer. As of right now, the Verbatim format is still broken on the encode side because it uses the old protocol. BUG=52883, 57351 TEST=still connects to chromoting_simple_host Review URL: http://codereview.chromium.org/3305001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61402 0039d316-1c4b-4281-b951-d872f2087c98
* Move the remoting gtest filter to the proper place.glider@chromium.org2010-10-041-5/+0
| | | | | | | BUG=57832 Review URL: http://codereview.chromium.org/3534009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61377 0039d316-1c4b-4281-b951-d872f2087c98
* Disable JingleChromotingConnectionTest tests that are failing under TSan onglider@chromium.org2010-10-041-0/+5
| | | | | | | | | | | Win. BUG=57832 TBR=timurrrr Review URL: http://codereview.chromium.org/3534007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61366 0039d316-1c4b-4281-b951-d872f2087c98
* clang/mac: Fix almost all problems when building all.xcodeproj.thakis@chromium.org2010-10-042-5/+10
| | | | | | | | BUG=None TEST=None TBR=hans git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61344 0039d316-1c4b-4281-b951-d872f2087c98
* Fix flakiness in JingleChromotingConnectionTestsergeyu@chromium.org2010-10-023-5/+16
| | | | | | | | TEST=remoting_unittests BUG=None TBR=hclam git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61314 0039d316-1c4b-4281-b951-d872f2087c98
* Fix intermittent failure in JingleThreadTest.PostDelayedTask.sergeyu@chromium.org2010-10-021-1/+1
| | | | | | TBR=hclam git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61268 0039d316-1c4b-4281-b951-d872f2087c98
* Disable TestVideoRtpChannel of remoting_unittests because of flakinesshclam@chromium.org2010-10-011-1/+3
| | | | | | | | | | | TBR=sergeyu BUG=None TEST=None Review URL: http://codereview.chromium.org/3555009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61260 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "FBTF: Another big ctor/dtor cleanup found by automated tools."erg@google.com2010-10-0113-51/+16
| | | | | | | | | | | This reverts commit 27ea47d65cf8767f350113d5ad9e25170efde811 (r61237). BUG=none TEST=compiles Review URL: http://codereview.chromium.org/3609005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61240 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Another big ctor/dtor cleanup found by automated tools.erg@google.com2010-10-0113-16/+51
| | | | | | | | | | | (Shaves ~2MB off Linux Debug .a files) BUG=none TEST=compiles Review URL: http://codereview.chromium.org/3563004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61237 0039d316-1c4b-4281-b951-d872f2087c98
* Unittests for chromotocol code.sergeyu@chromium.org2010-10-0112-24/+1057
| | | | | | | | | BUG=53986 TEST=Unittests FTW! Review URL: http://codereview.chromium.org/3407007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61228 0039d316-1c4b-4281-b951-d872f2087c98
* protobuf: switch to using third_party/protobufagl@chromium.org2010-10-012-8/+8
| | | | | | | | | | | | This change lights up new code, but the code should be identical to the code in protobuf2, so no functional changes are expected. BUG=56579 TEST=sync_unit_tests http://codereview.chromium.org/3598004/show git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61185 0039d316-1c4b-4281-b951-d872f2087c98
* Resubmit - Roll DEPS to include changes to FFmpeg build systemhclam@chromium.org2010-10-011-2/+4
| | | | | | | | | | | TBR=fbarchrd BUG=57266 TEST=Tree is green. ARM ffmpeg can be loaded. Review URL: http://codereview.chromium.org/3555004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61134 0039d316-1c4b-4281-b951-d872f2087c98
* VP8 decoder for chromotinghclam@chromium.org2010-09-295-3/+248
| | | | | | | | | | | Added DecoderVp8 and unit test for chromoting. TEST=remoting_unittests BUG=50235 Review URL: http://codereview.chromium.org/3032047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60960 0039d316-1c4b-4281-b951-d872f2087c98
* Implement VP8 encoder for chromotinghclam@chromium.org2010-09-287-169/+173
| | | | | | | | | | | Added EncoderVp8 with test for chromoting. TEST=remoting_unittests BUG=50235 Review URL: http://codereview.chromium.org/3005036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60854 0039d316-1c4b-4281-b951-d872f2087c98
* Implemented basic support Chromotocol connection.sergeyu@chromium.org2010-09-2817-47/+1771
| | | | | | | | | | New code supports multiple PseudoTCP and UDP channels. Client and host still use old JjngleChannel for connection. BUG=53986 TEST=None Review URL: http://codereview.chromium.org/3319021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60842 0039d316-1c4b-4281-b951-d872f2087c98
* Fix memory problem in DecoderVerbatimhclam@chromium.org2010-09-282-3/+15
| | | | | | | | | | | | Change from c_str() to data() also did some boundary checks to make there's no memory error. BUG=None TEST=None Review URL: http://codereview.chromium.org/3388024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60746 0039d316-1c4b-4281-b951-d872f2087c98
* Add in a new FrameConsumer interface, Decode API, and a ↵ajwong@chromium.org2010-09-276-1/+365
| | | | | | | | | | | | | | | RectangleUpdateDecoder abstraction. This should allow a decoder that can still request the view to allocate frames without being owned by the view itself. This allows for cleaner threading semantics and reduced coupling of classes. The new decoder API keeps the decoder from being aware of the network packet types tightening up the API layering. BUG=52833 TEST=None. This code isn't used yet. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=60703 Review URL: http://codereview.chromium.org/3335012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60721 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Add in a new FrameConsumer interface, Decode API, and a ↵ajwong@chromium.org2010-09-276-364/+1
| | | | | | | | | | RectangleUpdateDecoder abstraction." Reverting r60703 due to build break. TBR=willchan git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60705 0039d316-1c4b-4281-b951-d872f2087c98
* Add in a new FrameConsumer interface, Decode API, and a ↵ajwong@chromium.org2010-09-276-1/+364
| | | | | | | | | | | | | RectangleUpdateDecoder abstraction. This should allow a decoder that can still request the view to allocate frames without being owned by the view itself. This allows for cleaner threading semantics and reduced coupling of classes. The new decoder API keeps the decoder from being aware of the network packet types tightening up the API layering. BUG=52833 TEST=None. This code isn't used yet. Review URL: http://codereview.chromium.org/3335012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60703 0039d316-1c4b-4281-b951-d872f2087c98
* Fix mouse event handling in Chromoting plugin.garykac@chromium.org2010-09-243-3/+4
| | | | | | | | | | | | | | | Revision 60257 changed the Pepper PluginInstance to call HandleInputEvent instead of HandleEvent. Since HandleEvent was virtual in the subclass, this was not caught by the compiler. Here's where a proper 'override' keyword in C++ would've been nice. BUG=none TEST=verify mouse works Review URL: http://codereview.chromium.org/3440026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60467 0039d316-1c4b-4281-b951-d872f2087c98
* Modify Chromotocol to have a new RectangleUpdatePacket.ajwong@chromium.org2010-09-231-0/+50
| | | | | | | | | | | | This is a less nested version of UpdateStreamMessage. It will replace after a few more CLs are checked in. BUG=52833 TEST=none. adding new messages doesn't change functionality. Review URL: http://codereview.chromium.org/3352013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60360 0039d316-1c4b-4281-b951-d872f2087c98
* Pull latest PPAPI. Change key handling to support the new API.brettw@chromium.org2010-09-234-27/+33
| | | | | | | | TEST=PPAPI UI tests BUG=none Review URL: http://codereview.chromium.org/3386019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60258 0039d316-1c4b-4281-b951-d872f2087c98
* Add libsrtp in DEPS, gyp file for libsrtp.sergeyu@chromium.org2010-09-221-0/+1
| | | | | | | | | BUG=53986 TEST=none Review URL: http://codereview.chromium.org/3382013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60213 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 60142 - Add libsrtp in DEPS, gyp file for libsrtp.sergeyu@chromium.org2010-09-221-1/+0
| | | | | | | | | | | | BUG=53986 TEST=none Review URL: http://codereview.chromium.org/3382013 TBR=sergeyu@chromium.org Review URL: http://codereview.chromium.org/3381016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60144 0039d316-1c4b-4281-b951-d872f2087c98
* Add libsrtp in DEPS, gyp file for libsrtp.sergeyu@chromium.org2010-09-221-0/+1
| | | | | | | | | BUG=53986 TEST=none Review URL: http://codereview.chromium.org/3382013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60142 0039d316-1c4b-4281-b951-d872f2087c98
* Removed use of XmppSocketAdapter by sync.akalin@chromium.org2010-09-109-2/+1115
| | | | | | | | | | | | | | | | Moved XmppSocketAdapter and friends to remoting directory. Removed some dead code in jingle/. This is a relanding of 59012, which was mistakenly reverted at 59104. BUG=54146,55116 TEST=none TBR=willchan Review URL: http://codereview.chromium.org/3300030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59134 0039d316-1c4b-4281-b951-d872f2087c98
* Revert r59012 which started using Chrome sockets for sync.willchan@chromium.org2010-09-109-1115/+2
| | | | | | | | | | | This caused us to write to ChromeNetLog from the sync thread. ChromeNetLog is not thread safe, so this causes problems. BUG=55116,54146 TEST=Start up chrome. Does not repeatedly hit DCHECKs on ChromeNetLog::AddEntry(). Review URL: http://codereview.chromium.org/3358028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59104 0039d316-1c4b-4281-b951-d872f2087c98
* Removed use of XmppSocketAdapter by sync.akalin@chromium.org2010-09-099-2/+1115
| | | | | | | | | | | | | Moved XmppSocketAdapter and friends to remoting directory. Removed some dead code in jingle/. BUG=54146 TEST=none Review URL: http://codereview.chromium.org/3298021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59012 0039d316-1c4b-4281-b951-d872f2087c98
* Improve UI of Chromoting client UI to select available hosts.garykac@chromium.org2010-09-089-139/+355
| | | | | | | | | BUG=53988 TEST=none Review URL: http://codereview.chromium.org/3338014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58862 0039d316-1c4b-4281-b951-d872f2087c98
* Delayed event support in JingleThread.sergeyu@chromium.org2010-09-072-23/+86
| | | | | | | | | BUG=54347 TEST=unittests Review URL: http://codereview.chromium.org/3333014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58773 0039d316-1c4b-4281-b951-d872f2087c98
* Add mouse/keyboard event support to Chromoting client (Pepper and X11).garykac@chromium.org2010-09-077-6/+49
| | | | | | | | | BUG=50247 TEST=none Review URL: http://codereview.chromium.org/3341005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58761 0039d316-1c4b-4281-b951-d872f2087c98
* Fix manifest permissions in Chromoting Client UIgarykac@chromium.org2010-09-031-1/+1
| | | | | | | | | | | | Add googleapis.com to the manifest so that we can access the Chromoting Directory. BUG=none TEST=none Review URL: http://codereview.chromium.org/3356003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58497 0039d316-1c4b-4281-b951-d872f2087c98
* Fix valgrind memory leak in session_manager_unittest.garykac@chromium.org2010-09-031-10/+1
| | | | | | | | | BUG=46678,53277 TEST=remove 2 suppressions, run valgrind linux Review URL: http://codereview.chromium.org/3170049 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58496 0039d316-1c4b-4281-b951-d872f2087c98
* Update gettoken.py to generate chromoting directory token.garykac@chromium.org2010-09-031-12/+32
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3325005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58495 0039d316-1c4b-4281-b951-d872f2087c98
* Allows chromoting host to be connected again after connection drops.hclam@chromium.org2010-09-032-12/+10
| | | | | | | | | | | | | | | | | | | BUG=52890 TEST=Connects to chromoting disconnect, and then try again This allows basic reconnect functions of the chromoting host. We still have the following problems: 1. We don't have a disconnect command. 2. If we are waiting for client connection to time out we don't allow second connection. 3. Need to force a full frame update after a new client is connected. I am going to solve the above problems in separate patches but this patch will allow basic reconnect to work. Review URL: http://codereview.chromium.org/3320003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58441 0039d316-1c4b-4281-b951-d872f2087c98
* Re-lands 58186:bryeung@chromium.org2010-09-032-167/+168
| | | | | | | | | | | | | | | | | | | | Original message: """ Move the keyboard files from base/ to app/. Also moves the associated classes/enums from base:: to app::. TEST=try bots compile BUG=NONE Review URL: http://codereview.chromium.org/3165064 """ TBR=phajdan.jr@chromium.org Review URL: http://codereview.chromium.org/3354005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58438 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 58388 - Revert 58215 - Revert 58186 - Move the keyboard files from ↵akalin@chromium.org2010-09-022-168/+167
| | | | | | | | | | | | | | | | | | | | | base/ to app/. Also moves the associated classes/enums from base:: to app::. TEST=try bots compile BUG=NONE Review URL: http://codereview.chromium.org/3165064 TBR=bryeung@chromium.org TBR=phajdan.jr@chromium.org Review URL: http://codereview.chromium.org/3361003 TBR=bryeung@chromium.org Review URL: http://codereview.chromium.org/3337006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58390 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 58215 - Revert 58186 - Move the keyboard files from base/ to app/.bryeung@chromium.org2010-09-022-167/+168
| | | | | | | | | | | | | | | | Also moves the associated classes/enums from base:: to app::. TEST=try bots compile BUG=NONE Review URL: http://codereview.chromium.org/3165064 TBR=bryeung@chromium.org TBR=phajdan.jr@chromium.org Review URL: http://codereview.chromium.org/3361003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58388 0039d316-1c4b-4281-b951-d872f2087c98
* Unrevert 58354, reland PPAPI with the instance test disabled.brettw@chromium.org2010-09-021-1/+1
| | | | | | Review URL: http://codereview.chromium.org/3323005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58358 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 58328 - Pull new PPAPI, reland 58319 and 58321.rohitrao@chromium.org2010-09-021-1/+1
| | | | | | | | | | TEST=unit test included BUG=none TBR=brettw@chromium.org Review URL: http://codereview.chromium.org/3358006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58354 0039d316-1c4b-4281-b951-d872f2087c98
* Pull new PPAPI, reland 58319 and 58321.brettw@chromium.org2010-09-021-1/+1
| | | | | | | TEST=unit test included BUG=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58328 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 58319 - Pull latest version of Pepper. This includes a reworked find ↵brettw@chromium.org2010-09-021-1/+1
| | | | | | | | | | | | interface. TEST=none BUG=none TBR=brettw@chromium.org Review URL: http://codereview.chromium.org/3301009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58323 0039d316-1c4b-4281-b951-d872f2087c98
* Pull latest version of Pepper. This includes a reworked find interface.brettw@chromium.org2010-09-021-1/+1
| | | | | | | TEST=none BUG=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58319 0039d316-1c4b-4281-b951-d872f2087c98
* Basic user access check for chromoting host.sergeyu@chromium.org2010-09-0211-13/+237
| | | | | | | | | BUG=53984 TEST=unittests Review URL: http://codereview.chromium.org/3303001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58301 0039d316-1c4b-4281-b951-d872f2087c98
* Fixed tsan warnings in remoting_unittestssergeyu@chromium.org2010-09-011-0/+3
| | | | | | | | | BUG=53766,53837 TEST=remoting_unittests under tsan Review URL: http://codereview.chromium.org/3269005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58235 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 58186 - Move the keyboard files from base/ to app/.phajdan.jr@chromium.org2010-09-012-168/+167
| | | | | | | | | | | | | Also moves the associated classes/enums from base:: to app::. TEST=try bots compile BUG=NONE Review URL: http://codereview.chromium.org/3165064 TBR=bryeung@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58215 0039d316-1c4b-4281-b951-d872f2087c98
* Move the keyboard files from base/ to app/.bryeung@chromium.org2010-09-012-167/+168
| | | | | | | | | | | Also moves the associated classes/enums from base:: to app::. TEST=try bots compile BUG=NONE Review URL: http://codereview.chromium.org/3165064 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58186 0039d316-1c4b-4281-b951-d872f2087c98