summaryrefslogtreecommitdiffstats
path: root/remoting/client
Commit message (Collapse)AuthorAgeFilesLines
* Cleanup RectangleUpdateDecoder and VideoStubsergeyu@chromium.org2012-08-303-93/+16
| | | | | | | | | | Previously RectangleUpdateDecoder maintaned list of pending packets. That's not really necessary. Simplified it by queue tasks on the decode thread. Also some minor cleanups for the VideoStub interface - removed GetPendingVideoPackets() that wasn't used anywhere. Review URL: https://chromiumcodereview.appspot.com/10879085 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154272 0039d316-1c4b-4281-b951-d872f2087c98
* Fix UdpPacketSocket to handle callbacks after the socket is destroyed.sergeyu@chromium.org2012-08-291-6/+11
| | | | | | | | | | | | | PPB_UDPSocket_Private doesn't make guarantees about callbacks not being called after Close() - we may still get some callbacks asynchronously with error codes different from PP_ERROR_ABORTED. UdpPacketSocket wasn't handling this case properly. BUG=143839 Review URL: https://chromiumcodereview.appspot.com/10900028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153972 0039d316-1c4b-4281-b951-d872f2087c98
* Updating chrome to libjingle r175mallinath@chromium.org2012-08-261-2/+3
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10882037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153413 0039d316-1c4b-4281-b951-d872f2087c98
* Moved video stub implementation to RectangleUpdateDecoder.kxing@chromium.org2012-08-245-175/+206
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10867039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153305 0039d316-1c4b-4281-b951-d872f2087c98
* Renamed Decoder -> VideoDecoder.kxing@chromium.org2012-08-242-5/+5
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10873047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153140 0039d316-1c4b-4281-b951-d872f2087c98
* Moved the video encoders/decoders to the codec directory.kxing@chromium.org2012-08-232-4/+4
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10877014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152912 0039d316-1c4b-4281-b951-d872f2087c98
* Add a query function for the scaling feature.brettw@chromium.org2012-08-221-1/+1
| | | | | | | | | | | | | | | | We need to be able to query whether scaling is supported before creating a Graphics2D object so we know which code path to use. This just adds a function to do that checking. I also renamed Graphics2DDev to Graphics2D_Dev for consistency with the other Dev wrapper objects. TEST=none BUG=none Review URL: https://chromiumcodereview.appspot.com/10832428 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152829 0039d316-1c4b-4281-b951-d872f2087c98
* Added more error checking for audio packets.kxing@chromium.org2012-08-212-5/+7
| | | | | | | | | BUG= Review URL: https://chromiumcodereview.appspot.com/10823420 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152578 0039d316-1c4b-4281-b951-d872f2087c98
* Changed AudioPacket data to a repeated field.kxing@chromium.org2012-08-161-3/+10
| | | | | | | | | BUG= Review URL: https://chromiumcodereview.appspot.com/10827324 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151825 0039d316-1c4b-4281-b951-d872f2087c98
* Tidy up ChromotingInstance members a little.wez@chromium.org2012-08-152-45/+31
| | | | | | | | | | | | - Group member variables more appropriately. - Create the input pipeline as member objects rather than separate objects, to remove the need for pointer checks in various places. TEST=Existing unit-tests. Client plugin continues to work in manual tests. Review URL: https://chromiumcodereview.appspot.com/10831221 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151680 0039d316-1c4b-4281-b951-d872f2087c98
* Update JingleThreadWrapper to allow it to be created using task runner.sergeyu@chromium.org2012-08-141-1/+1
| | | | | | | | | | | | | Previously JingleThreadWrapper required a MessageLoop. In NaCl we don't have MessageLoop for the main plugin thread, but have a task runner. This CL allows to use JingleThreadWrapper on the main NaCl thread. BUG=134216 TBR=tommi@chromium.org Review URL: https://chromiumcodereview.appspot.com/10823224 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151458 0039d316-1c4b-4281-b951-d872f2087c98
* Piping for audio decoding.kxing@chromium.org2012-08-049-28/+154
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10843031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149990 0039d316-1c4b-4281-b951-d872f2087c98
* Replace PluginMessageLoopProxy with PluginThreadTaskRunner.sergeyu@chromium.org2012-07-273-18/+18
| | | | | | | | | | | | Previosly plugin code was using PluginMessageLoopProxy that implements MessageLoopProxy interface. We no longer need MessageLoopProxy. Replacing PluginMessageLoopProxy with PluginThreadTaskRunner that implements SingleThreadTaskRunner interface Review URL: https://chromiumcodereview.appspot.com/10830016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148801 0039d316-1c4b-4281-b951-d872f2087c98
* Coverity: Initialize member variables.jhawkins@chromium.org2012-07-251-0/+1
| | | | | | | | | | | | | | CID_COUNT=12 CID=104349,104361,104373,104374,104399,104420,104421,104422,104466,104476, 104532,104538 BUG=none TEST=none R=groby TBR=brettw,satorux,mnissler Review URL: https://chromiumcodereview.appspot.com/10833006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148430 0039d316-1c4b-4281-b951-d872f2087c98
* Support for both 44.1 kHz and 48 kHz on the client.kxing@chromium.org2012-07-244-33/+65
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10795066 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148203 0039d316-1c4b-4281-b951-d872f2087c98
* Make Chromoting client plugin always render at device DPI.wez@chromium.org2012-07-244-25/+51
| | | | | | | | | | | | | This addresses image quality issues when to a high-resolution device from a high-DPI but lower resolution client. In this case Chromoting would down-scale to the plugin's DIP dimensions, only for Chrome to up-scale to device dimensions. Follow-up CLs will add DPI information to the FrameConsumer interface, and try to optimize the plugin's up-scaling mode. BUG=133349,135089 Review URL: https://chromiumcodereview.appspot.com/10785041 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148012 0039d316-1c4b-4281-b951-d872f2087c98
* Create and destroy Pepper audio player on connect and disconnect.kxing@chromium.org2012-07-231-1/+2
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10800077 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147920 0039d316-1c4b-4281-b951-d872f2087c98
* Show connection state in the Chromoting client UI.sergeyu@chromium.org2012-07-235-0/+14
| | | | | | | | | | | | Now screen is desaturated when we get notification that video channel is not working. BUG=131411 Review URL: https://chromiumcodereview.appspot.com/10692179 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147830 0039d316-1c4b-4281-b951-d872f2087c98
* Rename ChromotingView to ClientUserInterface. Clean it up.sergeyu@chromium.org2012-07-237-251/+153
| | | | | | | | | | | | | ChromotingView wasn't really doing what it was initially meant to do, now FrameProducer and FrameConsumer interfaces are used to process video stream. Renamed and simplified that interface. Also moved implementation from PepperView to ChromotingInstance. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=147595 Review URL: https://chromiumcodereview.appspot.com/10703144 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147828 0039d316-1c4b-4281-b951-d872f2087c98
* Propagate DPI information to web-app.jamiewalch@chromium.org2012-07-209-19/+49
| | | | | | | | | | BUG=135089 TEST= Review URL: https://chromiumcodereview.appspot.com/10801003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147605 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 147595 - Rename ChromotingView to ClientUserInterface. Clean it up.sergeyu@chromium.org2012-07-207-148/+251
| | | | | | | | | | | | | | ChromotingView wasn't really doing what it was initially meant to do, now FrameProducer and FrameConsumer interfaces are used to process video stream. Renamed and simplified that interface. Also moved implementation from PepperView to ChromotingInstance. Review URL: https://chromiumcodereview.appspot.com/10703144 TBR=sergeyu@chromium.org Review URL: https://chromiumcodereview.appspot.com/10800048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147596 0039d316-1c4b-4281-b951-d872f2087c98
* Rename ChromotingView to ClientUserInterface. Clean it up.sergeyu@chromium.org2012-07-207-251/+148
| | | | | | | | | | | ChromotingView wasn't really doing what it was initially meant to do, now FrameProducer and FrameConsumer interfaces are used to process video stream. Renamed and simplified that interface. Also moved implementation from PepperView to ChromotingInstance. Review URL: https://chromiumcodereview.appspot.com/10703144 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147595 0039d316-1c4b-4281-b951-d872f2087c98
* Fix pepper crash when destroying client.sergeyu@chromium.org2012-07-181-3/+8
| | | | | | | | | | | After crrev.com/145781 the UDPPacketSocket wasn't handling PP_ERROR_ABORTED properly and this may cause crash when disconnecting. BUG=137632 Review URL: https://chromiumcodereview.appspot.com/10806020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147336 0039d316-1c4b-4281-b951-d872f2087c98
* Rolling libjingle to 159.ronghuawu@chromium.org2012-07-142-2/+6
| | | | | | | | TEST=unit tests, webrtc pytest Review URL: https://chromiumcodereview.appspot.com/10689180 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146709 0039d316-1c4b-4281-b951-d872f2087c98
* Ignore all sendto() errors for P2P sockets on the client side.sergeyu@chromium.org2012-07-091-2/+18
| | | | | | | | | | This is a workaround for the linked bug. BUG=135210 Review URL: https://chromiumcodereview.appspot.com/10752005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145781 0039d316-1c4b-4281-b951-d872f2087c98
* Client code for playing back audiokxing@chromium.org2012-07-027-4/+238
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10699030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145217 0039d316-1c4b-4281-b951-d872f2087c98
* Roll libjingle 153:157sergeyu@chromium.org2012-06-291-1/+6
| | | | | | | | | | Among other things this roll fixes relay server support (bug 134007). BUG=134007 Review URL: https://chromiumcodereview.appspot.com/10705008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144860 0039d316-1c4b-4281-b951-d872f2087c98
* Renamed VideoStub::GetPendingPackets() to VideoStub::GetPendingVideoPackets()kxing@chromium.org2012-06-262-2/+2
| | | | | | | | | BUG=133823 Review URL: https://chromiumcodereview.appspot.com/10662021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144236 0039d316-1c4b-4281-b951-d872f2087c98
* RefCounted types should not have public destructors, delegate cleanuprsleevi@chromium.org2012-06-262-4/+7
| | | | | | | | | | | | | | | | | | | | | | | | For Delegate/Observer-type classes that specify an interface but do not have any particular lifetime requirements, make their destructors protected. This is to allow their interfaces to be implemented safely by RefCounted types. With public destructors, it's possible to do "scoped_ptr<Delegate> foo", and then assign a RefCountedDelegateImpl, which would lead to a double free. As none of these Delegates actually need public destructors (ownership of the Delegate* is not transferred during a function call / class constructor), mark the destructors protected so that it becomes a compile warning to try to delete them via the Delegate*. BUG=123295 TEST=it compiles Review URL: https://chromiumcodereview.appspot.com/10383262 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144086 0039d316-1c4b-4281-b951-d872f2087c98
* Added piping for sending audio packets from host to client.kxing@chromium.org2012-06-252-2/+15
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10532211 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143977 0039d316-1c4b-4281-b951-d872f2087c98
* Add an explicit function to init NSS for SSL server socketsrsleevi@chromium.org2012-06-131-0/+8
| | | | | | | | | | 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
* 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
* Replace ScopedThreadProxy with MessageLoopProxy & WeakPtrs.wez@chromium.org2012-06-074-45/+51
| | | | | | | | | | | | | | | | | | | 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
* [Chromoting] Initial plumbing for cursor shape.garykac@chromium.org2012-06-027-1/+75
| | | | | | | | | | | | | | | | | This cl contains: * protocol for sending cursor shape on control channel from host to client * cross-platform (Pepper) client code for rendering host cursor * Linux host support for reading current cursor shape Separate CLs will follow with Mac and Windows host support. BUG=116229 TEST=none Review URL: https://chromiumcodereview.appspot.com/10382184 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@140205 0039d316-1c4b-4281-b951-d872f2087c98
* Remove old MIME type for client plugin, and cleanup unused include.wez@chromium.org2012-06-021-2/+0
| | | | | | | | | BUG=86353,112532 Review URL: https://chromiumcodereview.appspot.com/10478002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@140168 0039d316-1c4b-4281-b951-d872f2087c98
* Disable V1 auth support in remoting client and Me2Me host.sergeyu@chromium.org2012-05-314-31/+17
| | | | | | | | BUG=110483 Review URL: https://chromiumcodereview.appspot.com/10446084 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139815 0039d316-1c4b-4281-b951-d872f2087c98
* Remove legacy scriptable-object interface from client plugin.wez@chromium.org2012-05-314-742/+8
| | | | | | | | | | BUG=130335 TEST=Manual verification of Chromoting client functionality, and PyAuto tests. Review URL: https://chromiumcodereview.appspot.com/10454078 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139720 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 139623 - Replace ScopedThreadProxy with MessageLoopProxy & WeakPtrs.wez@chromium.org2012-05-304-43/+38
| | | | | | | | | | | | | | | | | | | | 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. TEST=Existing unit-tests, and manual testing. Review URL: https://chromiumcodereview.appspot.com/10454040 TBR=wez@chromium.org Review URL: https://chromiumcodereview.appspot.com/10446088 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139633 0039d316-1c4b-4281-b951-d872f2087c98
* Replace ScopedThreadProxy with MessageLoopProxy & WeakPtrs.wez@chromium.org2012-05-304-38/+43
| | | | | | | | | | | | | | | | | 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. TEST=Existing unit-tests, and manual testing. Review URL: https://chromiumcodereview.appspot.com/10454040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139623 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 139424 - Remove V1 authenticators.sergeyu@chromium.org2012-05-305-28/+51
| | | | | | | | | | | | | | | All currently used versions of chromoting should support V2 auth, so we no longer need V1 auth support. BUG=110483 Review URL: https://chromiumcodereview.appspot.com/10453064 TBR=sergeyu@chromium.org Review URL: https://chromiumcodereview.appspot.com/10453080 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139578 0039d316-1c4b-4281-b951-d872f2087c98
* Remove V1 authenticators.sergeyu@chromium.org2012-05-295-51/+28
| | | | | | | | | | | | All currently used versions of chromoting should support V2 auth, so we no longer need V1 auth support. BUG=110483 Review URL: https://chromiumcodereview.appspot.com/10453064 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139424 0039d316-1c4b-4281-b951-d872f2087c98
* [Chromoting] Make the client plugin tell the webapp when the first video ↵simonmorris@chromium.org2012-05-254-1/+15
| | | | | | | | | | | | | | buffer arrives. A follow-up CL will make the pyauto test wait until the first video buffer arrives before declaring that a connection has succeeded. BUG=129791 Review URL: https://chromiumcodereview.appspot.com/10444035 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139131 0039d316-1c4b-4281-b951-d872f2087c98
* Don't depend on MessageLoopProxy in remoting protocol code.sergeyu@chromium.org2012-05-241-2/+1
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10332304 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138731 0039d316-1c4b-4281-b951-d872f2087c98
* [Chromoting] Add a filter that will stop the host sending unnecessary ↵simonmorris@chromium.org2012-05-221-1/+0
| | | | | | | | | | | | clipboard events to the client. The host does not yet send any clipboard events to the client. A follow-up CL will make that happen. BUG=117473 Review URL: https://chromiumcodereview.appspot.com/10399052 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138377 0039d316-1c4b-4281-b951-d872f2087c98
* Changes needed to roll libjingle r141 to chrome.sergeyu@chromium.org2012-05-222-15/+19
| | | | | | | | | 1. Setting ice username and password for P2PTransportChannel. 2. Updating XMPP Auth cookie with Auth Token Review URL: https://chromiumcodereview.appspot.com/10382003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138334 0039d316-1c4b-4281-b951-d872f2087c98
* [chromoting] Fix sticky c key with cmd on mac using usb keycodehclam@chromium.org2012-05-223-230/+143
| | | | | | | | | | | Switch to use USB keycode for the fix for sticky keys on Mac. BUG=101937 TEST=Build on Mac and run chromoting and press cmd+c Review URL: https://chromiumcodereview.appspot.com/10387226 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138205 0039d316-1c4b-4281-b951-d872f2087c98
* Switch client to Pepper UDP API instead of Transport API.sergeyu@chromium.org2012-05-164-7/+29
| | | | | | | | BUG=109630 Review URL: https://chromiumcodereview.appspot.com/10284008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137487 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup: Remove unneeded scoped_ptr.h includes from ppapi, printing, ↵thestig@chromium.org2012-05-161-3/+1
| | | | | | | | | | | | remoting, and sync. BUG=none TEST=none TBR=brettw,hclam,akalin,abodenha Review URL: https://chromiumcodereview.appspot.com/10387107 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137376 0039d316-1c4b-4281-b951-d872f2087c98
* Fix PepperPortAllocator to resolve STUN addresses using HostResolverPrivate.sergeyu@chromium.org2012-05-165-90/+229
| | | | | | | | | | | Also the port allocator now disables TCP ports as it is supposed to. BUG=109630 Review URL: https://chromiumcodereview.appspot.com/10384171 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137355 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 136692 - Switch client to Pepper UDP API instead of Transport API.sergeyu@chromium.org2012-05-145-30/+7
| | | | | | | | | | | | BUG=109630 Review URL: https://chromiumcodereview.appspot.com/10284008 TBR=sergeyu@chromium.org Review URL: https://chromiumcodereview.appspot.com/10387113 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136908 0039d316-1c4b-4281-b951-d872f2087c98