Closed Bug 130410 Opened 22 years ago Closed 22 years ago

test feedback for bug 110544 - setting comment node text is slow

Categories

(Core :: DOM: Core & HTML, defect)

x86
Windows 2000
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: hjtoi-bugzilla, Assigned: hjtoi-bugzilla)

References

Details

Drivers asked for more testing on bug 110544 before giving approval. I will
provide a test binary, and ask people to comment here on their findings.
Here is the executable, internal link first:

http://green.nscp.aoltw.net/heikki/mozilla-2002-03-11-13.tgz

Build done at about 2002-03-11 13:00

Loan, could you please put this on mozilla.org somewhere so that I can get
external testers for this binary as well?

I have r & sr for the bug, but drivers want to see at least one day of testing
for this build so I am kinda anxious to get it done as soon as possible,
especially since I have been working on the perf bug for over 3 months...
in the below mentioned URL i see the word "INCORRECTLY" in all the tests. Is 
this a failure or is there something i am missing here.


http://lxr.mozilla.org/mozilla/source/htmlparser/tests/logparse/comments.html
Huh, what build are you using? And do you see the word incorrectly in the
colored boxes, or in the sentences describing the tests? That test is most
certainly working correctly for me... If you have a local copy where you have
remoevd the doctype then you might see different results (we have strict and
quirks mode comment parsing).
This is a Windows binary. If you have tar and gzip available, use them to unpack
the zip:

tar zxf mozilla-2002-03-11-13.tgz

WinZip might work, but I have not tested.

Once you have the binary, I'd like you to try the sites you normally go to, and
a few extra as well. If you know of some comment tests, please try them as well.
While testing, look for:

* Hang
* Incorrect page (missing content or too much content)
* Any other problems
yeah, winzip works too, I see the following message in the box "This browser 
parses comments correctly." All the four tests are passing
I ran the browser and international browser smokettests on this. I didn't see 
anyhting out of the ordinary in comparison to recent builds.  Looks good.
testing done, fixed.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Component: DOM: Core → DOM: Core & HTML
QA Contact: stummala → general
You need to log in before you can comment on or make changes to this bug.