cloudtrends / chromiumembedded

Automatically exported from code.google.com/p/chromiumembedded
1 stars 1 forks source link

handleDownloadResponse called for files that are handled by plugins #227

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Launch cefclient.exe
2. Enter a URL for content that will handled by a plugin.  Here are some 
examples that work on my machine:
http://www.spinnerdisc.com/spinner.swf (Flash plugin)
http://research.microsoft.com/pubs/79655/gazelle.pdf (Reader plugin)
http://www.jhepple.com/support/SampleMovies/vcd.mpg (QuickTime plugin)

What is the expected output? What do you see instead?
Expected behavior: The content will load in the browser window
Observed behavior: The content loads in the browser window.  The file is also 
downloaded.  With cefclient, I get a message box indicating successful download.

What version of the product are you using? On what operating system?
I was able to reproduce this with the binary releases of r212 and r195, but not 
with r181.  Based on this window, I'd guess the bug was introduced with change 
http://code.google.com/p/chromiumembedded/source/detail?r=186

Please provide any additional information below.

Original issue reported on code.google.com by mikeyk...@gmail.com on 29 Apr 2011 at 6:03

GoogleCodeExporter commented 9 years ago
It looks like there's a new way to check if a plugin is enabled.  Chrome made a 
similar change a while back:

http://src.chromium.org/viewvc/chrome/trunk/src/content/browser/renderer_host/bu
ffered_resource_handler.cc?r1=71285&r2=72341

Original comment by mikeyk...@gmail.com on 29 Apr 2011 at 10:14

Attachments:

GoogleCodeExporter commented 9 years ago
I've also come across this issue.

HandlwDownloadResponse() is called even if the server sends 
Content-Disposition: inline;

Original comment by powerbf...@gmail.com on 5 May 2011 at 3:34

GoogleCodeExporter commented 9 years ago
Thanks, committed as revision 225 with correct line wrapping.

Original comment by magreenb...@gmail.com on 9 May 2011 at 3:12