js_ReportOutOfMemory needs to support enhanced error reporting

RESOLVED FIXED

Status

()

Product:
Core
Component:
JavaScript Engine
Status:
RESOLVED FIXED
Reported:
13 years ago
Modified:
12 years ago

People

(Reporter: daumling, Unassigned)

Tracking

({fixed1.8.1})

Version:
Trunk
Platform:
x86
Windows XP
Keywords:
fixed1.8.1
Points:
---
Bug Flags:
blocking1.8.1 -
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Patch replaces call to js_GetErrorMessage callback with call to js_GetLocalizedErrorMessage()
3.16 KB, patch
brendan
: review+
mtschrep
: approval1.8.1+
Details | Diff | Splinter Review
(Reporter)

Description

•
13 years ago 
This is a reminder bug. It needs to be fixed as soon as 215173 has been integrated. I'll supply a patch then.
 (Reporter)

Updated

•
13 years ago
Status: NEW → ASSIGNED
 (Reporter)

Comment 1

•
13 years ago 
Created attachment 205024 [details] [diff] [review]
Patch replaces call to js_GetErrorMessage callback with call to js_GetLocalizedErrorMessage()
Attachment #205024 - Flags: review?(brendan)

Comment 2

•
13 years ago 
Comment on attachment 205024 [details] [diff] [review]
Patch replaces call to js_GetErrorMessage callback with call to js_GetLocalizedErrorMessage()

Nice, thanks.  I'll land after testing.

/be
Attachment #205024 - Flags: review?(brendan) → review+

Comment 3

•
13 years ago 
Fixed on trunk.  Another one to pick up for js1.6rc1.

/be
Blocks: 309169
Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED

Updated

•
13 years ago
Flags: testcase-

Updated

•
13 years ago
Flags: blocking1.8.1?

Updated

•
13 years ago
Attachment #205024 - Flags: approval1.8.1+

Updated

•
13 years ago
Flags: blocking1.8.1? → blocking1.8.1-

Comment 4

•
12 years ago 
Fixed on the 1.8 branch by the js1.7 landing.
Keywords: fixed1.8.1

Comment 5

•
12 years ago 
not fixed on 1.8.0, not making js16
No longer blocks: 309169
You need to log in before you can comment on or make changes to this bug.