Extraneous .Truncate() calls in nsGenericHTMLElement::GetAttr()

RESOLVED FIXED in mozilla1.4alpha

Status

()

--
minor
RESOLVED FIXED
16 years ago
16 years ago

People

(Reporter: emaijala+moz, Assigned: emaijala+moz)

Tracking

Trunk
mozilla1.4alpha
x86
All
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [whiitebox])

Attachments

(2 attachments)

(Assignee)

Description

16 years ago
Stumbled on some extraneous string truncates in nsGenericHTMLElement::GetAttr().
Patch coming up.
(Assignee)

Comment 1

16 years ago
Oops, reassigning.
Assignee: frame → ere
(Assignee)

Comment 2

16 years ago
Created attachment 115046 [details] [diff] [review]
Small patch to remove the extra calls

This patch removes the extra calls.
(Assignee)

Updated

16 years ago
Severity: normal → minor
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.4alpha
(Assignee)

Comment 3

16 years ago
Created attachment 115052 [details] [diff] [review]
Same with more context
Comment on attachment 115052 [details] [diff] [review]
Same with more context

r=smontagu
Attachment #115052 - Flags: review+
(Assignee)

Comment 5

16 years ago
Comment on attachment 115052 [details] [diff] [review]
Same with more context

And now a semi-random selection of sr= candidate. Please?
Attachment #115052 - Flags: superreview?(bugmail)
Comment on attachment 115052 [details] [diff] [review]
Same with more context

i'm not a superreviewer
Attachment #115052 - Flags: superreview?(bugmail)
(Assignee)

Comment 7

16 years ago
Comment on attachment 115052 [details] [diff] [review]
Same with more context

Oops, sorry. A new try..
Attachment #115052 - Flags: superreview?(jst)
Comment on attachment 115052 [details] [diff] [review]
Same with more context

sr=jst
Attachment #115052 - Flags: superreview?(jst) → superreview+
(Assignee)

Comment 9

16 years ago
Checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Updated

16 years ago
Whiteboard: [whiitebox]
You need to log in before you can comment on or make changes to this bug.