summaryrefslogtreecommitdiffstats
path: root/jingle/glue/thread_wrapper_unittest.cc
Commit message (Collapse)AuthorAgeFilesLines
* Switch to standard integer types in jingle/.avi2015-12-221-2/+4
| | | | | | | | | BUG=138542 TBR=zea@chromium.org Review URL: https://codereview.chromium.org/1544473003 Cr-Commit-Position: refs/heads/master@{#366518}
* Remove base::MessageLoop::{Quit,QuitClosure} functionski.stfu2015-10-231-4/+2
| | | | | | | | | | | | | This patch removes aliases for base::MessageLoop::QuitWhenIdle & base::MessageLoop::QuitWhenIdleClosure, and includes minor formatting changes made by using git cl-format. BUG=131220 TEST= R=danakj@chromium.org,brettw@chromium.org CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1390513002 Cr-Commit-Position: refs/heads/master@{#355763}
* Standardize usage of virtual/override/final specifiers in jingle/.dcheng2014-12-221-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Google C++ style guide states: Explicitly annotate overrides of virtual functions or virtual destructors with an override or (less frequently) final specifier. Older (pre-C++11) code will use the virtual keyword as an inferior alternative annotation. For clarity, use exactly one of override, final, or virtual when declaring an override. To better conform to these guidelines, the following constructs have been rewritten: - if a base class has a virtual destructor, then: virtual ~Foo(); -> ~Foo() override; - virtual void Foo() override; -> void Foo() override; - virtual void Foo() override final; -> void Foo() final; This patch was automatically generated. The clang plugin can generate fixit hints, which are suggested edits when it is 100% sure it knows how to fix a problem. The hints from the clang plugin were applied to the source tree using the tool in https://codereview.chromium.org/598073004. Several formatting edits by clang-format were manually reverted, due to mangling of some of the more complicate IPC macros. BUG=417463 Review URL: https://codereview.chromium.org/822833002 Cr-Commit-Position: refs/heads/master@{#309482}
* Replacing the OVERRIDE with override and FINAL with final in jinglemohan.reddy2014-10-071-1/+1
| | | | | | | | | | | This step is a giant search and replace for OVERRIDE and FINAL to replace them with their lowercase versions. BUG=417463 Review URL: https://codereview.chromium.org/628123002 Cr-Commit-Position: refs/heads/master@{#298335}
* Update webrtc&libjingle 6774:6825.ronghuawu@chromium.org2014-08-061-23/+23
| | | | | | | | | | | | | BUG=N/A R=hellner@chromium.org TBR=darin@chromium.org, hclam@chromium.org, jochen@chromium.org, palmer@chromium.org, wez@chromium.org Commit on behalf of hellner@. Original CL tried and approved in https://codereview.chromium.org/429113002/ Review URL: https://codereview.chromium.org/450463002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@287788 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Update webrtc&libjingle 6774:6799."erikchen@chromium.org2014-07-311-23/+23
| | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit bf45dff0ca703e5ad3bb6c6754ebdcb0c1b1296a. (https://codereview.chromium.org/429253003) Broke sizes on Linux http://build.chromium.org/p/chromium/builders/Linux/builds/51786 https://code.google.com/p/webrtc/source/browse/trunk/webrtc/base/linux.cc defines static std::string lsb_release_string; among other things It's guarded in WEBRTC_CHROMIUM_BUILDs. Maybe it should be WEBRTC_CHROMIUM_BUILD. BUG=NONE TBR=darin@chromium.org, hclam@chromium.org, jochen@chromium.org, palmer@chromium.org, sergeyu@chromium.org, wez@chromium.org, ronghuawu@chromium.org NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/435823002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286914 0039d316-1c4b-4281-b951-d872f2087c98
* Update webrtc&libjingle 6774:6799.ronghuawu@chromium.org2014-07-311-23/+23
| | | | | | | | | | | | | BUG=N/A R=hellner@chromium.org TBR=darin@chromium.org, hclam@chromium.org, jochen@chromium.org, palmer@chromium.org, sergeyu@chromium.org, wez@chromium.org Commit on behalf of hellner@. Original CL tried and approved in https://codereview.chromium.org/429113002/ Review URL: https://codereview.chromium.org/429253003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286902 0039d316-1c4b-4281-b951-d872f2087c98
* Fixes for re-enabling more MSVC level 4 warnings: misc edition #2pkasting@chromium.org2014-07-181-1/+1
| | | | | | | | | | | | | | | | | | This contains fixes for the following sorts of issues: * Assignment inside conditional * Taking the address of a temporary * Octal escape sequence terminated by decimal number * Signedness mismatch * Possibly-uninitialized local variable This also contains a small number of cleanups to nearby code (e.g. no else after return). BUG=81439 TEST=none Review URL: https://codereview.chromium.org/382673002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283967 0039d316-1c4b-4281-b951-d872f2087c98
* Use a direct include of the message_loop header in dbus/, device/, ↵avi@chromium.org2013-07-171-1/+1
| | | | | | | | | | | | extensions/, google_apis/, gpu/, ipc/, jingle/. BUG=260807 TEST=none TBR=ben@chromium.org Review URL: https://chromiumcodereview.appspot.com/19607005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212157 0039d316-1c4b-4281-b951-d872f2087c98
* jingle: Use base::MessageLoop.xhwang@chromium.org2013-04-271-3/+5
| | | | | | | | | BUG=236029 R=ajwong@chromium.org Review URL: https://chromiumcodereview.appspot.com/14307021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@196960 0039d316-1c4b-4281-b951-d872f2087c98
* jingle: Update the calls from RunAllPending() to RunUntilIdle().tfarina@chromium.org2012-11-301-3/+3
| | | | | | | | | | | | RunAllPending() is deprecated and we should switch to RunUntilIdle(). BUG=131220 TBR=ajwong@chromium.org Review URL: https://chromiumcodereview.appspot.com/11421172 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170464 0039d316-1c4b-4281-b951-d872f2087c98
* Update JingleThreadWrapper to allow it to be created using task runner.sergeyu@chromium.org2012-08-141-2/+2
| | | | | | | | | | | | | 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
* Convert uses of int ms to TimeDelta in jingle and remoting.tedvessenes@gmail.com2012-03-051-5/+7
| | | | | | | | | | R=sergeyu@chromium.org BUG=108171 Review URL: http://codereview.chromium.org/9572038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@124915 0039d316-1c4b-4281-b951-d872f2087c98
* Removing MessageLoop::QuitTask() from jingle/dcheng@chromium.org2011-12-091-2/+2
| | | | | | | | | | | | base/task.h is going away and being replaced by base/bind.h and base/callback.h. This CL was automatically generated by sed. BUG=none TEST=none Review URL: http://codereview.chromium.org/8885018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113874 0039d316-1c4b-4281-b951-d872f2087c98
* Add OVERRIDE to jingle/.avi@chromium.org2011-11-171-0/+1
| | | | | | | | | BUG=104314 TEST=no change Review URL: http://codereview.chromium.org/8589003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110546 0039d316-1c4b-4281-b951-d872f2087c98
* Fix Dispose() method in JingleThreadWrapper.sergeyu@chromium.org2011-09-081-0/+28
| | | | | | | | | | BUG=93812 TEST=unittests, repro steps in the bug Review URL: http://codereview.chromium.org/7850007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100095 0039d316-1c4b-4281-b951-d872f2087c98
* Implement Send() in JingleThreadWrapper.sergeyu@chromium.org2011-08-011-50/+145
| | | | | | | | | | | | | Send() is used in some of libjingle code. WebRTC needs to create two separate threads which Send() messages to each other. This wasn't previously supported in JingleThreadWrapper. BUG=None TEST=Unittests Review URL: http://codereview.chromium.org/7520014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94992 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 94956 - Implement Send() in JingleThreadWrapper.sergeyu@chromium.org2011-08-011-145/+50
| | | | | | | | | | | | | | | | Send() is used in some of libjingle code. WebRTC needs to create two separate threads which Send() messages to each other. This wasn't previously supported in JingleThreadWrapper. BUG=None TEST=Unittests Review URL: http://codereview.chromium.org/7520014 TBR=sergeyu@chromium.org Review URL: http://codereview.chromium.org/7540030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94965 0039d316-1c4b-4281-b951-d872f2087c98
* Implement Send() in JingleThreadWrapper.sergeyu@chromium.org2011-08-011-50/+145
| | | | | | | | | | | | | Send() is used in some of libjingle code. WebRTC needs to create two separate threads which Send() messages to each other. This wasn't previously supported in JingleThreadWrapper. BUG=None TEST=Unittests Review URL: http://codereview.chromium.org/7520014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94956 0039d316-1c4b-4281-b951-d872f2087c98
* Fix stability issues with ThreadWrapperTest.sergeyu@chromium.org2011-07-141-33/+40
| | | | | | | | | BUG=86532 TEST=the test it not flaky anymore. Review URL: http://codereview.chromium.org/7327005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92470 0039d316-1c4b-4281-b951-d872f2087c98
* Mark ThreadWrapperTest.PostDelayed as DISABLED on Winakalin@chromium.org2011-06-171-1/+7
| | | | | | | | BUG=86532 TEST= TBR=sergeyu@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89503 0039d316-1c4b-4281-b951-d872f2087c98
* Implement P2P Transport Dev using P2PTransportImpl.sergeyu@chromium.org2011-04-121-6/+14
| | | | | | | | | BUG=None TEST=Unittests Review URL: http://codereview.chromium.org/6823021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81331 0039d316-1c4b-4281-b951-d872f2087c98
* P2P Transport implementation.sergeyu@chromium.org2011-04-061-15/+16
| | | | | | | | | TEST=Unittests. BUG=None Review URL: http://codereview.chromium.org/6791023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80717 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 80567 - P2P Transport implementation.TEST=Unittests.BUG=NoneReview ↵sergeyu@chromium.org2011-04-061-16/+15
| | | | | | | | | URL: http://codereview.chromium.org/6791023 TBR=sergeyu@chromium.org Review URL: http://codereview.chromium.org/6799003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80571 0039d316-1c4b-4281-b951-d872f2087c98
* P2P Transport implementation.sergeyu@chromium.org2011-04-061-15/+16
| | | | | | | | | TEST=Unittests. BUG=None Review URL: http://codereview.chromium.org/6791023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80567 0039d316-1c4b-4281-b951-d872f2087c98
* Add JingleThreadWrapper.sergeyu@chromium.org2011-03-281-0/+158
JingleThreadWrapper wraps chromium thread with a talk_base::Thread interface. BUG=None TEST=Unittests Review URL: http://codereview.chromium.org/6747017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79576 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2024-08-08 01:56:48 +0000