Yet another scrape atttempt

Scrape search definitions from forms that have onsubmit handlers. The scraping is done after submit events are handled by the page DOM so doing this is safe.

Adds test infrastructure for determining that scraping occurs on submit:
- allow testserver to be configured to serve pages from / on the server
- provide a ui test util that navigates and waits for N subsequent redirections/navigations before returning control to the test to handle automated submission

Eric, please review the test server changes.
Scott, please look over everything else.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=13444

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=13491

[eroman, 2009-04-09 04:12:02]

LGTM on testserver.py.

http://codereview.chromium.org/62145/diff/12/1006
File net/tools/testserver/testserver.py (right):

http://codereview.chromium.org/62145/diff/12/1006#newcode568
Line 568: prefix=self.server.file_root_url
can you add some spaces around the "="?

[sky, 2009-04-09 15:25:51]

LGTM with the following nits fixed.

http://codereview.chromium.org/62145/diff/12/1004
File chrome/browser/search_engines/template_url_scraper_unittest.cc (right):

http://codereview.chromium.org/62145/diff/12/1004#newcode77
Line 77: EXPECT_EQ(0, all_urls.size() - prepopulate_urls.size());
nit: use EXPECT_EQ(all_urls.size(), prepopulate_urls.size()). That's slightly
easier to debug if things go wrong.

http://codereview.chromium.org/62145/diff/12/1004#newcode78
Line 78: 
nit: nuke extra line.

http://codereview.chromium.org/62145/diff/12/1003
File chrome/test/in_process_browser_test.cc (right):

http://codereview.chromium.org/62145/diff/12/1003#newcode223
Line 223: // lookup.
could you nuke this todo, I don't think there is anything that needs to be done
here.

http://codereview.chromium.org/62145/diff/12/1002
File chrome/test/in_process_browser_test.h (right):

http://codereview.chromium.org/62145/diff/12/1002#newcode70
Line 70: // Allows subclasses to configure the host mapper.
Could you add that this blocks requests to google.com as by default chrome
startup pings google.com and we don't want that during testing.