Cancel prerenders that spawn post requests.

content/browser/renderer_host/resource_dispatcher_host.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // See http://dev.chromium.org/developers/design-documents/multi-process-resource-loading
 
 #include "content/browser/renderer_host/resource_dispatcher_host.h"
 
 #include <set>
 #include <vector>
@@ skipping 405 matching lines @@
                               std::make_pair(child_id, route_id),
                               request_data.url,
                               referrer,
                               is_prerendering));
       AbortRequestBeforeItStarts(filter_, sync_result, route_id, request_id);
       return;
     }
     // Otherwise, treat like a normal request, and fall-through.
   }
 
+  // Abort any prerenders that spawn requests that are not GET methods.
+  if (is_prerendering && request_data.method != "GET") {

[cbentzel, 2011/04/29 20:00:30]

HEAD should be allowed. 

OPTIONS could probably be allowed. 

[dominich, 2011/04/29 21:13:52]

Done.

+    CHECK(IsPrerenderingChildRoutePair(child_id, route_id));

[cbentzel, 2011/04/29 20:00:30]

This CHECK is not necessary - is_prerendering is set using the same code. 

[dominich, 2011/04/29 21:13:52]

Done.

+    BrowserThread::PostTask(
+        BrowserThread::UI, FROM_HERE,
+        NewRunnableFunction(
+            prerender::DestroyPreloadForChildRouteIdPairOnUIThread,
+            resource_context.prerender_manager(),
+            std::make_pair(child_id, route_id),
+            prerender::FINAL_STATUS_NOT_GET));
+    AbortRequestBeforeItStarts(filter_, sync_result, route_id, request_id);
+    return;
+  }
 
   // Construct the event handler.
   scoped_refptr<ResourceHandler> handler;
   if (sync_result) {
     handler = new SyncResourceHandler(
         filter_, request_data.url, sync_result, this);
   } else {
     handler = new AsyncResourceHandler(
         filter_, route_id, request_data.url,
         resource_context.host_zoom_map(),
@@ skipping 1553 matching lines @@
   return is_prefetch_enabled_;
 }
 
 // static
 void ResourceDispatcherHost::set_is_prefetch_enabled(bool value) {
   is_prefetch_enabled_ = value;
 }
 
 // static
 bool ResourceDispatcherHost::is_prefetch_enabled_ = false;