Remove Chrome Frame hacks from net tests.

Remove Chrome Frame hacks from net tests.

BUG=316496, 317432
R=wtc@chromium.org
CC=tzik@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=245262

[wtc, 2014-01-15 00:09:22]

Patch set 1 LGTM. Thanks.

https://codereview.chromium.org/138263003/diff/1/net/url_request/url_request_...
File net/url_request/url_request_unittest.cc (right):

https://codereview.chromium.org/138263003/diff/1/net/url_request/url_request_...
net/url_request/url_request_unittest.cc:1958: // This is to work around
mysterious failures in chrome_frame_net_tests. See:

This also seems related to ChromeFrame. Should this be changed?

https://codereview.chromium.org/138263003/diff/1/net/url_request/url_request_...
net/url_request/url_request_unittest.cc:4804: // TODO(tzik): Remove this #if
after we stop supporting Chrome Frame.

Should we also remove this?

[grt (UTC plus 2), 2014-01-15 14:29:10]

Thanks, PTAL.

https://codereview.chromium.org/138263003/diff/1/net/url_request/url_request_...
File net/url_request/url_request_unittest.cc (right):

https://codereview.chromium.org/138263003/diff/1/net/url_request/url_request_...
net/url_request/url_request_unittest.cc:1958: // This is to work around
mysterious failures in chrome_frame_net_tests. See:
On 2014/01/15 00:09:23, wtc wrote:
> 
> This also seems related to ChromeFrame. Should this be changed?

I'm not sure if the local server is still useful to make the tests more solid.
I've filed http://crbug.com/334634 for the original author to clean it up.

https://codereview.chromium.org/138263003/diff/1/net/url_request/url_request_...
net/url_request/url_request_unittest.cc:4804: // TODO(tzik): Remove this #if
after we stop supporting Chrome Frame.
On 2014/01/15 00:09:23, wtc wrote:
> 
> Should we also remove this?

Done. I wonder why I didn't remove it before...

[wtc, 2014-01-15 19:44:31]

Patch set 2 LGTM!

[commit-bot: I haz the power, 2014-01-15 21:30:33]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/grt@chromium.org/138263003/160001

[commit-bot: I haz the power, 2014-01-16 06:28:29]

Retried try job too often on win_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2014-01-16 14:48:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/grt@chromium.org/138263003/160001

[commit-bot: I haz the power, 2014-01-16 18:27:40]

Change committed as 245262