Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(134)

Unified Diff: chrome/browser/automation/automation_resource_message_filter.cc

Issue 8414038: Fix ChromeFrame test failures on IE8 which occurred because we did not handle the case (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 9 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/automation/automation_resource_message_filter.cc
===================================================================
--- chrome/browser/automation/automation_resource_message_filter.cc (revision 107804)
+++ chrome/browser/automation/automation_resource_message_filter.cc (working copy)
@@ -262,7 +262,11 @@
RenderViewMap::iterator automation_details_iter(
filtered_render_views_.Get().find(renderer_key));
- if (automation_details_iter != filtered_render_views_.Get().end()) {
+ // We need to match the renderer key and the AutomationResourceMessageFilter
+ // instances. If the filter instances are different it means that a new
+ // automation channel (External host process) was created for this tab.
+ if (automation_details_iter != filtered_render_views_.Get().end() &&
+ 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:-
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698