diff options
| author | annacc@chromium.org <annacc@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-12-16 18:49:52 +0000 |
|---|---|---|
| committer | annacc@chromium.org <annacc@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-12-16 18:49:52 +0000 |
| commit | 32ba25f8130e85dc5189b06490156c1df4f0cee8 (patch) | |
| tree | e2427dbc87efb9289315dff1cb416c8f823aac7a /webkit/tools | |
| parent | 1cf1e69b1ea029d5d99a9b25d9e505cb56170630 (diff) | |
| download | chromium_src-32ba25f8130e85dc5189b06490156c1df4f0cee8.zip chromium_src-32ba25f8130e85dc5189b06490156c1df4f0cee8.tar.gz chromium_src-32ba25f8130e85dc5189b06490156c1df4f0cee8.tar.bz2 | |
Separate BufferedDataSource and BufferedResourceLoader into two files.
It's time to finally separate the huge monster files buffered_data_source.[h/cc] into two.
ericroman: There are some variables and short methods that both BufferedDataSource and BufferedResourceLoader rely on and that I have moved to src/net/http/http_util.h. Could you please verify that this is an ok place to put them?
Also, appcache issues have been resolved (and approved by michaeln) so I've removed those comments.
BUG=None.
TEST=Compiles.
Review URL: http://codereview.chromium.org/5756004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69429 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/tools')
| -rw-r--r-- | webkit/tools/test_shell/test_shell.gypi | 3 | ||||
| -rw-r--r-- | webkit/tools/test_shell/test_webview_delegate.cc | 7 |
2 files changed, 1 insertions, 9 deletions
diff --git a/webkit/tools/test_shell/test_shell.gypi b/webkit/tools/test_shell/test_shell.gypi index 79191c6..0e46321 100644 --- a/webkit/tools/test_shell/test_shell.gypi +++ b/webkit/tools/test_shell/test_shell.gypi @@ -396,8 +396,7 @@ '../../glue/glue_serialize_unittest.cc', '../../glue/iframe_redirect_unittest.cc', '../../glue/media/buffered_data_source_unittest.cc', - '../../glue/media/media_resource_loader_bridge_factory_unittest.cc', - '../../glue/media/mock_media_resource_loader_bridge_factory.h', + '../../glue/media/buffered_resource_loader_unittest.cc', '../../glue/media/simple_data_source_unittest.cc', '../../glue/mimetype_unittest.cc', '../../glue/mock_resource_loader_bridge.h', diff --git a/webkit/tools/test_shell/test_webview_delegate.cc b/webkit/tools/test_shell/test_webview_delegate.cc index b791cf2..3f72f2c 100644 --- a/webkit/tools/test_shell/test_webview_delegate.cc +++ b/webkit/tools/test_shell/test_webview_delegate.cc @@ -58,9 +58,6 @@ #include "third_party/WebKit/WebKit/chromium/public/WebWindowFeatures.h" #include "webkit/appcache/web_application_cache_host_impl.h" #include "webkit/glue/glue_serialize.h" -#include "webkit/glue/media/buffered_data_source.h" -#include "webkit/glue/media/media_resource_loader_bridge_factory.h" -#include "webkit/glue/media/simple_data_source.h" #include "webkit/glue/media/video_renderer_impl.h" #include "webkit/glue/plugins/webplugin_impl.h" #include "webkit/glue/plugins/plugin_list.h" @@ -738,10 +735,6 @@ WebMediaPlayer* TestWebViewDelegate::createMediaPlayer( scoped_ptr<media::FilterCollection> collection( new media::FilterCollection()); - // TODO(annacc): do we still need appcache_host? http://crbug.com/65135 - // appcache::WebApplicationCacheHostImpl* appcache_host = - // appcache::WebApplicationCacheHostImpl::FromFrame(frame); - scoped_refptr<webkit_glue::VideoRendererImpl> video_renderer( new webkit_glue::VideoRendererImpl(false)); collection->AddVideoRenderer(video_renderer); |