nsRange::SurroundContents() : not all control paths return value

VERIFIED FIXED in mozilla0.9.2

Status

()

VERIFIED FIXED
18 years ago
18 years ago

People

(Reporter: hjtoi-bugzilla, Assigned: anthonyd)

Tracking

Trunk
mozilla0.9.2
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: FIX IN HAND; waiting for checkin)

Attachments

(1 attachment)

Forgot a return statement?
(Assignee)

Comment 1

18 years ago
setting to 0.9.3.  SurroundContents probably needs to be re-written.

anthonyd
Target Milestone: --- → mozilla0.9.3
(Assignee)

Comment 2

18 years ago
Im attaching a patch to recitify this, the patch file I am attaching has more in 
it than just the fix for this.

anthonyd
Status: NEW → ASSIGNED
Whiteboard: FIX IN HAND
(Assignee)

Comment 3

18 years ago
Created attachment 37473 [details] [diff] [review]
new patch
(Assignee)

Comment 4

18 years ago
moving to 0.9.2 to get this in.
Target Milestone: mozilla0.9.3 → mozilla0.9.2

Updated

18 years ago
Whiteboard: FIX IN HAND → FIX IN HAND need r=, sr=
(Assignee)

Comment 5

18 years ago
r= from mjudge, and sr= from kin.

sending away for a=

anthonyd

Updated

18 years ago
Blocks: 83989

Comment 6

18 years ago
a= asa@mozilla.org for checkin to the trunk.
(on behalf of drivers)

Updated

18 years ago
Whiteboard: FIX IN HAND need r=, sr= → FIX IN HAND; waiting for checkin
(Assignee)

Comment 7

18 years ago
fix checked in
anthonyd
(Assignee)

Comment 8

18 years ago
damnit, really its in!
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Comment 9

18 years ago
Verified fix checked into lxr.mozilla.org
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.