Bug 23331 – LayoutTests/webarchive/test-link-rel-icon.html fails in different ways on buildbots

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[David Kilzer (:ddkilzer)]

* SUMMARY
The webarchive/test-link-rel-icon.html test fails in different ways on different buildbots.

On PPC Tiger <http://build.webkit.org/results/trunk-mac-ppc-release/16131/>:

 				<key>expectedContentLength</key>
-				<integer>0</integer>
+				<integer>1150</integer>

On Intel Leopard(?) <http://build.webkit.org/results/trunk-mac-intel-pixel/1370/>, the entire favicon resource is missing from the webarchive.

Disabling this test for now to keep the buildbots green.

[David Kilzer (:ddkilzer)]

See also Bug 22795 (which added this test) and Bug 21026 (which may fix these issues once its issue is fixed).

[David Kilzer (:ddkilzer)]

(In reply to comment #0)
> Disabling this test for now to keep the buildbots green.

$ git svn dcommit
Committing to http://svn.webkit.org/repository/webkit/trunk ...
	R	LayoutTests/webarchive/test-link-rel-icon.html => LayoutTests/webarchive/test-link-rel-icon.html-disabled
	M	LayoutTests/ChangeLog
Committed r39911

http://trac.webkit.org/changeset/39911

[David Kilzer (:ddkilzer)]

This layout test may work again after <http://trac.webkit.org/changeset/40335>.

[David Kilzer (:ddkilzer)]

Created attachment 30191 [details]
Patch v1

This was reviewed by Brady.

[David Kilzer (:ddkilzer)]

$ git svn dcommit
Committing to http://svn.webkit.org/repository/webkit/trunk ...
	D	LayoutTests/webarchive/test-link-rel-icon.html-disabled
	M	LayoutTests/ChangeLog
	M	LayoutTests/webarchive/test-link-rel-icon-expected.webarchive
	A	LayoutTests/webarchive/test-link-rel-icon.html
Committed r43499

http://trac.webkit.org/changeset/43499

[David Kilzer (:ddkilzer)]

Added webarchive/test-link-rel-icon.html to the gtk Skipped list:

$ git svn dcommit
Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	LayoutTests/ChangeLog
	M	LayoutTests/platform/gtk/Skipped
Committed r43501

http://trac.webkit.org/changeset/43501