|Summary:||Don't test blocking 255.255.255.255 with no port|
|Product:||WebKit||Reporter:||Pam Greene (IRC:pamg) <pam>|
|Component:||Page Loading||Assignee:||Pam Greene (IRC:pamg) <pam>|
|Version:||528+ (Nightly build)|
Description Pam Greene (IRC:pamg) 2008-10-10 13:26:19 PDT
In security/block-test.html, the action taken when trying to load the initial resource (http://255.255.255.255/test.jpg) depends on the network proxy settings of the machine running the test. That nondeterminism should be removed by not testing the case with no ports specified.
Comment 1 Pam Greene (IRC:pamg) 2008-10-10 13:56:56 PDT
Created attachment 24274 [details] Fixed test and Mac + Leopard results Start with port 1 on the initial load rather than no port.
Comment 2 Darin Adler 2008-10-10 14:20:35 PDT
Comment on attachment 24274 [details] Fixed test and Mac + Leopard results This doesn't seem right. We do need a test of the case with no specific port. It's a separate code path, and in the past there have been bugs with it. And this regression test will catch us if we break it again. I assume that people at Google are running into this because of the Google proxy configuration. Or is this a Chrome-specific issue? What possible solutions are there besides removing the test? review- for now
Comment 3 Brett Wilson (Google) 2008-10-10 14:56:22 PDT
Many proxies (such as squid proxies) will send you content with an error message in it rather than reporting a nonexistant domain. Although I think this is stupid, it's pretty common. I don't think it's possible to write a test that depends on the network to return that some address doesn't exist.
Comment 4 Pam Greene (IRC:pamg) 2009-03-31 15:50:14 PDT
Could we split the no-port case into its own test file? That way Chromium (and any other ports subject to the same problem) could skip that one part while still keeping deterministic coverage of all the rest.
Comment 5 Darin Fisher (:fishd, Google) 2009-03-31 17:05:30 PDT
+darin since he might not have seen pam's comment.
Comment 6 Pam Greene (IRC:pamg) 2009-04-02 13:11:28 PDT
Created attachment 29204 [details] split no-port case into separate test For concrete consideration, here's a patch that splits the test as described above, moving the no-port case to a separate file.