Closed Bug 1361171 Opened 3 years ago Closed 3 years ago

Remove CloudSync

Categories

(Firefox :: Sync, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
Firefox 55
Tracking Status
firefox55 --- fixed

People

(Reporter: Lina, Assigned: Lina)

References

Details

Attachments

(1 file)

59 bytes, text/x-review-board-request
markh
: review+
Details
It's fairly self-contained, but unsupported, hasn't been updated in years, and is certainly broken.
Duplicate of this bug: 1184003
Priority: -- → P2
Comment on attachment 8863503 [details]
Bug 1361171 - Remove CloudSync.

https://reviewboard.mozilla.org/r/135276/#review138208

Awesome!

::: browser/base/content/browser-syncui.js
(Diff revision 1)
>   * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
>  
>  Cu.import("resource://gre/modules/XPCOMUtils.jsm");
>  Cu.import("resource://gre/modules/Services.jsm");
>  
> -if (AppConstants.MOZ_SERVICES_CLOUDSYNC) {

this is going to conflict with eoger (not a problem, just a heads-up)
Attachment #8863503 - Flags: review?(markh) → review+
Comment on attachment 8863503 [details]
Bug 1361171 - Remove CloudSync.

https://reviewboard.mozilla.org/r/135276/#review138208

> this is going to conflict with eoger (not a problem, just a heads-up)

Cool, I'll wait for him to land. Thanks for the heads-up!
https://hg.mozilla.org/mozilla-central/rev/fa803dc1b64c
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
You need to log in before you can comment on or make changes to this bug.