diff options
| author | hiroshige <hiroshige@chromium.org> | 2015-08-06 02:28:14 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-08-06 09:28:53 +0000 |
| commit | a116e2287895078d72b2766ab1ac44582b27a7c6 (patch) | |
| tree | 4b6d08f341c9c5edaa285ed8a3d5a4fc3dc4693d /ipc | |
| parent | cffa650f655e0c47e77d6375cc073728ea08c5e2 (diff) | |
| download | chromium_src-a116e2287895078d72b2766ab1ac44582b27a7c6.zip chromium_src-a116e2287895078d72b2766ab1ac44582b27a7c6.tar.gz chromium_src-a116e2287895078d72b2766ab1ac44582b27a7c6.tar.bz2 | |
Revert of Re-enable ChannelMojo (patchset #1 id:20001 of https://codereview.chromium.org/1262503011/ )
Reason for revert:
Suspecting this CL causes many Layout test failures (https://crbug.com/517391).
Original issue's description:
> Re-enable ChannelMojo
>
> The performance regression seen previously was probably
> misattributed to ChannelMojo. Also, given that a great deal of
> code will now take advantage of the channel's thread-safe Send,
> this is expected to result in an net performance improvement.
>
> BUG=500019
> TBR=jam@chromium.org
>
> Committed: https://crrev.com/ab1a53c964a77dabb6d08b7199111b51a330885e
> Cr-Commit-Position: refs/heads/master@{#342055}
TBR=rockot@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=500019
Review URL: https://codereview.chromium.org/1265383006
Cr-Commit-Position: refs/heads/master@{#342085}
Diffstat (limited to 'ipc')
| -rw-r--r-- | ipc/mojo/ipc_channel_mojo.cc | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/ipc/mojo/ipc_channel_mojo.cc b/ipc/mojo/ipc_channel_mojo.cc index 9994f8b..08c3458 100644 --- a/ipc/mojo/ipc_channel_mojo.cc +++ b/ipc/mojo/ipc_channel_mojo.cc @@ -234,7 +234,9 @@ void ChannelMojo::ChannelInfoDeleter::operator()( // static bool ChannelMojo::ShouldBeUsed() { - return true; + // TODO(rockot): Investigate performance bottlenecks and hopefully reenable + // this at some point. http://crbug.com/500019 + return false; } // static |