Closed
Bug 891694
Opened 11 years ago
Closed 11 years ago
Dump more of long JS strings when writing a GC log
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
People
(Reporter: justin.lebar+bug, Assigned: justin.lebar+bug)
References
(Blocks 1 open bug)
Details
(Whiteboard: [LeoVB+])
Attachments
(2 files)
(deleted),
patch
|
n.nethercote
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
n.nethercote
:
review+
|
Details | Diff | Splinter Review |
Right now we dump at most 1024 chars per line when doing a GC log.
1024 chars isn't necessarily enough to decode a data URI of a PNG into something recognizable. I'd like to make this much longer -- maybe 32kb.
Assignee | ||
Comment 1•11 years ago
|
||
Attachment #773056 -
Flags: review?(n.nethercote)
Assignee | ||
Comment 2•11 years ago
|
||
I'm a bit concerned about allocating this much stack memory; I don't want us to overflow our stack. Maybe it would be better to allocate it on the heap.
But it probably doesn't matter; the worst that happens is we crash, and this doesn't get triggered unless you ask for it.
Updated•11 years ago
|
Attachment #773056 -
Flags: review?(n.nethercote) → review+
Assignee | ||
Comment 3•11 years ago
|
||
Comment 4•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
Comment 6•11 years ago
|
||
Needs a branch-specific patch for uplift.
status-b2g18:
--- → affected
status-b2g18-v1.0.0:
--- → wontfix
status-b2g18-v1.0.1:
--- → wontfix
status-b2g-v1.1hd:
--- → affected
status-firefox23:
--- → wontfix
status-firefox24:
--- → wontfix
status-firefox25:
--- → fixed
Flags: needinfo?(justin.lebar+bug)
Keywords: branch-patch-needed
Assignee | ||
Comment 7•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Attachment #774898 -
Attachment description: Dump more of long strings when writing a GC log. → Patch for b2g18, v1
Attachment #774898 -
Flags: review?(n.nethercote)
Assignee | ||
Updated•11 years ago
|
Flags: needinfo?(justin.lebar+bug)
Updated•11 years ago
|
Attachment #774898 -
Flags: review?(n.nethercote) → review+
Updated•11 years ago
|
Keywords: branch-patch-needed
Assignee | ||
Comment 8•11 years ago
|
||
Updated•11 years ago
|
Comment 9•11 years ago
|
||
Updated•11 years ago
|
Whiteboard: [LeoVB+]
You need to log in
before you can comment on or make changes to this bug.
Description
•