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)
Tracking
()
VERIFIED
FIXED
mozilla20
People
(Reporter: jaas, Assigned: jaas)
Details
Attachments
(1 file)
1.23 KB,
patch
|
dougt
:
review+
bajaj
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
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.
Attachment #695273 -
Flags: review?(doug.turner)
Comment 2•12 years ago
|
||
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.
Updated•12 years ago
|
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?
m-c try run, os x only:
https://tbpl.mozilla.org/?tree=Try&rev=118b7b2faaa3
pushed to mozilla-inbound
http://hg.mozilla.org/integration/mozilla-inbound/rev/c0ba38205c41
Comment 7•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla20
Updated•12 years ago
|
Attachment #695273 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
pushed to mozilla-aurora
http://hg.mozilla.org/releases/mozilla-aurora/rev/679426677f68
status-firefox19:
--- → fixed
status-firefox20:
--- → fixed
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
Comment 10•12 years ago
|
||
Can we consider this verified based on comment 26 from issue 827979?
Comment 11•12 years ago
|
||
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.
Comment 12•12 years ago
|
||
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.
Description
•