nsROCSSPrimitiveValue::SetFoo() methods should return an nsresult

NEW
Unassigned

Status

()

Core
DOM: CSS Object Model
16 years ago
7 years ago

People

(Reporter: Christopher Aillon (sabbatical, not receiving bugmail), Unassigned)

Tracking

({helpwanted})

Trunk
x86
Linux
helpwanted
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Filing this bug after a conversation with jst on irc.  We currently assume that
all SetFoo methods succeed.  If it fails in an OOM situation, we need to return
an error from there and handle the callers appropriately.  Assigning to Boris
who will be doing similar rework here in the future.
Priority: -- → P2
Target Milestone: --- → mozilla1.1alpha
1.1alpha is frozen.  Unsetting milestone and will retriage in a few days when I
can make a realistic assessment of the situation.
Target Milestone: mozilla1.1alpha → ---
Priority: P2 → P3
Target Milestone: --- → mozilla1.5beta
Created attachment 132323 [details] [diff] [review]
Partial patch

This does not update all callers; someone would need to do that...
Keywords: helpwanted
Priority: P3 → P5
Target Milestone: mozilla1.5beta → Future
No plans to work on this any time in the foreseeable future, so to default owner.
Assignee: bz-vacation → general
Priority: P5 → --
Target Milestone: Future → ---
Assignee: general → nobody
QA Contact: ian → general
QA Contact: general → style-system
You need to log in before you can comment on or make changes to this bug.