Closed
Bug 1501479
Opened 6 years ago
Closed 6 years ago
Add even _more_ ClearDocumentDependentSlots crash instrumentation
Categories
(Core :: DOM: Core & HTML, enhancement)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla65
Tracking | Status | |
---|---|---|
firefox65 | --- | fixed |
People
(Reporter: bzbarsky, Assigned: bzbarsky)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
(deleted),
patch
|
mccr8
:
review+
|
Details | Diff | Splinter Review |
What I'm seeing is us failing to allocate objects, but for no good reason... :(
Assignee | ||
Comment 1•6 years ago
|
||
Attachment #9019529 -
Flags: review?(continuation)
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Comment 2•6 years ago
|
||
I actually hit one of these crashes on my Windows machine at home. Sadly I could not reproduce it.
Comment 3•6 years ago
|
||
Comment on attachment 9019529 [details] [diff] [review]
Add instrumentation to figure out why Document.prototype creation is failing
Review of attachment 9019529 [details] [diff] [review]:
-----------------------------------------------------------------
::: dom/bindings/Codegen.py
@@ +2973,4 @@
> # if we don't need to create anything, why are we generating this?
> assert needInterfaceObject or needInterfacePrototypeObject
>
> + def maybecrash(reason):
Should this be "maybeCrash"? I don't know what the Python style is in this file. Also a name like "crashIfDocument" would be more descriptive. On the other hand, it would be less adaptable if you need to add yet more checks later.
@@ +2974,5 @@
> assert needInterfaceObject or needInterfacePrototypeObject
>
> + def maybecrash(reason):
> + if self.descriptor.name == "Document":
> + return 'MOZ_CRASH("%s");\n' % reason
Should the bug number go in here?
Attachment #9019529 -
Flags: review?(continuation) → review+
Assignee | ||
Comment 4•6 years ago
|
||
> Should this be "maybeCrash"?
Then I couldn't use "maybeCrash" for the variable name.
> Should the bug number go in here?
Yeah, I guess that's a good idea. At this point I'm just querying for crash reports with ClearDocumentDepemdentSlots in the stack, but having the bug# in there is good.
Pushed by bzbarsky@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f7910a01b533
Add instrumentation to figure out why Document.prototype creation is failing. r=mccr8
Comment 6•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•