806351 - disable Notification Center support on beta (Firefox 17), it isn't ready

Status: RESOLVED FIXED

(Toolkit :: General, defect)

Description

[Josh Aas]

Attachment: fix v1.0

I'm not confident enough in our new Notification Center support to ship it in Firefox 17. We're putting more effort into refining alerts on OS X in upcoming releases. We should disable it on beta.

Comment 1

[Doug Turner (:dougt)]

Comment on attachment 676135 [details] [diff] [review]
fix v1.0

Use a pref, maybe?

Comment 2

[Josh Aas]

(In reply to Doug Turner (:dougt) from comment #1)
> Comment on attachment 676135 [details] [diff] [review]
> fix v1.0
> 
> Use a pref, maybe?

I'm not a fan of adding prefs in general, this seems too minor until it actually works well.

Comment 3

[Josh Aas]

Comment on attachment 676135 [details] [diff] [review]
fix v1.0

[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 728106
User impact if declined: OS X 10.8 users will have a poor Notification Center experience
Testing completed (on m-c, etc.): none
Risk to taking this patch (and alternatives if risky): not risky
String or UUID changes made by this patch: none

Comment 4

[Lukas Blakk [:lsblakk] use ?needinfo]

Comment on attachment 676135 [details] [diff] [review]
fix v1.0

Approving the disabling for 17 (Beta) and marking tracking/affected for 18 as well so we can check in again in the next Beta cycle to ensure this can stay in.

Comment 5

[Josh Aas]

pushed to mozilla-beta

http://hg.mozilla.org/releases/mozilla-beta/rev/9cfe29fd1612

Comment 6

[Josh Aas]

Please clear the flags for Firefox 18 on this bug. This bug is only about Firefox 17, bug 805837 is about Firefox 18.