Commit 75fb9ae1 authored by ananta@chromium.org's avatar ananta@chromium.org

Fix the JavascriptWindow open CF tests which failed because the pending render view status was not

propagated correctly.

BUG=ChromeFrame tests should now pass on IE8.
TBR=robertshield
Review URL: http://codereview.chromium.org/8418041

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107832 0039d316-1c4b-4281-b951-d872f2087c98
parent 727915e3
......@@ -269,7 +269,7 @@ void AutomationResourceMessageFilter::RegisterRenderViewInIOThread(
automation_details_iter->second.filter == filter) {
DCHECK_GT(automation_details_iter->second.ref_count, 0);
automation_details_iter->second.ref_count++;
// The tab handle may have changed:-
// The tab handle and the pending status may have changed:-
// 1.A external tab container is being destroyed and a new one is being
// created.
// 2.The external tab container being destroyed receives a RVH created
......@@ -279,6 +279,7 @@ void AutomationResourceMessageFilter::RegisterRenderViewInIOThread(
// We need to replace the handle of the external tab being destroyed with
// the new one that is being created."
automation_details_iter->second.tab_handle = tab_handle;
automation_details_iter->second.is_pending_render_view = pending_view;
} else {
filtered_render_views_.Get()[renderer_key] =
AutomationDetails(tab_handle, filter, pending_view);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment