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

Unified Diff: chrome/browser/sync/test/integration/sync_test.cc

Issue 11308072: Include v2 packaged apps in sync. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: comment explaining index Created 8 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/sync/test/integration/sync_test.cc
diff --git a/chrome/browser/sync/test/integration/sync_test.cc b/chrome/browser/sync/test/integration/sync_test.cc
index d64411c689308fa95a2dbd9c64de853a9658cb65..911d580d43cfb25dc4d910e53331a789f8875634 100644
--- a/chrome/browser/sync/test/integration/sync_test.cc
+++ b/chrome/browser/sync/test/integration/sync_test.cc
@@ -360,6 +360,9 @@ bool SyncTest::SetupSync() {
}
void SyncTest::CleanUpOnMainThread() {
+ // Some of the pending messages might rely on browser windows still being
+ // around, so run messages both before and after closing all browsers.
+ content::RunAllPendingInMessageLoop();
// Close all browser windows.
browser::CloseAllBrowsers();
content::RunAllPendingInMessageLoop();

Powered by Google App Engine
This is Rietveld 408576698