Cancel prerenders for pages that call window.print()

chrome/browser/chrome_content_browser_client.cc

Index: chrome/browser/chrome_content_browser_client.cc
diff --git a/chrome/browser/chrome_content_browser_client.cc b/chrome/browser/chrome_content_browser_client.cc
index b7ce694efd6419e642304cbdf1c234454e4787f0..3392c32313de97f86de7db2047a692f510731ccd 100644
--- a/chrome/browser/chrome_content_browser_client.cc
+++ b/chrome/browser/chrome_content_browser_client.cc
@@ -16,6 +16,7 @@
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/google/google_util.h"
 #include "chrome/browser/prefs/pref_service.h"
+#include "chrome/browser/prerender/prerender_message_filter.h"
 #include "chrome/browser/printing/printing_message_filter.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/renderer_host/chrome_render_message_filter.h"
@@ -76,6 +77,8 @@ void ChromeContentBrowserClient::BrowserRenderProcessHostCreated(
   Profile* profile = host->profile();
   host->channel()->AddFilter(new ChromeRenderMessageFilter(
       id, profile, profile->GetRequestContextForRenderProcess(id)));
+  host->channel()->AddFilter(new prerender::PrerenderMessageFilter(

[cbentzel, 2011/05/13 12:00:28]

Does ordering of filters matter? I'm guessing it does in this case, where you
want the prerendermessagefilter to intercept the printing IPCs before
PrintingMessageFilter does.

If so, please comment on the ordering dependency.

[dominich, 2011/05/13 15:05:33]

Done.

+      host, profile->GetPrerenderManager()));
   host->channel()->AddFilter(new PrintingMessageFilter());
   host->channel()->AddFilter(
       new SearchProviderInstallStateMessageFilter(id, profile));