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 = |