Bug 1397744 - Show the Sync animation immediately after clicking the manual sync button. r=markh
☠☠ backed out by aa97adfab999 ☠ ☠
authorEdouard Oger <eoger@fastmail.com>
Wed, 20 Sep 2017 13:16:19 -0400
changeset 383236 11d5f873f4094211cef995b58c41408b41fd9b95
parent 383235 263b2736bf7656ae3051d75b8354afde54ee171d
child 383237 cf5ecd16d87ed1afb96e4956d06fa2b50f260bea
push id52159
push usereoger@mozilla.com
push dateWed, 27 Sep 2017 14:28:15 +0000
treeherderautoland@11d5f873f409 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmarkh
bugs1397744
milestone58.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 1397744 - Show the Sync animation immediately after clicking the manual sync button. r=markh MozReview-Commit-ID: AIBjAZZlYX6
browser/base/content/browser-sync.js
--- a/browser/base/content/browser-sync.js
+++ b/browser/base/content/browser-sync.js
@@ -519,16 +519,17 @@ var gSync = {
   // doSync forces a sync - it *does not* return a promise as it is called
   // via the various UI components.
   doSync() {
     if (!UIState.isReady()) {
       return;
     }
     const state = UIState.get();
     if (state.status == UIState.STATUS_SIGNED_IN) {
+      this.updateSyncStatus({ syncing: true });
       setTimeout(() => Weave.Service.errorHandler.syncAndReportErrors(), 0);
     }
   },
 
   openPrefs(entryPoint = "syncbutton", origin = undefined) {
     window.openPreferences("paneSync", { origin, urlParams: { entrypoint: entryPoint } });
   },