Add automation call to wait for multiple navigations.

Add automation call to wait for multiple navigations.

Convert one ErrorPage UI test to use it.

The rest of the tests will require more work.

TEST=Covered by ui_tests.
http://crbug.com/19361, http://crbug.com/19395

[tony, 2009-08-18 20:04:22]

http://codereview.chromium.org/174015/diff/1/7
File chrome/test/automation/automation_messages_internal.h (right):

http://codereview.chromium.org/174015/diff/1/7#newcode86
Line 86: IPC_SYNC_MESSAGE_ROUTED3_1(AutomationMsg_NavigateToURL, int, GURL, int,
I think this is going to break when the reference build runs the tests. 
Instead, we normally just add messages to the end of this file (see the comment
at the top of the file).

[Paweł Hajdan Jr., 2009-08-18 20:58:25]

Fixed.

[tony, 2009-08-18 21:15:17]

LGTM

http://codereview.chromium.org/174015/diff/1003/1009
File chrome/test/automation/automation_messages_internal.h (right):

http://codereview.chromium.org/174015/diff/1003/1009#newcode85
Line 85: // TODO(phajdan.jr): Remove when the reference build gets updated.
Nit: If there are no more callers, maybe update the comment to say the method is
deprecated and to use NavigateToURLBlockUntilNavigationsComplete instead.

[huanr, 2009-08-19 04:05:25]

LGTM

http://codereview.chromium.org/174015/diff/1003/1007
File chrome/browser/renderer_host/resource_dispatcher_host_uitest.cc (right):

http://codereview.chromium.org/174015/diff/1003/1007#newcode151
Line 151: tab->NavigateToURL(server->TestServerPageW(L"files/title2.html"));
Is it safe to switch to NavigateToURL? Based on comments of this test case, the
bug will make NavigateToURL not return. Same for the other case below.

[Paweł Hajdan Jr., 2009-08-19 19:43:38]

I had to revert because it broke reference page cycler tests. After this fix
they will be fine. Please review.

[tony, 2009-08-19 19:48:11]

I see, you had to still send the old IPC.  Yes, LGTM.

[huanr, 2009-08-19 20:03:33]

LGTM