Closed Bug 822620 Opened 12 years ago Closed 12 years ago

disable Notification Center support for Firefox 19 and 20, it isn't ready

Categories

(Toolkit :: UI Widgets, defect)

19 Branch
All
macOS
defect
Not set
normal

Tracking

()

VERIFIED FIXED
mozilla20
Tracking Status
firefox19 --- verified
firefox20 --- verified

People

(Reporter: jaas, Assigned: jaas)

Details

Attachments

(1 file)

I left Notification Center support enabled in Firefox 19 and 20 on the assumption that the W3C notification work (bug 782211) would complete in those releases. It never happened, so we need to disable our incomplete Notification Center support in Firefox 19 and 20.
Attached patch fix v1.0Splinter Review
Attachment #695273 - Flags: review?(doug.turner)
Comment on attachment 695273 [details] [diff] [review] fix v1.0 Review of attachment 695273 [details] [diff] [review]: ----------------------------------------------------------------- ::: toolkit/components/alerts/mac/nsMacAlertsService.mm @@ +100,5 @@ > nsresult > nsMacAlertsService::Init() > { > + // Notification Center support isn't ready, don't use it. > + return NS_ERROR_NOT_AVAILABLE; How about adding a preference and using the new: #ifdef RELEASE_BUILD
(In reply to Doug Turner (:dougt) from comment #2) > How about adding a preference and using the new: > > #ifdef RELEASE_BUILD I'm intending for all of it to just get replaced, the feature as I wrote it probably isn't going to be further developed. I don't think we gain anything by letting people turn it on.
Attachment #695273 - Flags: review?(doug.turner) → review+
Comment on attachment 695273 [details] [diff] [review] fix v1.0 [Approval Request Comment] Bug caused by (feature/regressing bug #): bug 728106 User impact if declined: We'll ship an incomplete Notification Center implementation to users. Testing completed (on m-c, etc.): This has been disabled on beta for a while now. Risk to taking this patch (and alternatives if risky): very little risk String or UUID changes made by this patch: none
Attachment #695273 - Flags: approval-mozilla-aurora?
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla20
Attachment #695273 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Not to nitpick, but should the status flags be set to "disabled" in this case? Additionally, is there anything QA can do to double check this is really disabled?
Keywords: verifyme
Can we consider this verified based on comment 26 from issue 827979?
I don't use Mac day to day but I don't think Growl and Notification Centre are the same thing. In other words, the verification steps might be the same as bug 827979 but you'd need to test *this* bug with Growl removed and Notification Centre enabled.
Removed Growl and verified that Notification Center does not support Firefox on Mac OSX 10.8 using: - Firefox 19 RC (build ID: 20130215130331) - Firefox 20 beta 5 (Build ID:20130313170052).
Status: RESOLVED → VERIFIED
Keywords: verifyme
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: