Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(631)

Side by Side Diff: content/public/test/render_view_test.cc

Issue 1645923002: RenderViewTest: really drain the event loop before shutting down Blink. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | content/renderer/render_view_browsertest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/public/test/render_view_test.h" 5 #include "content/public/test/render_view_test.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <cctype> 9 #include <cctype>
10 10
(...skipping 391 matching lines...) Expand 10 before | Expand all | Expand 10 after
402 // (http://crbug.com/21508). 402 // (http://crbug.com/21508).
403 base::RunLoop().RunUntilIdle(); 403 base::RunLoop().RunUntilIdle();
404 404
405 #if defined(OS_MACOSX) 405 #if defined(OS_MACOSX)
406 // Needs to run before blink::shutdown(). 406 // Needs to run before blink::shutdown().
407 autorelease_pool_.reset(NULL); 407 autorelease_pool_.reset(NULL);
408 #endif 408 #endif
409 409
410 leak_detector->collectGarbageAndReport(); 410 leak_detector->collectGarbageAndReport();
411 411
412 base::RunLoop().RunUntilIdle();
413
412 blink_platform_impl_.Shutdown(); 414 blink_platform_impl_.Shutdown();
413 blink::shutdown(); 415 blink::shutdown();
414 416
415 platform_->PlatformUninitialize(); 417 platform_->PlatformUninitialize();
416 platform_.reset(); 418 platform_.reset();
417 params_.reset(); 419 params_.reset();
418 command_line_.reset(); 420 command_line_.reset();
419 421
420 test_io_thread_.reset(); 422 test_io_thread_.reset();
421 ipc_support_.reset(); 423 ipc_support_.reset();
(...skipping 294 matching lines...) Expand 10 before | Expand all | Expand 10 after
716 TestRenderFrame* frame = 718 TestRenderFrame* frame =
717 static_cast<TestRenderFrame*>(impl->GetMainRenderFrame()); 719 static_cast<TestRenderFrame*>(impl->GetMainRenderFrame());
718 frame->Navigate(common_params, StartNavigationParams(), request_params); 720 frame->Navigate(common_params, StartNavigationParams(), request_params);
719 721
720 // The load actually happens asynchronously, so we pump messages to process 722 // The load actually happens asynchronously, so we pump messages to process
721 // the pending continuation. 723 // the pending continuation.
722 FrameLoadWaiter(frame).Wait(); 724 FrameLoadWaiter(frame).Wait();
723 } 725 }
724 726
725 } // namespace content 727 } // namespace content
OLDNEW
« no previous file with comments | « no previous file | content/renderer/render_view_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698