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

VERIFIED FIXED in mozilla0.9.2

Status

()

Core
Selection
VERIFIED FIXED
17 years ago
17 years ago

People

(Reporter: Heikki Toivonen (remove -bugzilla when emailing directly), 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

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

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

Comment 2

17 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

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

Comment 4

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

Updated

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

Comment 5

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

sending away for a=

anthonyd

Updated

17 years ago
Blocks: 83989

Comment 6

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

Updated

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

Comment 7

17 years ago
fix checked in
anthonyd
(Assignee)

Comment 8

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

Comment 9

17 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.