Closed
Bug 631918
Opened 15 years ago
Closed 15 years ago
Sync Now can briefly block the menu when not logged in
Categories
(Firefox :: Sync, defect)
Firefox
Sync
Tracking
()
VERIFIED
FIXED
People
(Reporter: philikon, Assigned: philikon)
References
Details
Attachments
(1 file)
|
1.10 KB,
patch
|
mconnor
:
review+
mconnor
:
approval2.0+
|
Details | Diff | Splinter Review |
Bug 597788 wrapped Service.sync() in a setTimeout call, but not the login() (which blocks, too). We shouldn't have to call it anymore, anyway. Service.sync() does it for us now.
|
Assignee | |
Comment 1•15 years ago
|
||
No need to call login(), just call sync(). Also get rid of no longer needed UI methods (the buttons were removed).
|
|
||
Updated•15 years ago
|
Attachment #510136 -
Flags: review?(mconnor)
Attachment #510136 -
Flags: review+
|
|
||
Updated•15 years ago
|
Attachment #510136 -
Flags: review+
|
|
Assignee | |
Updated•15 years ago
|
Attachment #510136 -
Flags: approval2.0?
|
|
||
Updated•15 years ago
|
Attachment #510136 -
Flags: approval2.0? → approval2.0+
|
|
Assignee | |
Comment 2•15 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
|
||
Comment 3•15 years ago
|
||
verified with nightly minefield build of 20110218
Status: RESOLVED → VERIFIED
|
||
Updated•8 years ago
|
Component: Firefox Sync: UI → Sync
Product: Cloud Services → Firefox
You need to log in
before you can comment on or make changes to this bug.
Description
•