summaryrefslogtreecommitdiffstats
path: root/jingle
Commit message (Collapse)AuthorAgeFilesLines
* 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}
* Implement GetTotalReceivedBytes() for StreamSockettbansal2015-10-145-0/+17
| | | | | | | | | | | | | | | | | | GetTotalReceivedBytes() returns the number of bytes read from the network. Currently, only TCPClientSocket implements GetTotalReceivedBytes(). Tests are in ssl_client_socket_unittest.cc. Right now GetTotalReceivedBytes() is not plumbed to the higher layers. Once that's done, this will help us in precisely computing the network bytes used by a request. BUG=537754 Review URL: https://codereview.chromium.org/1343563002 Cr-Commit-Position: refs/heads/master@{#354092}
* Reland remove reference counting from HttpNetworkSession.mmenke2015-10-132-2/+2
| | | | | | | | | | | | | | | This was originally landed in https://crrev.com/468be2ff331c13b1a081d10a3c17e5366d26e577 and reverted in https://crrev.com/6b3af6e07d19ebb3f86df612ee5a08c8dc8f78ab, due to a new leak in a test causing bots to crash in a fun and exciting manner. BUG=515947 TBR=sgurun@chromium.org,davidben@chromium.org,droger@chromium.org,zea@chromium.org Review URL: https://codereview.chromium.org/1389743003 Cr-Commit-Position: refs/heads/master@{#353774}
* Update to use third_party/webrtc_overrides instead of ↵grunell2015-09-302-2/+3
| | | | | | | | | | | | | | third_party/webrtc/overrides. This is one step in moving the overrides files from WebRTC to Chromium. See bug comment #65 for all steps. Depends on https://codereview.chromium.org/1345873004/ and a WebRTC roll (specifically https://codereview.webrtc.org/1345313004/). BUG=468375 Review URL: https://codereview.chromium.org/1357913002 Cr-Commit-Position: refs/heads/master@{#351554}
* Use FRIEND_TEST_ALL_PREFIXES instead of FRIEND_TEST.thestig2015-09-251-7/+8
| | | | | | | | Also remove a custom FRIEND_TEST_ALL_PREFIXES #define from 6 years ago. Review URL: https://codereview.chromium.org/1370563002 Cr-Commit-Position: refs/heads/master@{#350719}
* base: Template methods on Timer classes instead of the classes themselves.danakj2015-09-241-1/+1
| | | | | | | | | | | | | | | | | | | The base class for OneShotTimer and DelayedTimer is templated but only the Start method needs to be (which has a TODO to make it go away entirely too). The DelayedTimer class is also templated but only its constructor needs to be, and the type can be inferred at the callsite, so less typing all around. R=thakis@chromium.org TBR=sky,brettw BUG=148832 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1355063004 Cr-Commit-Position: refs/heads/master@{#350496}
* Added some missing includes, removed one include and removed an undef.grunell2015-09-182-0/+2
| | | | | | | | | | | | | Found this when working on removing WebRTC's dependency on Chromium. These updates makes the WebRTC Chromium FYI bots green. BUG=468375 TBR=sergeyu@chromium.org,nick@chromium.org Review URL: https://codereview.chromium.org/1346883003 Cr-Commit-Position: refs/heads/master@{#349631}
* Compile remoting client plugin with GNsergeyu2015-09-141-0/+8
| | | | | | | | BUG=512899 Review URL: https://codereview.chromium.org/1312463005 Cr-Commit-Position: refs/heads/master@{#348752}
* Revert of Remove reference counting from HttpNetworkSession. (patchset #11 ↵mmenke2015-09-122-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | id:430001 of https://codereview.chromium.org/1298253002/ ) Reason for revert: Causing net tests to fail on a couple memory FYI bots with a stack that makes no sense, as there should be SpdySessionPool created until after SetUp has completed: [----------] 488 tests from NextProto/HttpNetworkTransactionTest [ RUN ] NextProto/HttpNetworkTransactionTest.Basic/0 [4832:3079:0911/175605:3116231085741:FATAL:weak_ptr.h(211)] Check failed: get() != NULL. 0 net_unittests 0x0000000100e4e4a3 base::debug::StackTrace::StackTrace() + 19 1 net_unittests 0x0000000100e4e4c9 base::debug::StackTrace::StackTrace() + 9 2 net_unittests 0x0000000100e66ad3 logging::LogMessage::~LogMessage() + 67 3 net_unittests 0x0000000100e662d9 logging::LogMessage::~LogMessage() + 9 4 net_unittests 0x00000001004a8d3a base::WeakPtr<net::HttpServerProperties>::operator->() const + 90 5 net_unittests 0x0000000101257267 net::SpdySessionPool::OnIPAddressChanged() + 439 6 net_unittests 0x00000001010422a7 _ZN4base20DispatchToMethodImplIN3net21NetworkChangeNotifier17IPAddressObserverEMS3_FvvEJEJEEEvPT_T0_RKNS_5TupleIJDpT1_EEENS_13IndexSequenceIJXspT2_EEEE + 23 7 net_unittests 0x0000000101042289 _ZN4base16DispatchToMethodIN3net21NetworkChangeNotifier17IPAddressObserverEMS3_FvvEJEEEvPT_T0_RKNS_5TupleIJDpT1_EEE + 9 8 net_unittests 0x00000001010420ea _ZNK4base8internal13UnboundMethodIN3net21NetworkChangeNotifier17IPAddressObserverEMS4_FvvENS_5TupleIJEEEE3RunEPS4_ + 26 9 net_unittests 0x0000000101041b3b _ZN4base22ObserverListThreadSafeIN3net21NetworkChangeNotifier17IPAddressObserverEE13NotifyWrapperIMS3_FvvENS_5TupleIJEEEEEvPNS4_19ObserverListContextERKNS_8internal13UnboundMethodIS3_T_T0_EE + 203 10 net_unittests 0x0000000101041fb5 _ZN4base8internal15RunnableAdapterIMNS_22ObserverListThreadSafeIN3net21NetworkChangeNotifier17IPAddressObserverEEEFvPNS6_19ObserverListContextERKNS0_13UnboundMethodIS5_MS5_FvvENS_5TupleIJEEEEEEE3RunEPS6_RKS8_SG_ + 69 11 net_unittests 0x0000000101041efb _ZN4base8internal12InvokeHelperILb0EvNS0_15RunnableAdapterIMNS_22ObserverListThreadSafeIN3net21NetworkChangeNotifier17IPAddressObserverEEEFvPNS7_19ObserverListContextERKNS0_13UnboundMethodIS6_MS6_FvvENS_5TupleIJEEEEEEEENS0_8TypeListIJRKPS7_RKS9_SH_EEEE8MakeItSoESK_SO_SQ_SH_ + 75 12 net_unittests 0x0000000101041e99 _ZN4base8internal7InvokerINS_13IndexSequenceIJLm0ELm1ELm2EEEENS0_9BindStateINS0_15RunnableAdapterIMNS_22ObserverListThreadSafeIN3net21NetworkChangeNotifier17IPAddressObserverEEEFvPNSA_19ObserverListContextERKNS0_13UnboundMethodIS9_MS9_FvvENS_5TupleIJEEEEEEEEFvPSA_SC_SK_ENS0_8TypeListIJSO_SC_SI_EEEEENSQ_IJNS0_12UnwrapTraitsISO_EENST_ISC_EENST_ISI_EEEEENS0_12InvokeHelperILb0EvSN_NSQ_IJRKSO_RKSC_SK_EEEEEFvvEE3RunEPNS0_13BindStateBaseE + 105 13 net_unittests 0x00000001000d31b4 base::Callback<void ()()>::Run() const + 20 14 net_unittests 0x0000000100e4eea5 base::debug::TaskAnnotator::RunTask(char const*, base::PendingTask const&) + 165 15 net_unittests 0x0000000100e751fa base::MessageLoop::RunTask(base::PendingTask const&) + 282 16 net_unittests 0x0000000100e7538c base::MessageLoop::DeferOrRunPendingTask(base::PendingTask const&) + 28 17 net_unittests 0x0000000100e75462 base::MessageLoop::DoWork() + 146 18 net_unittests 0x0000000100e3f42a base::MessagePumpLibevent::Run(base::MessagePump::Delegate*) + 106 19 net_unittests 0x0000000100e74f5e base::MessageLoop::RunHandler() + 158 20 net_unittests 0x0000000100e953b3 base::RunLoop::Run() + 51 21 net_unittests 0x0000000100e954bd base::RunLoop::RunUntilIdle() + 13 22 net_unittests 0x0000000100e74711 base::MessageLoop::RunUntilIdle() + 113 23 net_unittests 0x00000001004ad776 net::HttpNetworkTransactionTest::SetUp() + 22 24 net_unittests 0x0000000100f2d837 void testing::internal::HandleSehExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::*)(), char const*) + 23 25 net_unittests 0x0000000100f278e9 void testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::*)(), char const*) + 57 26 net_unittests 0x0000000100f20566 testing::Test::Run() + 70 27 net_unittests 0x0000000100f20ab3 testing::TestInfo::Run() + 163 28 net_unittests 0x0000000100f20e52 testing::TestCase::Run() + 178 29 net_unittests 0x0000000100f24733 testing::internal::UnitTestImpl::RunAllTests() + 499 30 net_unittests 0x0000000100f2e897 bool testing::internal::HandleSehExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool>(testing::internal::UnitTestImpl*, bool (testing::internal::UnitTestImpl::*)(), char const*) + 23 31 net_unittests 0x0000000100f28619 bool testing::internal::HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool>(testing::internal::UnitTestImpl*, bool (testing::internal::UnitTestImpl::*)(), char const*) + 57 32 net_unittests 0x0000000100f24523 testing::UnitTest::Run() + 115 33 net_unittests 0x0000000101649281 RUN_ALL_TESTS() + 17 34 net_unittests 0x0000000101648852 base::TestSuite::Run() + 98 35 net_unittests 0x0000000100c9509c base::internal::RunnableAdapter<int (base::TestSuite::*)()>::Run(base::TestSuite*) + 28 36 net_unittests 0x0000000100c95049 _ZN4base8internal12InvokeHelperILb0EiNS0_15RunnableAdapterIMNS_9TestSuiteEFivEEENS0_8TypeListIJP12NetTestSuiteEEEE8MakeItSoES6_S9_ + 41 37 net_unittests 0x0000000100c9500e _ZN4base8internal7InvokerINS_13IndexSequenceIJLm0EEEENS0_9BindStateINS0_15RunnableAdapterIMNS_9TestSuiteEFivEEEFiPS6_ENS0_8TypeListIJNS0_17UnretainedWrapperI12NetTestSuiteEEEEEEENSC_IJNS0_12UnwrapTraitsISF_EEEEENS0_12InvokeHelperILb0EiS9_NSC_IJPSE_EEEEEFivEE3RunEPNS0_13BindStateBaseE + 46 38 net_unittests 0x00000001010e4fb4 base::Callback<int ()()>::Run() const + 20 39 net_unittests 0x0000000101640c78 base::(anonymous namespace)::LaunchUnitTestsInternal(base::Callback<int ()()> const&, int, bool, base::Callback<void ()()> const&) + 264 40 net_unittests 0x0000000101640b53 base::LaunchUnitTests(int, char**, base::Callback<int ()()> const&) + 83 41 net_unittests 0x0000000100c94d6b main + 123 42 net_unittests 0x0000000100001674 start + 52 43 ??? 0x0000000000000005 0x0 + 5 Original issue's description: > Remove reference counting from HttpNetworkSession. > > This makes lifetime cleaner, and helps us avoid the case > where an HttpNetworkSession outlives the components it points > to. > > Also remove some weird uses a null NetLog in GCM. > > TBR=sgurun@chromium.org,davidben@chromium.org,droger@chromium.org > BUG=515947 > > Committed: https://crrev.com/468be2ff331c13b1a081d10a3c17e5366d26e577 > Cr-Commit-Position: refs/heads/master@{#348483} TBR=pauljensen@chromium.org,zea@chromium.org,sgurun@chromium.org,davidben@chromium.org,droger@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=515947 Review URL: https://codereview.chromium.org/1342613002 Cr-Commit-Position: refs/heads/master@{#348518}
* Remove reference counting from HttpNetworkSession.mmenke2015-09-112-2/+2
| | | | | | | | | | | | | | | This makes lifetime cleaner, and helps us avoid the case where an HttpNetworkSession outlives the components it points to. Also remove some weird uses a null NetLog in GCM. TBR=sgurun@chromium.org,davidben@chromium.org,droger@chromium.org BUG=515947 Review URL: https://codereview.chromium.org/1298253002 Cr-Commit-Position: refs/heads/master@{#348483}
* Fix sbox_validation_tests & Make isolate_driver.py include the executable itselfmaruel2015-09-091-1/+0
| | | | | | | | | | | | | | | | | | | | | | This is a redo of https://codereview.chromium.org/1245873002 c6649f2e014a24a909 with fix for Official build. Further reduce the boilerplate in .isolate files and is necessary to add debug symbols, to be done in a follow up. Slightly accelerate ninja file parsing by blacklisting more extensions. sbox_validation_tests wasn't run on Windows since https://codereview.chromium.org/1212003003/ (~2 months ago). Thankfully this CL uncovered this copy-paste error. TBR=jam@chromium.org BUG=98637 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1320793004 Cr-Commit-Position: refs/heads/master@{#347912}
* Revert of Fix sbox_validation_tests & Make isolate_driver.py include the ↵stgao2015-09-031-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | executable itself (patchset #3 id:90001 of https://codereview.chromium.org/1245873002/ ) Reason for revert: This CL seems to break compile http://build.chromium.org/p/chromium.chrome/builders/Google%20Chrome%20Linux%20x64/builds/3900/. If the revert doesn't fix it, I will re-land this CL. Original issue's description: > Fix sbox_validation_tests & Make isolate_driver.py include the executable itself > > Further reduce the boilerplate in .isolate files and is necessary to add debug > symbols, to be done in a follow up. > > Slightly accelerate ninja file parsing by blacklisting more extensions. > > sbox_validation_tests wasn't run on Windows since > https://codereview.chromium.org/1212003003/ (~2 months ago). Thankfully this CL > uncovered this copy-paste error. > > TBR=jam@chromium.org > BUG=98637 > CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel > > Committed: https://crrev.com/c6649f2e014a24a9093140a06fcadcaf2a06513c > Cr-Commit-Position: refs/heads/master@{#347027} TBR=jam@chromium.org,maruel@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=98637 Review URL: https://codereview.chromium.org/1329783003 Cr-Commit-Position: refs/heads/master@{#347071}
* Fix sbox_validation_tests & Make isolate_driver.py include the executable itselfmaruel2015-09-021-1/+0
| | | | | | | | | | | | | | | | | | | Further reduce the boilerplate in .isolate files and is necessary to add debug symbols, to be done in a follow up. Slightly accelerate ninja file parsing by blacklisting more extensions. sbox_validation_tests wasn't run on Windows since https://codereview.chromium.org/1212003003/ (~2 months ago). Thankfully this CL uncovered this copy-paste error. TBR=jam@chromium.org BUG=98637 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1245873002 Cr-Commit-Position: refs/heads/master@{#347027}
* Sync more GN unit tests with GYP.brettw2015-08-211-1/+4
| | | | | | | | | | | | | | | This adds about 600 more tests. Tests added: invalidation, navigation interception, feedback, search, sync driver, enhanced bookmarks, search engines, sessions, data reduction proxy, favicon, constrained window, autofill, metrics, keyed service, bookmarks, suggestions, audio modem I renamed ui_zoom to "zoom" according to GN style. I tweaked some jingle public dependencies so the right include directories get forwarded along with the corresponding uses in the headers. Reland of http://crrev.com/1301743003 TBR=dpranke@chromium.org Review URL: https://codereview.chromium.org/1304163005 Cr-Commit-Position: refs/heads/master@{#344874}
* Revert of Sync more GN unit tests with GYP. (patchset #3 id:40001 of ↵nasko2015-08-211-4/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/1301743003/ ) Reason for revert: This has broken the GN builds. Windows is failing compilation and Mac is failing generate_build_files. http://build.chromium.org/p/chromium.mac/builders/Mac%20GN/builds/15185/steps/generate_build_files/logs/stdio http://build.chromium.org/p/chromium.win/builders/Win%20x64%20GN/builds/5562/steps/compile/logs/stdio Original issue's description: > Sync more GN unit tests with GYP. > > This adds about 600 more tests. > > Tests added: invalidation, navigation interception, feedback, search, sync driver, enhanced bookmarks, search engines, sessions, data reduction proxy, favicon, constrained window, autofill, metrics, keyed service, bookmarks, suggestions, audio modem > > I renamed ui_zoom to "zoom" according to GN style. I tweaked some jingle public dependencies so the right include directories get forwarded along with the corresponding uses in the headers. > > BUG= > R=dpranke@chromium.org > > Committed: https://chromium.googlesource.com/chromium/src/+/a7cde4bd39dc814952d885e54eee33ffdac1a881 TBR=dpranke@chromium.org,thestig@chromium.org,msramek@chromium.org,brettw@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1306153002 Cr-Commit-Position: refs/heads/master@{#344760}
* Sync more GN unit tests with GYP.Brett Wilson2015-08-201-1/+4
| | | | | | | | | | | | | | | This adds about 600 more tests. Tests added: invalidation, navigation interception, feedback, search, sync driver, enhanced bookmarks, search engines, sessions, data reduction proxy, favicon, constrained window, autofill, metrics, keyed service, bookmarks, suggestions, audio modem I renamed ui_zoom to "zoom" according to GN style. I tweaked some jingle public dependencies so the right include directories get forwarded along with the corresponding uses in the headers. BUG= R=dpranke@chromium.org Review URL: https://codereview.chromium.org/1301743003 . Cr-Commit-Position: refs/heads/master@{#344577}
* s/use_alternate_protocols/use_alternative_services/gbnc2015-08-191-2/+2
| | | | | | | | | | | | | | Rename HttpNetworkSession::Params::use_alternate_protocols, SpdySessionDependencies::use_alternate_protocols, and IOThread::Globals::use_alternate_protocols to use_alternative_services. Also reformat some test-level comments, and remove some obsolete comments. BUG=392575 Review URL: https://codereview.chromium.org/1268313004 Cr-Commit-Position: refs/heads/master@{#344284}
* Remove remaining legacy SplitString calls.brettw2015-08-121-2/+2
| | | | | | | | TBR=zelidrag@chromium.org (chromeos) Review URL: https://codereview.chromium.org/1284833004 Cr-Commit-Position: refs/heads/master@{#342983}
* Add yet more GN isolates ...dpranke2015-07-311-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This should be almost everything left that is currently running on the Linux GN bot. - events_unittests - extensions_browsertests - extensions_unittests - gcm_unit_tests - gfx_unittests - gpu_unittests - jingle_unittests - media_unittests - mojo_common_unittests - mojo_public_bindings_unittests - mojo_public_environment_unittests - nacl_loader_unittests - net_unittests - printing_unittests - remoting_unittests - sql_unittests (isolate existed, just swarmed) - sync_integration_tests - sync_unit_tests - unit_tests There are still more test steps that need isolates that are not run on the GN bot, most notably browser_tests and interactive_ui_tests (which is partially done in this CL but doesn't quite work right). I'll get to those in upcoming CLs. R=brettw@chromium.org BUG=504079 CQ_EXTRA_TRYBOTS=tryserver.chromium.mac:mac_chromium_gn_rel Review URL: https://codereview.chromium.org/1258753002 Cr-Commit-Position: refs/heads/master@{#341392}
* Move remoting-specific adapters from jingle/glue to remoting/protocolsergeyu2015-07-069-1579/+0
| | | | | | | | | | | | | PseudoTcpAdapter and ChannelSocketAdapter are used only in remoting, so remoting/protocol is a better place for them. This change also includes two very minor cleanups in the moved code: 1. scoped_ptr<> is used to pass socket ownership in PseudoTcpAdapter 2. base::ThreadCheker is now used to check threads in ChannelSocketAdapter instead of base::MessageLoop::current(). Review URL: https://codereview.chromium.org/1177983009 Cr-Commit-Position: refs/heads/master@{#337432}
* Convert jingle_unittests to run exclusively on Swarmingmaruel2015-06-302-0/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 0 configs already ran on Swarming 29 used to run locally and were converted: - chromium.chromiumos.json: Linux ChromiumOS Ozone Tests (1) - chromium.chromiumos.json: Linux ChromiumOS Tests (1) - chromium.chromiumos.json: Linux ChromiumOS Tests (dbg)(1) - chromium.fyi.json: CrWinAsan tester - chromium.fyi.json: CrWinAsan(dll) tester - chromium.fyi.json: Linux Trusty - chromium.fyi.json: Linux Trusty (dbg) - chromium.linux.json: Cast Linux - chromium.linux.json: Linux Tests - chromium.linux.json: Linux Tests (dbg)(1) - chromium.linux.json: Linux Tests (dbg)(1)(32) - chromium.mac.json: Mac10.10 Tests - chromium.mac.json: Mac10.6 Tests - chromium.mac.json: Mac10.8 Tests - chromium.mac.json: Mac10.9 Tests - chromium.mac.json: Mac10.9 Tests (dbg) - chromium.memory.fyi.json: Linux ChromeOS MSan Tests - chromium.memory.fyi.json: Linux MSan Tests - chromium.memory.fyi.json: Linux TSan Tests - chromium.memory.json: Linux ASan LSan Tests (1) - chromium.memory.json: Linux Chromium OS ASan LSan Tests (1) - chromium.memory.json: Mac ASan 64 Tests (1) - chromium.webkit.json: Linux ChromiumOS Tests (1) - chromium.webkit.json: Linux ChromiumOS Tests (dbg)(1) - chromium.win.json: Vista Tests (1) - chromium.win.json: Win 7 Tests x64 (1) - chromium.win.json: Win7 Tests (1) - chromium.win.json: Win7 Tests (dbg)(1) - chromium.win.json: XP Tests (1) Ran: ./manage.py --convert jingle_unittests R=sergeyu@chromium.org,dpranke@chromium.org TBR= BUG=98637 Review URL: https://codereview.chromium.org/1221843002 Cr-Commit-Position: refs/heads/master@{#336858}
* Migrate callers of message_loop_proxy() to task_runner()skyostil2015-06-173-3/+3
| | | | | | | | | | | | | Migrate callers of {MessageLoop,Thread}::message_loop_proxy() to {MessageLoop,Thread}::task_runner(). Since the types at the call sites have already been updated, this is just a bulk rename. BUG=465354 TBR=armansito@chromium.org Review URL: https://codereview.chromium.org/1179163002 Cr-Commit-Position: refs/heads/master@{#334840}
* Replace more ObserverList with base::ObserverList.brettw2015-06-033-3/+3
| | | | | | | | | | | | This is everything but the chrome directory CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel TBR=zelidrag@chromium.org (chromeos) TBR=keybuk@chromium.org (device/bluetooth) Review URL: https://codereview.chromium.org/1162943002 Cr-Commit-Position: refs/heads/master@{#332626}
* [jingle] Replace MessageLoopProxy usage with ThreadTaskRunnerHandleashlin.j2015-05-205-6/+8
| | | | | | | | | | | | | | | | | Replace MessageLoopProxy usage with ThreadTaskRunnerHandle in jingle module. MessageLoopProxy is deprecated. This basically does a search and replace of: MessageLoopProxy::current() -> ThreadTaskRunnerHandle::Get(). MessageLoopProxy -> SingleThreadTaskRunner BUG=391045 Review URL: https://codereview.chromium.org/1128793002 Cr-Commit-Position: refs/heads/master@{#330740}
* Convert JsonWriter::Write to taking a const ref for the in-paramestade2015-05-161-1/+1
| | | | | | | | | | | | Clearer API; flushes out a lot of unnecessary heap allocations. depends on https://codereview.chromium.org/1129083003/ BUG=none Review URL: https://codereview.chromium.org/1131113004 Cr-Commit-Position: refs/heads/master@{#330255}
* Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProviderrch2015-05-151-0/+8
| | | | | | | | | | | | | | | | | | | | so that all provider subclasses can have their state verified. Committed: https://crrev.com/bce10d97d6fc5233071427cf622aaad7abde9e19 Cr-Commit-Position: refs/heads/master@{#328154} Committed: https://crrev.com/269875cf0d23881a7ad119bae730ff54fdd4dd0b Cr-Commit-Position: refs/heads/master@{#328351} Committed: https://crrev.com/d833a674a5e4f3b4e47f229b3d5411a8d509c2c0 Cr-Commit-Position: refs/heads/master@{#328909} Committed: https://crrev.com/1d6f67fe72b4a9ae94fcb0a6e051d803a180bde1 Cr-Commit-Position: refs/heads/master@{#329003} Review URL: https://codereview.chromium.org/1114383003 Cr-Commit-Position: refs/heads/master@{#330139}
* Collect all ConnectionAttempts from both sockets in TransportConnectJob.ttuttle2015-05-157-0/+31
| | | | | | | | | | | | | | | | | Before, the TransportConnectJob simply inferred that, if the main socket failed to connect, the address it was using was the last address in the list. With this change, the TCPClientSocket actually tracks all of the connection attempts made (as it tries each address in the list), and the TransportConnectJob copies the attempts from both the main and fallback sockets and records all of them in the ClientSocketHandle in GetAdditionalErrorState. BUG=480565 TBR=jam Review URL: https://codereview.chromium.org/1096203006 Cr-Commit-Position: refs/heads/master@{#330012}
* Roll WebRTC 9187:9189tommi2015-05-141-7/+6
| | | | | | | | | | | WebRTC 9187:9189 Changes: https://chromium.googlesource.com/external/webrtc/trunk/webrtc.git/+log/65d8695..872cf25 TBR=sergeyu@chromium.org Review URL: https://codereview.chromium.org/1136483003 Cr-Commit-Position: refs/heads/master@{#329839}
* Revert of Add AllReadDataConsumed and AllWriteDataConsumed methods to ↵rch2015-05-081-8/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | SocketDataProvider (patchset #6 id:100001 of https://codereview.chromium.org/1114383003/) Reason for revert: Even adding methods which are not called breaks this bot. How in the hell is this possible? http://build.chromium.org/p/chromium.mac/builders/iOS_Simulator_%28dbg%29/builds/26579 http://build.chromium.org/p/chromium.mac/builders/iOS_Simulator_%28dbg%29/builds/26579/steps/net_unittests%20%28iPhone%205%20iOS%208.1%29/logs/stdio Original issue's description: > Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProvider > so that all provider subclasses can have their state verified. > > Committed: https://crrev.com/bce10d97d6fc5233071427cf622aaad7abde9e19 > Cr-Commit-Position: refs/heads/master@{#328154} > > Committed: https://crrev.com/269875cf0d23881a7ad119bae730ff54fdd4dd0b > Cr-Commit-Position: refs/heads/master@{#328351} > > Committed: https://crrev.com/d833a674a5e4f3b4e47f229b3d5411a8d509c2c0 > Cr-Commit-Position: refs/heads/master@{#328909} > > Committed: https://crrev.com/1d6f67fe72b4a9ae94fcb0a6e051d803a180bde1 > Cr-Commit-Position: refs/heads/master@{#329003} TBR=rtenneti@chromium.org,sergeyu@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1126113003 Cr-Commit-Position: refs/heads/master@{#329015}
* Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProviderrch2015-05-081-0/+8
| | | | | | | | | | | | | | | | | so that all provider subclasses can have their state verified. Committed: https://crrev.com/bce10d97d6fc5233071427cf622aaad7abde9e19 Cr-Commit-Position: refs/heads/master@{#328154} Committed: https://crrev.com/269875cf0d23881a7ad119bae730ff54fdd4dd0b Cr-Commit-Position: refs/heads/master@{#328351} Committed: https://crrev.com/d833a674a5e4f3b4e47f229b3d5411a8d509c2c0 Cr-Commit-Position: refs/heads/master@{#328909} Review URL: https://codereview.chromium.org/1114383003 Cr-Commit-Position: refs/heads/master@{#329003}
* Revert of Add AllReadDataConsumed and AllWriteDataConsumed methods to ↵rch2015-05-081-8/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | SocketDataProvider (patchset #5 id:80001 of https://codereview.chromium.org/1114383003/) Reason for revert: Even this revised patch seems to break on iOS. http://build.chromium.org/p/chromium.mac/builders/iOS_Simulator_%28dbg%29/builds/26548/steps/net_unittests%20%28iPhone%205%20iOS%208.1%29/logs/stdio WTF!?!?!?!!? Original issue's description: > Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProvider > so that all provider subclasses can have their state verified. > > Committed: https://crrev.com/bce10d97d6fc5233071427cf622aaad7abde9e19 > Cr-Commit-Position: refs/heads/master@{#328154} > > Committed: https://crrev.com/269875cf0d23881a7ad119bae730ff54fdd4dd0b > Cr-Commit-Position: refs/heads/master@{#328351} > > Committed: https://crrev.com/d833a674a5e4f3b4e47f229b3d5411a8d509c2c0 > Cr-Commit-Position: refs/heads/master@{#328909} TBR=rtenneti@chromium.org,sergeyu@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1123383004 Cr-Commit-Position: refs/heads/master@{#328922}
* Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProviderrch2015-05-081-0/+8
| | | | | | | | | | | | | | so that all provider subclasses can have their state verified. Committed: https://crrev.com/bce10d97d6fc5233071427cf622aaad7abde9e19 Cr-Commit-Position: refs/heads/master@{#328154} Committed: https://crrev.com/269875cf0d23881a7ad119bae730ff54fdd4dd0b Cr-Commit-Position: refs/heads/master@{#328351} Review URL: https://codereview.chromium.org/1114383003 Cr-Commit-Position: refs/heads/master@{#328909}
* Revert of Add AllReadDataConsumed and AllWriteDataConsumed methods to ↵piman2015-05-051-8/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | SocketDataProvider (patchset #4 id:60001 of https://codereview.chromium.org/1114383003/) Reason for revert: Suspected cause of IOS failure https://build.chromium.org/p/chromium.mac/builders/iOS_Simulator_%28dbg%29/builds/26420/steps/net_unittests%20%28iPhone%205%20iOS%208.1%29/logs/crash%20report%20%282015-05-05-113310%29 Original issue's description: > Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProvider > so that all provider subclasses can have their state verified. > > Committed: https://crrev.com/bce10d97d6fc5233071427cf622aaad7abde9e19 > Cr-Commit-Position: refs/heads/master@{#328154} > > Committed: https://crrev.com/269875cf0d23881a7ad119bae730ff54fdd4dd0b > Cr-Commit-Position: refs/heads/master@{#328351} TBR=rtenneti@chromium.org,sergeyu@chromium.org,rch@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1114353005 Cr-Commit-Position: refs/heads/master@{#328375}
* Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProviderrch2015-05-051-0/+8
| | | | | | | | | | | so that all provider subclasses can have their state verified. Committed: https://crrev.com/bce10d97d6fc5233071427cf622aaad7abde9e19 Cr-Commit-Position: refs/heads/master@{#328154} Review URL: https://codereview.chromium.org/1114383003 Cr-Commit-Position: refs/heads/master@{#328351}
* Revert of Add AllReadDataConsumed and AllWriteDataConsumed methods to ↵dbeam2015-05-051-8/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | SocketDataProvider (patchset #3 id:40001 of https://codereview.chromium.org/1114383003/) Reason for revert: Final stop on the revert train! net_unittests on iOS Simulator (dbg) (iPhone 5 ios 8.1) should be fixed after this? https://build.chromium.org/p/chromium.mac/builders/iOS_Simulator_(dbg) Here's a log of the break: https://build.chromium.org/p/chromium.mac/builders/iOS_Simulator_%28dbg%29/builds/26378/steps/net_unittests%20%28iPhone%205%20iOS%208.1%29/logs/crash%20report%20%282015-05-04-164655%29 Original issue's description: > Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProvider > so that all provider subclasses can have their state verified. > > Committed: https://crrev.com/bce10d97d6fc5233071427cf622aaad7abde9e19 > Cr-Commit-Position: refs/heads/master@{#328154} TBR=rtenneti@chromium.org,sergeyu@chromium.org,rch@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1119803006 Cr-Commit-Position: refs/heads/master@{#328271}
* Revert of Revert some other possible suspects of a 0.43% size increase of ↵dbeam2015-05-041-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | setup.exe: https://build.chromium.o… (patchset #1 id:1 of https://codereview.chromium.org/1124563003/) Reason for revert: Broke the compile because one of these CLs is already depended on (rch@'s bce10d97d6fc5233071427cf622aaad7abde9e19). Original issue's description: > Revert some other possible suspects of a 0.43% size increase of setup.exe: https://build.chromium.org/p/chromium/builders/Win/builds/31562 > > Revert "Add granular base::File tracing." > > This reverts commit a6e05c977096a03774e5406d63ad80c0166f9adc. > > Revert "Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProvider" > > This reverts commit bce10d97d6fc5233071427cf622aaad7abde9e19. > > Revert "Avoid unnecessary memory allocations at PlatformThread::SetName()" > > This reverts commit 4839a142bf95776323647c82ca9dc0725f7c4f15. > > TBR=tdresser@chromium.org > BUG=none > TEST=green sizes > > Committed: https://chromium.googlesource.com/chromium/src/+/c41830d6f55f85e2f5c8841db4b6ed81239b671d TBR=tdresser@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=none Review URL: https://codereview.chromium.org/1123833003 Cr-Commit-Position: refs/heads/master@{#328207}
* Revert of Add AllReadDataConsumed and AllWriteDataConsumed methods to ↵dbeam2015-05-041-8/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | SocketDataProvider so that all provider… (patchset #1 id:1 of https://codereview.chromium.org/1127623003/) Reason for revert: Still not compiling. Order-dependent revert fun to ensue. End result: this CL and any dependent CL will be re-instated. Original issue's description: > Re-instating r328154 because it's depended on to compile. > > Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProvider so that all provider subclasses can have their state verified. > > Review URL: https://codereview.chromium.org/1114383003 > > Cr-Commit-Position: refs/heads/master@{#328154} > > TBR=rch@chromium.org > BUG= > > Committed: https://chromium.googlesource.com/chromium/src/+/2b2149aa9dfc5fbbb4ef0df2b89b5f392383f41a TBR=rch@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review URL: https://codereview.chromium.org/1123643006 Cr-Commit-Position: refs/heads/master@{#328206}
* Re-instating r328154 because it's depended on to compile.Dan Beam2015-05-041-0/+8
| | | | | | | | | | | | | | | Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProvider so that all provider subclasses can have their state verified. Review URL: https://codereview.chromium.org/1114383003 Cr-Commit-Position: refs/heads/master@{#328154} TBR=rch@chromium.org BUG= Review URL: https://codereview.chromium.org/1127623003 Cr-Commit-Position: refs/heads/master@{#328205}
* Revert some other possible suspects of a 0.43% size increase of setup.exe: ↵Dan Beam2015-05-041-8/+0
| | | | | | | | | | | | | | | | | | | | | | | | https://build.chromium.org/p/chromium/builders/Win/builds/31562 Revert "Add granular base::File tracing." This reverts commit a6e05c977096a03774e5406d63ad80c0166f9adc. Revert "Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProvider" This reverts commit bce10d97d6fc5233071427cf622aaad7abde9e19. Revert "Avoid unnecessary memory allocations at PlatformThread::SetName()" This reverts commit 4839a142bf95776323647c82ca9dc0725f7c4f15. TBR=tdresser@chromium.org BUG=none TEST=green sizes Review URL: https://codereview.chromium.org/1124563003 Cr-Commit-Position: refs/heads/master@{#328199}
* Add AllReadDataConsumed and AllWriteDataConsumed methods to SocketDataProviderrch2015-05-041-0/+8
| | | | | | | | so that all provider subclasses can have their state verified. Review URL: https://codereview.chromium.org/1114383003 Cr-Commit-Position: refs/heads/master@{#328154}
* Remove outdated instrumentation and move a few remaining bits around.pkasting2015-05-021-5/+0
| | | | | | | | | | BUG=424359,455469,455884,457528,483686,483689 TEST=none TBR=mattm,erikwright,zea Review URL: https://codereview.chromium.org/1119163004 Cr-Commit-Position: refs/heads/master@{#328038}
* Update {virtual,override} to follow C++11 style in jingle.nick2015-04-231-3/+3
| | | | | | | | | | | | The Google style guide states that only one of {virtual,override,final} should be used for each declaration, since override implies virtual and final implies both virtual and override. This patch was manually generated using a regex and a text editor. BUG=417463 Review URL: https://codereview.chromium.org/1063453006 Cr-Commit-Position: refs/heads/master@{#326596}
* NaCl/GYP: remove references to prep_toolchain.ncbray2015-04-211-1/+0
| | | | | | | | | | prep_toolchain is now a no-op. BUG=456902 Review URL: https://codereview.chromium.org/1097153002 Cr-Commit-Position: refs/heads/master@{#326044}
* Remove hclam from OWNERS files.nick2015-04-151-1/+0
| | | | | | | | | | hclam is not actively working on the project anymore BUG=None Review URL: https://codereview.chromium.org/1055933008 Cr-Commit-Position: refs/heads/master@{#325333}
* Remove ajwong from OWNERS files.nick2015-04-151-1/+0
| | | | | | | | | | ajwong is not actively working on the project anymore. BUG=None Review URL: https://codereview.chromium.org/1088743004 Cr-Commit-Position: refs/heads/master@{#325319}
* Rename SocketDataProvider::GetNextRead() to OnRead()rch2015-04-141-1/+1
| | | | | | | | | which makes it parallel to OnWrite(). It also makes it clear that it is handling the read and not simply returning the next available read. Review URL: https://codereview.chromium.org/1081713002 Cr-Commit-Position: refs/heads/master@{#325095}
* Update instrumentation for various functions janky on the Mac IO thread.pkasting2015-04-141-0/+5
| | | | | | | | | | BUG=455469,458404,457528,457525,455884,454983 TEST=none TBR=zea,mmenke,jochen Review URL: https://codereview.chromium.org/1062683005 Cr-Commit-Position: refs/heads/master@{#324995}
* Remove force_spdy_always, force_spdy_over_ssl and force_spdy_single_domain.rch2015-04-101-2/+0
| | | | | | | | | | | | | | | This required a major overhaul of SpdyNetworkTransactionTests because it was forcing SPDY in one of its modes of operation. This is terrile, because it turns out to never have been testing the "normal" mode of operations in which SPDY support was discovered in the course of the TLS handshake for an HTTPS URL. *ouch* Add a default_url_ member to SpdyTestUtil to simplify tests which do both http:// and https:// requests. Review URL: https://codereview.chromium.org/1075623002 Cr-Commit-Position: refs/heads/master@{#324562}
* Simplify the interface to ProxyClientSocket::BuildTunnelRequestrch2015-04-071-1/+1
| | | | | | | | | | | | | | | to only take the user-agent instead of a full HttpRequestInfo object. This also changes the host header being sent in CONNECT requests to include the port if it is 80, to match the behavior of the spec: https://www.ietf.org/rfc/rfc2817.txt Remove unused url argument to LogBlockedTunnelResponse Review URL: https://codereview.chromium.org/1063563002 Cr-Commit-Position: refs/heads/master@{#324054}
* Update remaining code to include NetLog files from net/log directory rather ↵eroman2015-04-023-3/+3
| | | | | | | | | | | | | than net/base. (Some files in net/base got moved to net/log) BUG=434856 TBR=armansito@chromium.org,jam@chromium.org Review URL: https://codereview.chromium.org/1050923005 Cr-Commit-Position: refs/heads/master@{#323525}
generated by cgit v1.1 at 2025-02-04 10:10:22 +0000