Bug 1383835 - Increase the amount of time TPS waits after a tab sync before continuing r=markh
authorThom Chiovoloni <tchiovoloni@mozilla.com>
Mon, 24 Jul 2017 14:01:03 -0400
changeset 419541 ca6f811e807ea7c4a2296de5838d4fa61b566d14
parent 419540 8a193ce80a59d7a2062783efdf215c4c36fd8d6f
child 419542 5105e408029e772a9d18b39e25aec655a9e52cb4
push id7566
push usermtabara@mozilla.com
push dateWed, 02 Aug 2017 08:25:16 +0000
treeherdermozilla-beta@86913f512c3c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmarkh
bugs1383835
milestone56.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1383835 - Increase the amount of time TPS waits after a tab sync before continuing r=markh MozReview-Commit-ID: 1cvmtpkjSbX
services/sync/tps/extensions/tps/resource/tps.jsm
--- a/services/sync/tps/extensions/tps/resource/tps.jsm
+++ b/services/sync/tps/extensions/tps/resource/tps.jsm
@@ -320,21 +320,21 @@ var TPS = {
           let that = this;
           let taburi = tab.uri;
           BrowserTabs.Add(tab.uri, function() {
             that._tabsFinished++;
             Logger.logInfo("tab for " + taburi + " finished loading");
             if (that._tabsFinished == that._tabsAdded) {
               Logger.logInfo("all tabs loaded, continuing...");
 
-              // Wait a second before continuing to be sure tabs can be synced,
-              // otherwise we can get 'error locating tab'
+              // Wait some time before continuing to be sure tabs can be synced,
+              // otherwise we can get 'error locating tab' (bug 1383832).
               Utils.namedTimer(function() {
                 that.FinishAsyncOperation();
-              }, 1000, this, "postTabsOpening");
+              }, 2500, this, "postTabsOpening");
             }
           });
           break;
         case ACTION_VERIFY:
           Logger.AssertTrue(typeof(tab.profile) != "undefined",
             "profile must be defined when verifying tabs");
           Logger.AssertTrue(
             BrowserTabs.Find(tab.uri, tab.title, tab.profile), "error locating tab");