summaryrefslogtreecommitdiffstats
path: root/remoting/base
Commit message (Collapse)AuthorAgeFilesLines
* iwyu: Include callback_old.h where appropriate, final.jhawkins@chromium.org2011-05-111-1/+1
| | | | | | | | | BUG=82098 TEST=none Review URL: http://codereview.chromium.org/7003003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@85003 0039d316-1c4b-4281-b951-d872f2087c98
* iwyu: Use callback_old.h where appropriate, part 1.jhawkins@chromium.org2011-05-101-1/+1
| | | | | | | | | | | BUG=none TEST=none R=thakis@chromium.org Review URL: http://codereview.chromium.org/6985009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84725 0039d316-1c4b-4281-b951-d872f2087c98
* Fix memory leak in EncoderVp8, and add unit-test for it.lambroslambrou@chromium.org2011-05-093-1/+49
| | | | | | | | | | | Leak occurred when changing host's screen resolution. BUG=None TEST=Valgrind bots stay green Review URL: http://codereview.chromium.org/6951010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84633 0039d316-1c4b-4281-b951-d872f2087c98
* Chromoting to report roundtrip latencyhclam@chromium.org2011-05-063-0/+12
| | | | | | | | | | | | | | | | | Doing so by sending a sequence number, essentially the timestamp in every envet message. Capturer at the host will pick up the latest sequence number and pass it through the pipeline. Client will then receive it and determine the latency. This roundtrip latency number however doesn't include time in decoding and rendering. BUG=None TEST=None Review URL: http://codereview.chromium.org/6792038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84504 0039d316-1c4b-4281-b951-d872f2087c98
* Report capture and encode time for chromotinghclam@google.com2011-04-054-1/+14
| | | | | | | | | | | Add hooks to record these numbers and report them in the client. BUG=None TEST=None Review URL: http://codereview.chromium.org/6767009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80516 0039d316-1c4b-4281-b951-d872f2087c98
* Measure bandwidth for chromoting video channelhclam@google.com2011-04-054-0/+206
| | | | | | | | | | | | | | | Define RunningAverage, TimedRunningAverage and use that to record video bandwidth. This doesn't account for overhead of protobuf envelop. However the number should be small that can be ignored. BUG=None TEST=None Review URL: http://codereview.chromium.org/6736009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80486 0039d316-1c4b-4281-b951-d872f2087c98
* Move some files from base to base/memory.levin@chromium.org2011-03-2810-22/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | raw_scoped_refptr_mismatch_checker.h ref_counted.cc ref_counted.h ref_counted_memory.cc ref_counted_memory.h ref_counted_unittest.cc scoped_callback_factory.h scoped_comptr_win.h scoped_handle.h scoped_native_library.cc scoped_native_library.h scoped_native_library_unittest.cc scoped_nsobject.h scoped_open_process.h scoped_ptr.h scoped_ptr_unittest.cc scoped_temp_dir.cc scoped_temp_dir.h scoped_temp_dir_unittest.cc scoped_vector.h singleton.h singleton_objc.h singleton_unittest.cc linked_ptr.h linked_ptr_unittest.cc weak_ptr.cc weak_ptr.h weak_ptr_unittest.cc BUG=None TEST=Compile Review URL: http://codereview.chromium.org/6714032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79524 0039d316-1c4b-4281-b951-d872f2087c98
* fix commentsdilmah@chromium.org2011-03-221-1/+1
| | | | | | | | | | BUG=None TEST=None TBR=rvargas@chromium.org Review URL: http://codereview.chromium.org/6715024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@78977 0039d316-1c4b-4281-b951-d872f2087c98
* Improve AlignRect() readability, and handle bad rectangles more gracefully ↵lambroslambrou@chromium.org2011-03-173-19/+60
| | | | | | | | | | | | | (without crashing in gfx::Rect ctor). Also fix cpplint nit. BUG=74809 TEST=Follow repro steps of bug. Review URL: http://codereview.chromium.org/6628051 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@78541 0039d316-1c4b-4281-b951-d872f2087c98
* Tidy up after CL 6573005.simonmorris@chromium.org2011-03-107-59/+41
| | | | | | | | | | | | Replace width/height pairs with gfx::Size objects. Avoid overloading Capturer::InvalidateFullScreen. BUG=none TEST=none Review URL: http://codereview.chromium.org/6635039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77625 0039d316-1c4b-4281-b951-d872f2087c98
* Change other usages of .size() to .empty() when applicable.erg@google.com2011-03-042-5/+5
| | | | | | | | | BUG=carnitas TEST=compiles Review URL: http://codereview.chromium.org/6609008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76962 0039d316-1c4b-4281-b951-d872f2087c98
* Let the host change resolution.simonmorris@chromium.org2011-03-045-30/+57
| | | | | | | | | | | | | | | | The screen size flows through the video pipeline, instead of being set statically when that pipeline is constructed. Only the Windows host actually detects when the screen size has changed. BUG=72469 TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=76747 Review URL: http://codereview.chromium.org/6573005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76908 0039d316-1c4b-4281-b951-d872f2087c98
* Linux: roll ffmpeg and vpx so that we can build with system vpxphajdan.jr@chromium.org2011-03-032-6/+2
| | | | | | | | | | | | | | Also change remoting files to use libvpx shim header. This is upstreaming Gentoo Linux changes. TBR=scherkus TEST=none BUG=none Review URL: http://codereview.chromium.org/6588123 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76757 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 76747 - Let the host change resolution.simonmorris@chromium.org2011-03-035-57/+30
| | | | | | | | | | | | | | | | | The screen size flows through the video pipeline, instead of being set statically when that pipeline is constructed. Only the Windows host actually detects when the screen size has changed. BUG=72469 TEST=none Review URL: http://codereview.chromium.org/6573005 TBR=simonmorris@chromium.org Review URL: http://codereview.chromium.org/6610018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76748 0039d316-1c4b-4281-b951-d872f2087c98
* Let the host change resolution.simonmorris@chromium.org2011-03-035-30/+57
| | | | | | | | | | | | | | The screen size flows through the video pipeline, instead of being set statically when that pipeline is constructed. Only the Windows host actually detects when the screen size has changed. BUG=72469 TEST=none Review URL: http://codereview.chromium.org/6573005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76747 0039d316-1c4b-4281-b951-d872f2087c98
* Pure pedantry: Replace all ".size() == 0" with ".empty()".erg@google.com2011-03-021-1/+1
| | | | | | | | | BUG=carnitas TEST=compiles; existing unit tests. Review URL: http://codereview.chromium.org/6602049 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76467 0039d316-1c4b-4281-b951-d872f2087c98
* Change chromoting images to upright by defaulthclam@chromium.org2011-02-225-37/+2
| | | | | | | | | | | Remove all the ugly code to have reverse rows. All images are upright now. BUG=71872 TEST=Everything in chromoting still works and upright. Review URL: http://codereview.chromium.org/6546057 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75609 0039d316-1c4b-4281-b951-d872f2087c98
* Speed up encoding by using active maphclam@chromium.org2011-02-222-4/+64
| | | | | | | | | | | | | | | Using active map can greatly reduce the amount of macro blocks need to be encoded by vp8. This brings average encoding time from 35ms per frame to about 8ms on the tested system. However this change depends on an updated version of libvpx. BUG=None TEST=chromoting still works & no visual problems Review URL: http://codereview.chromium.org/6518011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75586 0039d316-1c4b-4281-b951-d872f2087c98
* Don't call the zlib functions with the MOZ_Z_ prefixwtc@chromium.org2011-02-152-14/+0
| | | | | | | | | | | in the source code. R=agl,hclam,jar BUG=none TEST=No build or test failures Review URL: http://codereview.chromium.org/6529006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74975 0039d316-1c4b-4281-b951-d872f2087c98
* Fix memory leak in chromoting codehclam@chromium.org2011-02-091-1/+1
| | | | | | | | | | | Fixing leak in DecompressorZlib and removed suppression. BUG=57910, 57799 TEST=valgrind bot is green Review URL: http://codereview.chromium.org/6458009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74256 0039d316-1c4b-4281-b951-d872f2087c98
* Another big out-of-lining of test code. Hits a lot of gmock objectserg@google.com2011-02-083-5/+19
| | | | | | | | | BUG=none TEST=compiles Review URL: http://codereview.chromium.org/6413036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74168 0039d316-1c4b-4281-b951-d872f2087c98
* gfx -> ui/gfx move to make check_deps happy.thestig@chromium.org2011-02-051-1/+1
| | | | | | | | BUG=none TEST=tree goes green Review URL: http://codereview.chromium.org/6413006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73899 0039d316-1c4b-4281-b951-d872f2087c98
* Change includes of gfx/* to ui/gfx/*sail@chromium.org2011-02-056-6/+6
| | | | | | | | | BUG=71063 TEST=compiled Review URL: http://codereview.chromium.org/6312156 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73890 0039d316-1c4b-4281-b951-d872f2087c98
* Lower image quality for chromoting to improve encode speed and compression ratiohclam@chromium.org2011-02-041-2/+2
| | | | | | | | | | | | | | We were using very high quality for images, giving high bandwidth consumption and more bits to encode. TBR=ajwong@chromium.org BUG=None TEST=None Review URL: http://codereview.chromium.org/6286114 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73811 0039d316-1c4b-4281-b951-d872f2087c98
* Speed up chromoting encoding and decoding path for VP8hclam@chromium.org2011-02-046-28/+165
| | | | | | | | | | | | | | Several enhancements in this patch: 1. Encoder reports updated rects 2. VP8 decoder only performs YUV conversion on updated rects 3. Painting only updated rects in pepper plugin BUG=71253 TEST=Use chromoting to watch video Review URL: http://codereview.chromium.org/6368070 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73737 0039d316-1c4b-4281-b951-d872f2087c98
* Chromoting host to convert only changed regionshclam@chromium.org2011-02-031-35/+48
| | | | | | | | | | | | This significantly reduces the number of pixels we need to perform color space conversion. BUG=71253 TEST=None Review URL: http://codereview.chromium.org/6334040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73553 0039d316-1c4b-4281-b951-d872f2087c98
* Changed MessageReader so that it doesn't read from the socket if there aresergeyu@chromium.org2011-01-221-0/+2
| | | | | | | | | | | 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
* Remove obsolete base/lock.h and fix up callers to use the new header file andbrettw@chromium.org2011-01-211-2/+2
| | | | | | | | | | | 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
* 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-011-1/+1
| | | | | | | | 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
* 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
* 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
* Revert 68932 - Make members of Singleton<T> private and only visible to the ↵satish@chromium.org2010-12-111-10/+4
| | | | | | | | | | | | | | | | | | singleton type. 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 TBR=satish@chromium.org Review URL: http://codereview.chromium.org/5721005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68936 0039d316-1c4b-4281-b951-d872f2087c98
* Make members of Singleton<T> private and only visible to the singleton type. ↵satish@chromium.org2010-12-111-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@68932 0039d316-1c4b-4281-b951-d872f2087c98
* Simplified frame rate control in the chromoting host.sergeyu@chromium.org2010-12-096-11/+47
| | | | | | | | | | | | | | | | Insted of keeping semi-fixed frame rate, now capturing rate is controlled by how fast we can send data to the client. Capturing of frame n is started only after frame n-2 is sent (while n-1 is being encoded). This guarantees that we don't clog the video channel buffers, and that we start capturing only if we know that the frame will not need to wait for too long in the buffer. TEST=None BUG=None Review URL: http://codereview.chromium.org/5634002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68688 0039d316-1c4b-4281-b951-d872f2087c98
* Move more code from headers to implementation.erg@google.com2010-12-072-0/+3
| | | | | | | | | BUG=none TEST=compiles Review URL: http://codereview.chromium.org/5624002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68534 0039d316-1c4b-4281-b951-d872f2087c98
* Reenable encoder/decoder tests.sergeyu@chromium.org2010-12-076-140/+69
| | | | | | | | | BUG=57351 TEST=Unittests Review URL: http://codereview.chromium.org/5635003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68504 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 68434 - Reenable encoder/decoder tests.scherkus@chromium.org2010-12-076-69/+140
| | | | | | | | | | | | BUG=57351 TEST=Unittests Review URL: http://codereview.chromium.org/5635003 TBR=sergeyu@chromium.org Review URL: http://codereview.chromium.org/5581007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68437 0039d316-1c4b-4281-b951-d872f2087c98
* Reenable encoder/decoder tests.sergeyu@chromium.org2010-12-076-140/+69
| | | | | | | | | BUG=57351 TEST=Unittests Review URL: http://codereview.chromium.org/5635003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68434 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor ZLib and Verbatim encoders.sergeyu@chromium.org2010-12-0613-262/+197
| | | | | | | | | | | Added EncoderRowBased that implements encoding for both ZLib and Verbatim. BUG=None TEST=None Review URL: http://codereview.chromium.org/5382008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68394 0039d316-1c4b-4281-b951-d872f2087c98
* Fix RGB->YUV conversion: input is BGR instead of RGB.sergeyu@chromium.org2010-12-061-6/+6
| | | | | | | | | TEST=Colors are not screwed up BUG=None Review URL: http://codereview.chromium.org/5510003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68388 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 68384: Landed with incorrect commit message.sergeyu@chromium.org2010-12-061-6/+6
| | | | | | | | | | | | | | | | Refactor ZLib and Verbatim encoders. Added EncoderRowBased that implements encoding for both ZLib and Verbatim. BUG=None TEST=None Review URL: http://codereview.chromium.org/5382008 TBR=sergeyu@chromium.org Review URL: http://codereview.chromium.org/5586006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68385 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor ZLib and Verbatim encoders.sergeyu@chromium.org2010-12-061-6/+6
| | | | | | | | | | | Added EncoderRowBased that implements encoding for both ZLib and Verbatim. BUG=None TEST=None Review URL: http://codereview.chromium.org/5382008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68384 0039d316-1c4b-4281-b951-d872f2087c98
* Move move classes to the remoting::protocol namespace. Minor cleanups.sergeyu@chromium.org2010-12-021-2/+1
| | | | | | | | | BUG=None TEST=compiles, unittests Review URL: http://codereview.chromium.org/5068001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68056 0039d316-1c4b-4281-b951-d872f2087c98
* Reverse rows in VP8 decoder.sergeyu@chromium.org2010-12-022-5/+14
| | | | | | | | | TEST=None BUG=None Review URL: http://codereview.chromium.org/5310011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67918 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 67662 - FBTF: Remove unneeded headers from base/ (part 10)thestig@chromium.org2010-11-304-8/+3
| | | | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/5139006 TBR=thestig@chromium.org Review URL: http://codereview.chromium.org/5270010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67674 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Remove unneeded headers from base/ (part 10)thestig@chromium.org2010-11-304-3/+8
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/5139006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67662 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-02-09 20:48:36 +0000