summaryrefslogtreecommitdiffstats
path: root/ipc
Commit message (Collapse)AuthorAgeFilesLines
* Rename FilePath -> base::FilePath in various toplevel directoriesbrettw@chromium.org2013-02-104-21/+21
| | | | | | Review URL: https://codereview.chromium.org/12211108 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181639 0039d316-1c4b-4281-b951-d872f2087c98
* ipc: Convert MessageLoop::RunUntilIdle() usage to base::RunLoop variant.tfarina@chromium.org2013-02-081-6/+6
| | | | | | | | | | | | | | The former method is deprecated and actually it is just using RunLoop internally. The later is the cannonical method and should be used instead. BUG=131220 TEST=ipc_tests R=darin@chromium.org Review URL: https://chromiumcodereview.appspot.com/12225081 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181539 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "GTTF: Add missing virtual destructors."phajdan.jr@chromium.org2013-02-061-2/+0
| | | | | | | | | | | | | | It turned out this is not necessary, we have a better warning already enabled in clang (-Wdelete-non-virtual-dtor, part of -Wall). TBR=darin,satorux,fischman,jamesr,jar,sky BUG=45135 Review URL: https://codereview.chromium.org/12224027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180971 0039d316-1c4b-4281-b951-d872f2087c98
* Linux/ChromeOS Chromium style checker cleanup, ipc/ edition.rsleevi@chromium.org2013-02-065-60/+60
| | | | | | | | | | | Automated clean up of style checker errors that were missed due to the plugin not being executed on implementation files. BUG=115047 Review URL: https://chromiumcodereview.appspot.com/12220032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180954 0039d316-1c4b-4281-b951-d872f2087c98
* GTTF: Add missing virtual destructors.phajdan.jr@chromium.org2013-02-051-0/+2
| | | | | | | | | | | R=jar TBR=darin,satorux,fischman,jamesr,sky BUG=45135 Review URL: https://codereview.chromium.org/12086018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180669 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor: Simplify WaitableEventWatcher.teravest@chromium.org2013-02-042-15/+27
| | | | | | | | | | | | | | | | | | | | | | | This change uses a callback instead of a delegate for specifying what should be called when a WaitableEvent occurs. This simplifies the class and gets rid of a workaround internal to the class to prevent name collision on "Delegate" inner classes. Tested (linux and windows): out/Debug/base_unittests --gtest_filter=*WaitableEventWatcherTest* out/Release/ipc_tests with valgrind and leak-check=yes Previously reverted at: https://codereview.chromium.org/12087120/ BUG= Review URL: https://chromiumcodereview.appspot.com/12094106 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180450 0039d316-1c4b-4281-b951-d872f2087c98
* Add FilePath to base namespace.brettw@chromium.org2013-02-021-3/+3
| | | | | | | This updates headers that forward-declare it and a few random places to use the namespace explicitly. There us a using declaration in file_path.h that makes the rest compile, which we can do in future passes. Review URL: https://codereview.chromium.org/12163003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180245 0039d316-1c4b-4281-b951-d872f2087c98
* Enable Chrome Signin from Webstoreguohui@chromium.org2013-02-011-0/+1
| | | | | | | | | | | | This CL adds webstore as a new source for Chrome signin. It also grabs password as soon as password form is submitted rather than at the end of navigation. The reason is webstore runs in its own context, redirect from gaia to webstore results in context swap during which password data is discarded, thus password must be sent to the oneclick signin helper before redirect occurs. PM mock at https://docs.google.com/a/google.com/presentation/d/1zCvdhBXKL8RDQwuRauSZdgCbiRexJRZ564z9Uw0wS6U/edit?disco=AAAAAFg3OEI#slide=id.g996e03e1_00. This CL implements the transition from slide 7 to 8. BUG=173433 Review URL: https://codereview.chromium.org/12086069 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180086 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 179987dmichael@chromium.org2013-01-312-24/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Caused memory leak: Leak_DefinitelyLost 4,196 (1,600 direct, 2,596 indirect) bytes in 4 blocks are definitely lost in loss record 608 of 639 operator new(unsigned long) (m_replacemalloc/vg_replace_malloc.c:1140) IPC::SyncChannel::SyncChannel(IPC::ChannelHandle const&, IPC::Channel::Mode, IPC::Listener*, base::SingleThreadTaskRunner*, bool, base::WaitableEvent*) (ipc/ipc_sync_channel.cc:410) (anonymous namespace)::RestrictedDispatchPipeWorker::Run() (ipc/ipc_sync_channel_unittest.cc:1636) (anonymous namespace)::Worker::OnStart() (ipc/ipc_sync_channel_unittest.cc:176) Suppression (error hash=#2A77226DFEFF6041#): For more info on using suppressions see http://dev.chromium.org/developers/tree-sheriffs/sheriff-details-chromium/memory-sheriff#TOC-Suppressing-memory-reports { <insert_a_suppression_name_here> Memcheck:Leak fun:_Znw* fun:_ZN3IPC11SyncChannelC1ERKNS_13ChannelHandleENS_7Channel4ModeEPNS_8ListenerEPN4base22SingleThreadTaskRunnerEbPNS8_13WaitableEventE fun:_ZN12_GLOBAL__N_128RestrictedDispatchPipeWorker3RunEv fun:_ZN12_GLOBAL__N_16Worker7OnStartEv } 15:02:51 memcheck_a http://build.chromium.org/p/chromium.memory.fyi/builders/Chromium%20OS%20%28valgrind%29%282%29/builds/22101 > Refactor: Simplify WaitableEventWatcher. > > This change uses a callback instead of a delegate for specifying what > should be called when a WaitableEvent occurs. > > This simplifies the class and gets rid of a workaround internal to the > class to prevent name collision on "Delegate" inner classes. > > Tested (on linux and windows): > ninja -C out/Debug chrome > out/Debug/base_unittests --gtest_filter=*WaitableEventWatcherTest* > > BUG= > > > Review URL: https://chromiumcodereview.appspot.com/11953112 TBR=teravest@chromium.org Review URL: https://codereview.chromium.org/12087120 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179993 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor: Simplify WaitableEventWatcher.teravest@chromium.org2013-01-312-15/+24
| | | | | | | | | | | | | | | | | | | This change uses a callback instead of a delegate for specifying what should be called when a WaitableEvent occurs. This simplifies the class and gets rid of a workaround internal to the class to prevent name collision on "Delegate" inner classes. Tested (on linux and windows): ninja -C out/Debug chrome out/Debug/base_unittests --gtest_filter=*WaitableEventWatcherTest* BUG= Review URL: https://chromiumcodereview.appspot.com/11953112 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179987 0039d316-1c4b-4281-b951-d872f2087c98
* ipc: Do not use Value::Create* functions.tfarina@chromium.org2013-01-302-15/+15
| | | | | | | | | | BUG=160586 R=darin@chromium.org,estade@chromium.org Review URL: https://chromiumcodereview.appspot.com/12079026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179538 0039d316-1c4b-4281-b951-d872f2087c98
* Don't allow '\0' characters in FilePath.aedla@chromium.org2013-01-282-17/+12
| | | | | | | | | BUG=169777 Review URL: https://chromiumcodereview.appspot.com/11642041 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179141 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor (many) IPC tests, notably most of the multiprocess tests.viettrungluu@chromium.org2013-01-2510-518/+472
| | | | | | | | | | | This factors out common code and, more importantly/usefully, makes test-specific code more local, and thus easier to add new tests and maintain existing ones. In particular, this allows you to add a new test "client" (running in another process) without modifying ipc_test_base.*. Review URL: https://codereview.chromium.org/12051048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@178901 0039d316-1c4b-4281-b951-d872f2087c98
* ipc: Kill the base::Passed(foo.Pass()) meme.tfarina@chromium.org2013-01-231-1/+1
| | | | | | | | | | | | This can be simply spelled as base::Passed(&foo). BUG=155593 R=dmichael@chromium.org Review URL: https://chromiumcodereview.appspot.com/12042027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@178209 0039d316-1c4b-4281-b951-d872f2087c98
* Componentize visitedlinks to src/components/visitedlinkboliu@chromium.org2013-01-231-0/+1
| | | | | | | | | | | | | | | All visitedlink files moved to src/components/visitedlink/[browser|common|renderer|test] Created static lib targets for browser, common, and renderer. Test files are moved but still uses old test targets in chrome/ Dcommitted due to AllowScopedIO being moved but still triggers presubmit error. Bots are mostly happy. BUG=168716 Review URL: https://codereview.chromium.org/11825011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@178176 0039d316-1c4b-4281-b951-d872f2087c98
* Disabling nacl_win64 targets when building in target_arch!=ia32 mode.bradnelson@google.com2013-01-211-1/+1
| | | | | | | | | | | | | | | | When building on windows with target_arch=x64, we no longer need win32 targets forced to be 64-bit. This gates out these targets when target_arch!=ia32. (Prior CL dropped the minimal set to break the dependency between these targets and the rest of the build. This eliminates them completely.) BUG=None TEST=None R=jschuh@chromium.org,thestig@chromium.org TBR=darin@chromium.org,abodenha@chromium.org,apatrick@chromium.org,sra@chromium.org,wtc@chromium.org Review URL: https://chromiumcodereview.appspot.com/11929039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@177959 0039d316-1c4b-4281-b951-d872f2087c98
* Replaced DictionaryValue::key_iterator by DictionaryValue::Iterator outside ↵pneubeck@chromium.org2013-01-181-16/+6
| | | | | | | | | | | | | | | of chrome/browser. Marked Iterator::HasNext() as deprecated and added a method Iterator::CanAdvance() as replacement. As DictionaryValue::Iterator is actually a const iterator, I had to add several missing const annotations, mostly, in json_schema_validator.* and command.* BUG=162611 TEST=No new tests. Only semantically equivalent refactorings. Review URL: https://chromiumcodereview.appspot.com/11418150 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@177673 0039d316-1c4b-4281-b951-d872f2087c98
* Remove IPC::Channel::set_listener().viettrungluu@chromium.org2013-01-165-25/+26
| | | | | | | | | | It wasn't used anywhere other than in ipc_perftests (which, up till very recently, weren't even compiled) in a non-essential way. Review URL: https://chromiumcodereview.appspot.com/11964002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@177192 0039d316-1c4b-4281-b951-d872f2087c98
* More IPC test header cleanup.viettrungluu@chromium.org2013-01-157-7/+1
| | | | | | Review URL: https://codereview.chromium.org/11858023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@176983 0039d316-1c4b-4281-b951-d872f2087c98
* Automate ipc_perftests.viettrungluu@chromium.org2013-01-151-81/+151
| | | | | | | | | | Make it run with a decent spread of parameters. TBR=jam Review URL: https://codereview.chromium.org/11886009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@176960 0039d316-1c4b-4281-b951-d872f2087c98
* Windows: Do not handle async read completions on IPC channel while the ↵apatrick@chromium.org2013-01-151-1/+1
| | | | | | | | | | | | channel is being closed. This resulted in messages being dispatched to objects while they were closing the IPC channel, potentially when they were in a partially destroyed state, leading to a crash. BUG=165269 Review URL: https://chromiumcodereview.appspot.com/11840003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@176818 0039d316-1c4b-4281-b951-d872f2087c98
* Reenable IPCSyncChannelTest.SendWithTimeoutMixedOKAndTimeoutviettrungluu@chromium.org2013-01-121-5/+1
| | | | | | | | | | | | | | This is speculative: I fixed crbug.com/25841, and hope that that's the underlying cause for crbug.com/70075 (flakiness on Windows). Or maybe the cause was something else that has also since gone away (since the test was reliable for a long time before). BUG=70075 Review URL: https://chromiumcodereview.appspot.com/11861018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@176578 0039d316-1c4b-4281-b951-d872f2087c98
* Trim #includes in ipc tests.viettrungluu@chromium.org2013-01-113-23/+3
| | | | | | Review URL: https://codereview.chromium.org/11858018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@176438 0039d316-1c4b-4281-b951-d872f2087c98
* Move IPC tests into anonymous namespaces.viettrungluu@chromium.org2013-01-1110-105/+54
| | | | | | | | | This reduces chances of (very confusing) name collisions between different tests. Review URL: https://codereview.chromium.org/11865015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@176430 0039d316-1c4b-4281-b951-d872f2087c98
* Make ipc_tests file structure a little saner and add an ipc_perftests target.viettrungluu@chromium.org2013-01-118-312/+413
| | | | | | | | | | This means that the (one, semi-manual) IPC perf test that we have will build without manual hackery (and do so separately from the ipc_tests target). Review URL: https://chromiumcodereview.appspot.com/11819041 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@176341 0039d316-1c4b-4281-b951-d872f2087c98
* Data collection part of webrtc-internals implementationjiayl@chromium.org2013-01-101-0/+1
| | | | | | | | | | | | | | | | | | | This change implements the data collection part of webrtc-internals. RTCPeerConnectionHandler is an existing class living in the main renderer thread and knows about the status of a peer connection. With this CL, it reports every peer connection creation to PeerConnectionTracker. PeerConnectionTracker is also running in the renderer process. It assigns an integer ID to each peer connectionstore and sends the basic peer connection info to PeerConnectionTrackerHost along with the ID. PeerConnectionTrackerHost is running in the browser process in the IO thread. It simply passes anything it receives from PeerConnectionTracker to WebRTCInternals. WebRTCInternals is a singlton running in the browser process. It sends the data received from PeerConnectionTrackerHost to every observer, i.e. webrtc-internals page. WebRTCInternalsUIObserver will be implemented in another change list. BUG=168232 Review URL: https://chromiumcodereview.appspot.com/11753020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@176012 0039d316-1c4b-4281-b951-d872f2087c98
* Fix shutdown race in IPCSyncChannelTest and get rid of ↵viettrungluu@chromium.org2013-01-082-26/+30
| | | | | | | | | | | | | | | | | | "suppressions"/annotations. Shut down explicitly, rather than doing it in a virtual destructor, which leads to a vtable race. (Possibly also related is http://crbug.com/70075, but I haven't been able to reproduce, so I'll try to re-enable that separately.) BUG=25841 TEST=ipc_tests + TSAN still happy Review URL: https://chromiumcodereview.appspot.com/11761038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@175487 0039d316-1c4b-4281-b951-d872f2087c98
* Make the IPC perf test build/run again, at least on Linux.viettrungluu@chromium.org2013-01-051-32/+38
| | | | | | | | | | It's still all terribly manual and fragile, but that'll be taken care of separately. Review URL: https://chromiumcodereview.appspot.com/11797002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@175267 0039d316-1c4b-4281-b951-d872f2087c98
* Fix typo in comment ("ammount").viettrungluu@chromium.org2013-01-051-1/+1
| | | | | | | | | | TBR=jam@chromium.org NOTRY=true Review URL: https://chromiumcodereview.appspot.com/11794011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@175262 0039d316-1c4b-4281-b951-d872f2087c98
* Use an explicit PID for duplicating Pepper handles rather than the Channel's.brettw@chromium.org2013-01-041-1/+8
| | | | | | | | | | | | When the browser process launches the plugin, it explicitly tells each side the PID of the other side, and we now use this PID for sharing handles. Previously we'd use the PID from the IPC channel. Using the PID from the IPC channel creates a race condition because the PID isn't set until the "hello" message from the opposite side is processed, which isn't guaranteed at any particular time. BUG=168222 Review URL: https://codereview.chromium.org/11722017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@175190 0039d316-1c4b-4281-b951-d872f2087c98
* reinterpret_cast -> static_cast in SyncMessageFilter::Send().viettrungluu@chromium.org2013-01-041-1/+1
| | | | | | Review URL: https://chromiumcodereview.appspot.com/11722033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@175174 0039d316-1c4b-4281-b951-d872f2087c98
* Validate that paths don't contain embedded NULLs at deserialization.cevans@chromium.org2013-01-032-0/+19
| | | | | | | | BUG=166867 Review URL: https://chromiumcodereview.appspot.com/11743009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@174935 0039d316-1c4b-4281-b951-d872f2087c98
* Remove IPC::MessageIterator.viettrungluu@chromium.org2012-12-174-55/+42
| | | | | | | | | | It's a rarely-used and completely unnecessary (and thin) wrapper around PickleIterator. Review URL: https://chromiumcodereview.appspot.com/11570038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@173525 0039d316-1c4b-4281-b951-d872f2087c98
* Get rid of ipc_sender.h include in ipc_message.h.viettrungluu@chromium.org2012-12-143-6/+2
| | | | | | Review URL: https://codereview.chromium.org/11574040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@173238 0039d316-1c4b-4281-b951-d872f2087c98
* Add tracing support to NaCljbauman@chromium.org2012-12-081-0/+1
| | | | | | | | | | Add Ppapi IPC messages and a handler inside of the NaCl IRT to enable, disable, and send the result of tracing. BUG=93839 Review URL: https://codereview.chromium.org/11411118 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171911 0039d316-1c4b-4281-b951-d872f2087c98
* Marked IPC::ChannelProxy as non thread-safe. Added DCHECKs to verify that ↵alexeypa@chromium.org2012-12-052-1/+22
| | | | | | | | | | | | public methods (with Send() being the only exception) are called on the thread that created the object. Tests calling Send() from a wrong thread should be fixed first before similar check can be added to Send(). See http://crbug.com/163523 for details. BUG=163091,163523 Review URL: https://chromiumcodereview.appspot.com/11308278 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171106 0039d316-1c4b-4281-b951-d872f2087c98
* Fix check for maximum IPC message size.rkc@chromium.org2012-11-281-2/+2
| | | | | | | | | | | If the incoming message length is larger than Channel::kMaximumMessageSize, the RHS for the check for message size will become negative - since both sides are promoted to unsigned, the negative will simply become a very large value hence breaking the check. R=tsepez@chromium.org BUG=None. Review URL: https://codereview.chromium.org/11413204 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170072 0039d316-1c4b-4281-b951-d872f2087c98
* Enable IPC logging for component buildsjochen@chromium.org2012-11-281-4/+1
| | | | | | | | | | | | Also fix a Mac test for the about ipc controller - it was just not executed on the bots before. BUG=none TEST=win dbg still compiles, no mac unit_tests failures Review URL: https://chromiumcodereview.appspot.com/11280186 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@169934 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 169453 - Enable IPC logging for component builddalecurtis@google.com2012-11-261-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | Since I refactored the registering of IPC loggers to go through a proper API, there's no dependency from ipc to chrome anymore, and we can enable it in debug builds BUG=none TEST=compiles in debug build ------------------------------------------------------ Caused "unresolved external" link errors on Windows Debug builds: http://build.chromium.org/p/chromium.win/builders/Win%20Builder%20%28dbg%29/builds/5494/steps/compile/logs/stdio Possibly the cause of various "malloc" related failures seen elsewhere in the tree: http://build.chromium.org/p/chromium.mac/buildstatus?builder=Mac10.6%20Sync&number=25248 Review URL: https://chromiumcodereview.appspot.com/11415122 TBR=jochen@chromium.org Review URL: https://codereview.chromium.org/11428013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@169486 0039d316-1c4b-4281-b951-d872f2087c98
* Enable IPC logging for component buildjochen@chromium.org2012-11-261-4/+1
| | | | | | | | | | | | | | Since I refactored the registering of IPC loggers to go through a proper API, there's no dependency from ipc to chrome anymore, and we can enable it in debug builds BUG=none TEST=compiles in debug build Review URL: https://chromiumcodereview.appspot.com/11415122 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@169453 0039d316-1c4b-4281-b951-d872f2087c98
* [base] Move AutoReset to base namespace.dbeam@chromium.org2012-11-211-1/+2
| | | | | | | | | | | NOTRY=true R=darin@chromium.org BUG=None Review URL: https://chromiumcodereview.appspot.com/11308132 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@168978 0039d316-1c4b-4281-b951-d872f2087c98
* Verify lfFaceName is NUL terminated in IPC deserializer.jschuh@chromium.org2012-11-211-7/+10
| | | | | | | | BUG=162066 Review URL: https://chromiumcodereview.appspot.com/11416115 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@168937 0039d316-1c4b-4281-b951-d872f2087c98
* ipc: Remove ipc_listener.h from ipc_channel.h and update the files.tfarina@chromium.org2012-11-174-5/+4
| | | | | | | | | | | Note: This was a TODO for brett in ipc_channel.h TBR=cpu@chromium.org Review URL: https://chromiumcodereview.appspot.com/11308082 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@168458 0039d316-1c4b-4281-b951-d872f2087c98
* ipc: Update calls from RunAllPending() to RunUntilIdle().tfarina@chromium.org2012-11-161-3/+3
| | | | | | | | | | | | RunAllPending() is deprecated and we should switch to RunUntilIdle(). BUG=131220 TBR=cpu@chromium.org NOTRY=true Review URL: https://chromiumcodereview.appspot.com/11420048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@168331 0039d316-1c4b-4281-b951-d872f2087c98
* Add missing includes to ipc_listener.htfarina@chromium.org2012-11-152-2/+4
| | | | | | | | | | | This is TODO in ipc_channel.h that I'm working on. TBR=brettw@chromium.org Review URL: https://chromiumcodereview.appspot.com/11416016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@168055 0039d316-1c4b-4281-b951-d872f2087c98
* LogData.message_name not marshalled across logging IPCs.tsepez@chromium.org2012-11-151-0/+2
| | | | | | Review URL: https://chromiumcodereview.appspot.com/11364242 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167838 0039d316-1c4b-4281-b951-d872f2087c98
* Move eintr_wrapper.h from base to base/posixbrettw@chromium.org2012-11-145-5/+5
| | | | | | Review URL: https://codereview.chromium.org/11366229 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167739 0039d316-1c4b-4281-b951-d872f2087c98
* net: Change type of UploadData::elements from std::vector to ScopedVectorhashimoto@chromium.org2012-11-141-0/+32
| | | | | | | | | | | | | | Using std::vector to hold UploadElement is bad for two reasons: 1. It results in a lot of unnecessary copy of uploaded data. 2. Appending new chunks may result in invalidating the pointer held by UploadBytesElementReader. BUG=160028 TEST=git try TBR=ananta@chromium.org, tony@chromium.org Review URL: https://chromiumcodereview.appspot.com/11275223 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167611 0039d316-1c4b-4281-b951-d872f2087c98
* Move global_descriptors to base/posix.brettw@chromium.org2012-11-093-3/+3
| | | | | | | | | | This file represents a posix-only concept. BUG= Review URL: https://codereview.chromium.org/11293210 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167008 0039d316-1c4b-4281-b951-d872f2087c98
* Split IPC logging between content and chrome.jochen@chromium.org2012-10-301-14/+4
| | | | | | | | | | | | | I add a new content API to register IPC message loggers, and modify the IPC message macros to not directly create the g_log_function_mapping. That allows for multiple files generating IPC loggers. Also, it gets rid of the ctor/dtor for the g_log_function_mapping. BUG=101600,111316,155765 Review URL: https://codereview.chromium.org/11347012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@164861 0039d316-1c4b-4281-b951-d872f2087c98