Chromium Code Reviews| Index: chrome/test/data/webui/settings/languages_page_browsertest.js |
| diff --git a/chrome/test/data/webui/settings/languages_page_browsertest.js b/chrome/test/data/webui/settings/languages_page_browsertest.js |
| index a03eaa3d0f764dfb9e80bdb83e9098c5353cdd83..f1aa4328f5ed2296f737bb53da4f073f6fcea768 100644 |
| --- a/chrome/test/data/webui/settings/languages_page_browsertest.js |
| +++ b/chrome/test/data/webui/settings/languages_page_browsertest.js |
| @@ -67,11 +67,18 @@ TEST_F('SettingsLanguagesPageBrowserTest', 'MAYBE_LanguagesPage', function() { |
| }.bind(this)); |
| teardown(function(done) { |
|
tommycli
2016/07/20 22:16:15
This teardown is needed to navigate back after eac
michaelpg
2016/07/20 22:40:08
yes, adjusted the comment
|
| - if (this.isAtRoot()) |
| - return done(); |
| - this.backToRoot(); |
| - setTimeout(done); |
| - }.bind(this)); |
| + if (advanced.currentRoute.subpage.length == 0) { |
| + done(); |
| + } else { |
| + // Close the section. |
| + advanced.currentRoute = { |
| + page: 'advanced', |
| + section: '', |
| + subpage: [], |
| + }; |
| + setTimeout(done); |
|
tommycli
2016/07/20 22:16:15
Does advanced.async make more sense?
michaelpg
2016/07/20 22:40:08
No, the tests can become flaky that way. I think w
|
| + } |
| + }); |
| test('manage languages', function() { |
| var manageLanguagesButton = |