From c7bef94c6c62b01493e368837d4c6cdc76233b23 Mon Sep 17 00:00:00 2001 From: "enal@chromium.org" Date: Tue, 21 Jun 2011 18:05:51 +0000 Subject: Not allow compression when requesting multimedia because ffmpeg does not expect compressed files. (It does not make sense to compress audio/video anyways...) http://codereview.chromium.org/7044092 BUG=47381 TEST=http://www.deafmac.org/html5/grinchsample.mp4 (Please clear browser cache, otherwise it can contain original GZIP-ed file requested by old browser) Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=89532 Review URL: http://codereview.chromium.org/7044092 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89858 0039d316-1c4b-4281-b951-d872f2087c98 --- webkit/glue/media/simple_data_source.cc | 9 +++++++++ 1 file changed, 9 insertions(+) (limited to 'webkit/glue/media/simple_data_source.cc') diff --git a/webkit/glue/media/simple_data_source.cc b/webkit/glue/media/simple_data_source.cc index 6647e52..57c51ad 100644 --- a/webkit/glue/media/simple_data_source.cc +++ b/webkit/glue/media/simple_data_source.cc @@ -9,12 +9,16 @@ #include "media/base/filter_host.h" #include "net/base/data_url.h" #include "net/base/load_flags.h" +#include "net/http/http_request_headers.h" #include "net/url_request/url_request_status.h" #include "third_party/WebKit/Source/WebKit/chromium/public/WebKit.h" #include "third_party/WebKit/Source/WebKit/chromium/public/WebKitClient.h" +#include "third_party/WebKit/Source/WebKit/chromium/public/WebString.h" #include "webkit/glue/media/web_data_source_factory.h" #include "webkit/glue/webkit_glue.h" +using WebKit::WebString; + namespace webkit_glue { static const char kDataScheme[] = "data"; @@ -300,6 +304,11 @@ void SimpleDataSource::StartTask() { frame_->setReferrerForRequest(request, WebKit::WebURL()); + // Disable compression, compression for audio/video doesn't make sense... + request.setHTTPHeaderField( + WebString::fromUTF8(net::HttpRequestHeaders::kAcceptEncoding), + WebString::fromUTF8("identity;q=1, *;q=0")); + // This flag is for unittests as we don't want to reset |url_loader| if (!keep_test_loader_) url_loader_.reset(frame_->createAssociatedURLLoader()); -- cgit v1.1