diff options
author | darin@chromium.org <darin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-02-23 07:12:22 +0000 |
---|---|---|
committer | darin@chromium.org <darin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-02-23 07:12:22 +0000 |
commit | 781a7edbdb3288b237acbeff4926b47b243cef17 (patch) | |
tree | 04a76bff6c816a5a1095583bd6029485fdc1312b /chrome/renderer/renderer_webkitclient_impl.cc | |
parent | aab929a85b3bfe1113902dec3e5c723c552fdfda (diff) | |
download | chromium_src-781a7edbdb3288b237acbeff4926b47b243cef17.zip chromium_src-781a7edbdb3288b237acbeff4926b47b243cef17.tar.gz chromium_src-781a7edbdb3288b237acbeff4926b47b243cef17.tar.bz2 |
Add MessageLoop::QuitNow and use it to terminate a sync IPC.
This is a test to observe the performance impact of this change.
As such, I'm reverting r39518 as part of this change.
R=jam
BUG=36310
TEST=none
Review URL: http://codereview.chromium.org/652089
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39699 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/renderer/renderer_webkitclient_impl.cc')
-rw-r--r-- | chrome/renderer/renderer_webkitclient_impl.cc | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/chrome/renderer/renderer_webkitclient_impl.cc b/chrome/renderer/renderer_webkitclient_impl.cc index 9b7327d..9a834ea 100644 --- a/chrome/renderer/renderer_webkitclient_impl.cc +++ b/chrome/renderer/renderer_webkitclient_impl.cc @@ -131,12 +131,8 @@ WebString RendererWebKitClientImpl::cookies( // when there is no active script context. int32 routing_id = RenderThread::RoutingIDForCurrentContext(); - // TODO(darin): We should use SendAndRunNestedMessageLoop here to avoid dead- - // locking the browser, but this causes a performance regression. Switching - // back to Send to verify. See http://crbug.com/36310. - std::string value_utf8; - RenderThread::current()->Send( + RenderThread::current()->SendAndRunNestedMessageLoop( new ViewHostMsg_GetCookies(routing_id, url, first_party_for_cookies, &value_utf8)); |