[Webkit-unassigned] [Bug 32437] [Qt] [LayoutTestController]Output of document.write() is vanished by waitUntilDone()/notifyDone()

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=32437


robert <robert at roberthogan.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #44761|                            |review?
               Flag|                            |




--- Comment #4 from robert <robert at roberthogan.net>  2009-12-13 13:55:27 PST ---
Created an attachment (id=44761)
 --> (https://bugs.webkit.org/attachment.cgi?id=44761)
Patch

This fixes it for me. It also fixes the following from the skipped list on my
machine:

 fast/forms/textarea-linewrap-dynamic.html
 fast/forms/textarea-setvalue-submit.html
 fast/forms/textarea-hard-linewrap-empty.html
 fast/forms/submit-to-url-fragment.html
 http/tests/misc/percent-sign-in-form-field-name.html
 http/tests/security/escape-form-data-field-names.html

The 'but' is that these 2 need to be added to the skipped list:

 http/tests/xmlhttprequest/access-control-basic-denied-preflight-cache.html
 svg/custom/use-instanceRoot-as-event-target.xhtml

I think the bug with the drt was masking deeper problems with these two. In the
first case, the test returns a 'SUCCESS' *and* an exception. I believe up to
now it wasn't catching this exception because notifyDone shut down the page
before it had finished loading.

In the second case, notifyDone seems to get called three times by WebCore,
hence the three sets of identical results for 'test 10'. So in a sense, the
test is actually three times more successful than before. ;-) Again, the
behaviour of the DRT would have been to shut down the still-loading page before
receiving these two extra bits of output.

Csaba, can you see if you can reproduce my success with the above tests with
this patch?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.