search

cloudtrends / chromiumembedded

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

Incorrect content_length when loading resource #83

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
The AsyncStart function in browser_resource_loader_bridge.cc contains the
following piece of code:

    long offset = resourceStream->Seek(0, SEEK_END);
    resourceStream->Seek(0, SEEK_SET);

    resource_stream_ = resourceStream;

    ResourceLoaderBridge::ResponseInfo info;
    info.content_length = static_cast<int64>(offset);

However, the documentation for CefStreamReader states that Seek returns
"zero on success and non-zero on failure", not an offset.  This means that
info.content_length is usually being set to 0.

Original issue reported on code.google.com by emerick on 12 Apr 2010 at 6:42

GoogleCodeExporter commented 9 years ago

Original comment by magreenb...@gmail.com on 27 Apr 2010 at 3:39

GoogleCodeExporter commented 9 years ago 
Fix committed as revision 83.

Original comment by magreenb...@gmail.com on 21 Jun 2010 at 6:09

GoogleCodeExporter commented 9 years ago 
magreenblatt, would you please add the binary of this of later revision to the 
downloads?
I'm using the dlls in Delphi and it's working fine except for a nasty crash 
problem which I believe will be fixed by this patch.

Original comment by carlosb...@gmail.com on 8 Jul 2010 at 2:33