Closed Bug 507591 Opened 12 years ago Closed 12 years ago

nsHTMLTableAccessible mLayoutHeuristic should not be an nsAutoString

Categories

(Core :: Disability Access APIs, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: timeless, Assigned: timeless)

References

()

Details

Attachments

(1 file)

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; }
Attached patch patchSplinter Review
Assignee: nobody → timeless
Status: NEW → ASSIGNED
Attachment #391845 - Flags: review?(marco.zehe)
Attachment #391845 - Flags: review?(marco.zehe) → review+
http://hg.mozilla.org/mozilla-central/rev/2a87d1efd189
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.