Chromium Code Reviews| 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. |
|
akalin
2012/11/19 19:28:59
whoa, what did you run into that needed this?
Marijn Kruisselbrink
2012/11/19 19:39:59
Without that line, I got the following stack trace
|
| + content::RunAllPendingInMessageLoop(); |
| // Close all browser windows. |
| browser::CloseAllBrowsers(); |
| content::RunAllPendingInMessageLoop(); |