diff options
author | piman@chromium.org <piman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-11-22 05:42:36 +0000 |
---|---|---|
committer | piman@chromium.org <piman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-11-22 05:42:36 +0000 |
commit | 0157282494c1096cef55f04d7b8936c7db022679 (patch) | |
tree | 33d30478c95e60b4ba629b098db06ae2d1418278 /webkit/glue/resource_loader_bridge.h | |
parent | 745816be61e207da7cef1f839591bfb35bd15fd3 (diff) | |
download | chromium_src-0157282494c1096cef55f04d7b8936c7db022679.zip chromium_src-0157282494c1096cef55f04d7b8936c7db022679.tar.gz chromium_src-0157282494c1096cef55f04d7b8936c7db022679.tar.bz2 |
Move some webkit_glue embedder functions into WebKitPlatformSupport virtual methods
This moves the functions into a pattern that is component-friendly.
BUG=98755
TEST=chrome test_shell test_shell_tests browser_tests DumpRenderTree
Review URL: http://codereview.chromium.org/8602002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@111101 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/glue/resource_loader_bridge.h')
-rw-r--r-- | webkit/glue/resource_loader_bridge.h | 21 |
1 files changed, 9 insertions, 12 deletions
diff --git a/webkit/glue/resource_loader_bridge.h b/webkit/glue/resource_loader_bridge.h index 6711039..da0a36e 100644 --- a/webkit/glue/resource_loader_bridge.h +++ b/webkit/glue/resource_loader_bridge.h @@ -4,8 +4,7 @@ // // The intent of this file is to provide a type-neutral abstraction between // Chrome and WebKit for resource loading. This pure-virtual interface is -// implemented by the embedder, which also provides a factory method Create -// to instantiate this object. +// implemented by the embedder. // // One of these objects will be created by WebKit for each request. WebKit // will own the pointer to the bridge, and will delete it when the request is @@ -197,7 +196,8 @@ struct ResourceResponseInfo { class ResourceLoaderBridge { public: - // Structure used when calling ResourceLoaderBridge::Create(). + // Structure used when calling + // WebKitPlatformSupportImpl::CreateResourceLoader(). struct RequestInfo { RequestInfo(); ~RequestInfo(); @@ -329,16 +329,10 @@ class ResourceLoaderBridge { const base::Time& completion_time) = 0; }; - // use Create() for construction, but anybody can delete at any time, - // INCLUDING during processing of callbacks. + // use WebKitPlatformSupportImpl::CreateResourceLoader() for construction, but + // anybody can delete at any time, INCLUDING during processing of callbacks. virtual ~ResourceLoaderBridge(); - // Call this method to make a new instance. - // - // For HTTP(S) POST requests, the AppendDataToUpload and AppendFileToUpload - // methods may be called to construct the body of the request. - static ResourceLoaderBridge* Create(const RequestInfo& request_info); - // Call this method before calling Start() to append a chunk of binary data // to the request body. May only be used with HTTP(S) POST requests. virtual void AppendDataToUpload(const char* data, int data_len) = 0; @@ -394,7 +388,10 @@ class ResourceLoaderBridge { virtual void UpdateRoutingId(int new_routing_id) = 0; protected: - // construction must go through Create() + // Construction must go through + // WebKitPlatformSupportImpl::CreateResourceLoader() + // For HTTP(S) POST requests, the AppendDataToUpload and AppendFileToUpload + // methods may be called to construct the body of the request. ResourceLoaderBridge(); private: |