Closed Bug 595273 Opened 11 years ago Closed 11 years ago
Bug 573588 added dom_notification.xpt and a bunch of DOM tests recently, and we are not packaging it. We should just include it.
Comment on attachment 474142 [details] [diff] [review] Patch v1.0 Just add it. rs+ if it fixes tests, but I somehow think this depends on doorhangers and real navigator.xul changes from a skim of the bug that added this.
gah, and I hoped I could escape looking into this review more deeply. I probably will not be able to do any meaningful work like digging into bugs until Monday.
Comment on attachment 474142 [details] [diff] [review] Patch v1.0 Just add it. Hmm this is not actually app-specific in any way. [yet] Please open a new bug to track Bug 594543
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
> rs+ if it fixes tests, but I somehow think this depends on doorhangers and real > navigator.xul changes from a skim of the bug that added this. Reading the patches that got checked in, everything else is either backend or Fennec specific. The front end UI changes to Firefox were not checked in (r-) but deferred to after Gecko 2.0.
> Please open a new bug to track Bug 594543 I filed Bug 595437 [Tracking] Implement Desktop Notifications in SeaMonkey
This did fix (bug 595024 comment 3) mochitests-3/5 failures. V.Fixed, per http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1284179343.1284181265.28030.gz Linux comm-central-trunk debug test mochitests-3/5 on 2010/09/10 21:29:03
Severity: normal → major
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.