Bug 1337940 - Part 2 - Make session store form data test work again. r?ahunt draft
authorJan Henning <jh+bugzilla@buttercookie.de>
Mon, 13 Feb 2017 22:16:36 +0100
changeset 490163 1a4acfb318f45ced29307701f464ea2897bd62cd
parent 490162 708b040823ccc5dc237e0be8d06e4ece1b325cbd
child 490164 8fc2a3a2b19ae0dacd368b2cedb09b11b3682e6c
push id47021
push usermozilla@buttercookie.de
push dateMon, 27 Feb 2017 19:33:04 +0000
reviewersahunt
bugs1337940
milestone54.0a1
Bug 1337940 - Part 2 - Make session store form data test work again. r?ahunt Collecting data for history changes causes an additional session store data update for that tab when navigating back, which needs to be accounted for in this test. Therefore we now also wait for DOMTitleChanged before assuming that the tab has navigated to its intended location. MozReview-Commit-ID: FDNQednXPWh
mobile/android/tests/browser/chrome/test_session_form_data.html
--- a/mobile/android/tests/browser/chrome/test_session_form_data.html
+++ b/mobile/android/tests/browser/chrome/test_session_form_data.html
@@ -220,17 +220,21 @@ add_task(function* test_formdata_navigat
       // Visit a different page.
       gBrowserApp.loadURI(otherURL, browser);
       yield promiseBrowserEvent(browser, "DOMContentLoaded");
       is(browser.currentURI.spec, otherURL, "navigated to a different page");
 
       // Go back.
       is(browser.canGoBack, true, "can go back");
       browser.goBack();
-      yield promiseTabEvent(browser, "SSTabDataUpdated");
+
+      let titleChange = promiseTabEvent(browser, "DOMTitleChanged");
+      let tabDataUpdate = promiseTabEvent(browser, "SSTabDataUpdated");
+      yield titleChange;
+      yield tabDataUpdate;
       is(browser.currentURI.spec, URL, "navigated back to form data page");
 
       // Make sure form data is still present.
       is(getInputValue(browser, {id: "txt"}), OUTER_VALUE, "outer value present after navigation");
       is(getInputValue(browser, {id: "txt", frame: 0}), INNER_VALUE, "inner value present after navigation");
 
       // Remove the tab.
       gBrowserApp.closeTab(tab);