Closed Bug 71399 Opened 24 years ago Closed 24 years ago

parser causes string assertion loading NYTimes

Categories

(Core :: DOM: HTML Parser, defect)

Product:
Core
Component:
DOM: HTML Parser
Platform:
x86
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: dbaron, Assigned: harishd)

References

(
URL
)

Details

When loading http://www.nytimes.com/ I see the assertion ###!!! ASSERTION: |First()| on an empty string: 'Length()>0', file ../../dist/include/string/nsAReadableString.h, line 630 ###!!! Break: at file ../../dist/include/string/nsAReadableString.h, line 630 which is caused by nsHTMLTokenizer.cpp, line 583. The code if((kForwardSlash==key.First()) && (0==text.Length())){ should probably be replaced by if (key.Length() && (kForwardSlash==key.First()) && (0==text.Length())){
ya I have fixed that in my tree.
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
FIXED.
*** Bug 71459 has been marked as a duplicate of this bug. ***
Verified on: build: 2001-03-29-09-Mtrunk Platform: Win NT The url loads fine.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.