"Assertion failure: !Failed(), at ErrorResult.h:103" - mozilla::dom::Notification::ShowInternal

RESOLVED FIXED in Firefox 45

Status

()

--
critical
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jruderman, Assigned: bzbarsky)

Tracking

(Blocks: 1 bug, {assertion, testcase})

Trunk
mozilla45
assertion, testcase
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox45 fixed)

Details

Attachments

(3 attachments)

(Reporter)

Description

3 years ago
Created attachment 8696130 [details]
testcase

Assertion failure: !Failed(), at ErrorResult.h:103

This assertion was added recently:

changeset:   https://hg.mozilla.org/mozilla-central/rev/ecb3051bba08
user:        Boris Zbarsky
date:        Fri Nov 20 16:29:41 2015 -0500
summary:     Bug 1224007 part 6.  Change MaybeSetPendingException to set the ErrorResult state to "not failed", just like SuppressException and StealNSResult already do, and assert in the destructor that the ErrorResult is not Failed().
(Reporter)

Comment 1

3 years ago
Created attachment 8696131 [details]
stack
(Assignee)

Comment 2

3 years ago
Created attachment 8696177 [details] [diff] [review]
Make Notification::ShowInternal explicitly suppress the exception from GetPermissionInternal
Attachment #8696177 - Flags: review?(bugs)
(Assignee)

Updated

3 years ago
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED

Comment 4

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/493f9c9164b7
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-firefox45: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
You need to log in before you can comment on or make changes to this bug.