nsHTMLTableAccessible mLayoutHeuristic should not be an nsAutoString

RESOLVED FIXED

Status

()

--
enhancement
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: timeless, Assigned: timeless)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

9 years ago
Defined as a variable in:
* accessible/src/html/nsHTMLTableAccessible.cpp
o line 1292 -- aDescription = mLayoutHeuristic;
* accessible/src/html/nsHTMLTableAccessible.h
o line 225 -- nsAutoString mLayoutHeuristic; 

Referenced in:
* accessible/src/html/nsHTMLTableAccessible.cpp
o line 1295 -- printf("\nTABLE: %s\n", NS_ConvertUTF16toUTF8(mLayoutHeuristic).get());
o line 1356 -- mLayoutHeuristic = isLayout ? NS_LITERAL_STRING("layout table: ") : NS_LITERAL_STRING("data table: "); \
o line 1357 -- mLayoutHeuristic += NS_LITERAL_STRING(heuristic); return NS_OK; }
(Assignee)

Comment 1

9 years ago
Created attachment 391845 [details] [diff] [review]
patch
Assignee: nobody → timeless
Status: NEW → ASSIGNED
Attachment #391845 - Flags: review?(marco.zehe)

Updated

9 years ago
Attachment #391845 - Flags: review?(marco.zehe) → review+
(Assignee)

Comment 2

9 years ago
http://hg.mozilla.org/mozilla-central/rev/2a87d1efd189
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.