Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(211)

Unified Diff: webkit/child/resource_loader_bridge.h

Issue 207603003: Extract RequestInfo struct from ResourceLoaderBridge. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fixes Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/test/test_webkit_platform_support.cc ('k') | webkit/child/resource_loader_bridge.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/child/resource_loader_bridge.h
diff --git a/webkit/child/resource_loader_bridge.h b/webkit/child/resource_loader_bridge.h
index 75323867b2c73e78f4eb73fdf29d2a1e47778087..efe8b77b8d7d203c54349d980a4a5f05dffdf8e9 100644
--- a/webkit/child/resource_loader_bridge.h
+++ b/webkit/child/resource_loader_bridge.h
@@ -26,12 +26,9 @@
#include "base/platform_file.h"
#include "base/values.h"
#include "net/base/request_priority.h"
-#include "third_party/WebKit/public/platform/WebReferrerPolicy.h"
-#include "third_party/WebKit/public/platform/WebURLRequest.h"
#include "url/gurl.h"
#include "webkit/child/webkit_child_export.h"
#include "webkit/common/resource_response_info.h"
-#include "webkit/common/resource_type.h"
// TODO(pilgrim) remove this once resource loader is moved to content
// http://crbug.com/338338
@@ -43,71 +40,6 @@ namespace webkit_glue {
class ResourceLoaderBridge {
public:
- // Structure used when calling
- // WebKitPlatformSupportImpl::CreateResourceLoader().
- struct WEBKIT_CHILD_EXPORT RequestInfo {
- RequestInfo();
- ~RequestInfo();
-
- // HTTP-style method name (e.g., "GET" or "POST").
- std::string method;
-
- // Absolute URL encoded in ASCII per the rules of RFC-2396.
- GURL url;
-
- // URL of the document in the top-level window, which may be checked by the
- // third-party cookie blocking policy.
- GURL first_party_for_cookies;
-
- // Optional parameter, a URL with similar constraints in how it must be
- // encoded as the url member.
- GURL referrer;
-
- // The referrer policy that applies to the referrer.
- blink::WebReferrerPolicy referrer_policy;
-
- // For HTTP(S) requests, the headers parameter can be a \r\n-delimited and
- // \r\n-terminated list of MIME headers. They should be ASCII-encoded using
- // the standard MIME header encoding rules. The headers parameter can also
- // be null if no extra request headers need to be set.
- std::string headers;
-
- // Composed of the values defined in url_request_load_flags.h.
- int load_flags;
-
- // Process id of the process making the request.
- int requestor_pid;
-
- // Indicates if the current request is the main frame load, a sub-frame
- // load, or a sub objects load.
- ResourceType::Type request_type;
-
- // Indicates the priority of this request, as determined by WebKit.
- net::RequestPriority priority;
-
- // Used for plugin to browser requests.
- uint32 request_context;
-
- // Identifies what appcache host this request is associated with.
- int appcache_host_id;
-
- // Used to associated the bridge with a frame's network context.
- int routing_id;
-
- // If true, then the response body will be downloaded to a file and the
- // path to that file will be provided in ResponseInfo::download_file_path.
- bool download_to_file;
-
- // True if the request was user initiated.
- bool has_user_gesture;
-
- // Extra data associated with this request. We do not own this pointer.
- blink::WebURLRequest::ExtraData* extra_data;
-
- private:
- DISALLOW_COPY_AND_ASSIGN(RequestInfo);
- };
-
// See the SyncLoad method declared below. (The name of this struct is not
// suffixed with "Info" because it also contains the response data.)
struct SyncLoadResponse : ResourceResponseInfo {
« no previous file with comments | « content/test/test_webkit_platform_support.cc ('k') | webkit/child/resource_loader_bridge.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698