Bug 21530

Summary: Don't test blocking 255.255.255.255 with no port
Product: WebKit Reporter: Pam Greene (IRC:pamg) <pam>
Component: Page LoadingAssignee: Pam Greene (IRC:pamg) <pam>
Status: RESOLVED FIXED    
Severity: Normal CC: darin
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Fixed test and Mac + Leopard results
darin: review-
split no-port case into separate test darin: review+

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.
Comment 7 Pam Greene (IRC:pamg) 2009-04-02 14:59:16 PDT
Added a ChangeLog entry and landed in r42188.