with rev. 2163, all markup is good except:-
sessionID is being attached to action with addition of ICEfaces libraries (not there when ICEfaces is not present):-
for example:-
with ICEfaces
action="/jsf-systest/faces/taglib/commandButton_param_test.jsp;jsessionid=3c6a61379a67da7000051bf0bce8"
without ICEfaces:-
action="/jsf-systest/faces/taglib/commandButton_param_test.jsp"

also extra <span> tags with id's are being inserted in the ICEfaces test with extra id's and the id's are now preceded by __ for each of these span Id's. None of these are present in the tests without ICEfaces.

Also extra with the ICEfaces tests is the following addition to the markup:-
<script type='text/javascript'>ice.captureSubmit('form01');</script>

These things may be required for push even though ICEfaces

Note that with rev 21163, all facelets tests pass the htmlunit tests.

correction to above*** the sessionid is included with the standard jsf testing (not the ICEfaces pages)
All the "golden" tests show the sessionid as part of the action of the page
The ICEfaces pages do not show this.
One other difference is the autocomplete="off" attribute on the <input> fields for the ViewState input tag that is of type="hidden". This is not present when ICEfaces jars are included.

To clarify, this issue is not specific to JSP, correct? ICEfaces 2.0 only supports Facelets.

changes made by Ted and Deryk to DOMResponseWriter allows jsp/golden tests to all pass.