Relanding this as the earlier attempt broke chrome frame tests.

Relanding this as the earlier attempt broke chrome frame tests.

Register the RenderViewHost instance created when a desktop notification bubble is displayed in CF
with the AutomationResourceMessagefilter. This ensures that any HTTP requests issued by this bubble
go through the host network stack implemenation. I also fixed a bug in the pending http jobs resumption
code in the AutomationResourceMessagefilter which could cause a job to be registered in the old filter
and the new filter for a while.

Fixes bug http://code.google.com/p/chromium/issues/detail?id=64251

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

[robertshield, 2011-10-28 19:54:23]

LGTM

http://codereview.chromium.org/8414030/diff/1/chrome/browser/automation/autom...
File chrome/browser/automation/automation_resource_message_filter.cc (right):

http://codereview.chromium.org/8414030/diff/1/chrome/browser/automation/autom...
chrome/browser/automation/automation_resource_message_filter.cc:268: // The tab
handle may have changed in the following scenario:-
nit: remove the -

http://codereview.chromium.org/8414030/diff/1/chrome/browser/automation/autom...
chrome/browser/automation/automation_resource_message_filter.cc:276: // external
tab which is being destroyed
Please clarify the last line of the comment. Does it mean that 
automation_details_iter->second.tab_handle is the tab that is being destroyed?
If so, consider
"Replace the handle of the external tab being destroyed with the new one that is
being created."

[ananta, 2011-10-28 21:13:06]

http://codereview.chromium.org/8414030/diff/1/chrome/browser/automation/autom...
File chrome/browser/automation/automation_resource_message_filter.cc (right):

http://codereview.chromium.org/8414030/diff/1/chrome/browser/automation/autom...
chrome/browser/automation/automation_resource_message_filter.cc:268: // The tab
handle may have changed in the following scenario:-
On 2011/10/28 19:54:23, robertshield wrote:
> nit: remove the -

Done.

http://codereview.chromium.org/8414030/diff/1/chrome/browser/automation/autom...
chrome/browser/automation/automation_resource_message_filter.cc:276: // external
tab which is being destroyed
On 2011/10/28 19:54:23, robertshield wrote:
> Please clarify the last line of the comment. Does it mean that 
> automation_details_iter->second.tab_handle is the tab that is being destroyed?
> If so, consider
> "Replace the handle of the external tab being destroyed with the new one that
is
> being created."

Done.