Stop resource loading for WebCore::MediaDocument...

webkit/glue/webframeloaderclient_impl.cc

Index: webkit/glue/webframeloaderclient_impl.cc
===================================================================
--- webkit/glue/webframeloaderclient_impl.cc	(revision 22447)
+++ webkit/glue/webframeloaderclient_impl.cc	(working copy)
@@ -982,6 +982,14 @@
       d->DidReceiveDocumentData(webframe_, data, length);
   }
 
+  // If we are sending data to WebCore::MediaDocument, we should stop here

[darin (slow to review), 2009/08/14 00:15:37]

One more tweak I would make:  move this code to the top of the function and
return after cancelling the main resource load.  There is no reason to call
DidReceiveDocumentData in this case, right?  Or, does that initial bit of
document data cause the isMediaDocument flag to be set?

[Alpha Left Google, 2009/08/14 00:32:13]

Yes, it is the first bit of data that triggers the isMediaDocument(), if we move
this block up and return it will actually cause rendering problem (a red page is
displayed). 

[darin (slow to review), 2009/08/14 07:30:44]

OK, that makes sense.  Thanks!

+  // and cancel the request.
+  if (webframe_->frame()->document() &&
+      webframe_->frame()->document()->isMediaDocument()) {
+    loader->cancelMainResourceLoad(
+        pluginWillHandleLoadError(loader->response()));
+  }
+
   // The plugin widget could have been created in the webframe_->DidReceiveData
   // function.
   if (plugin_widget_.get()) {