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

Unified Diff: webkit/common/resource_request_body.h

Issue 16831010: Move some classes out of webkit/glue/. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Move UploadDataStreamBuilder to content/browser Created 7 years, 6 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 | « webkit/common/resource_devtools_info.cc ('k') | webkit/common/resource_request_body.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/common/resource_request_body.h
diff --git a/webkit/glue/resource_request_body.h b/webkit/common/resource_request_body.h
similarity index 59%
rename from webkit/glue/resource_request_body.h
rename to webkit/common/resource_request_body.h
index 71a049577c389d759d303e9fbfe0ca42f21d09c5..9050b39a0575eb36a5a9827880ed8861a47d5c75 100644
--- a/webkit/glue/resource_request_body.h
+++ b/webkit/common/resource_request_body.h
@@ -1,40 +1,28 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Copyright (c) 2013 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef WEBKIT_GLUE_RESOURCE_REQUEST_BODY_H_
-#define WEBKIT_GLUE_RESOURCE_REQUEST_BODY_H_
+#ifndef WEBKIT_COMMON_RESOURCE_REQUEST_BODY_H_
+#define WEBKIT_COMMON_RESOURCE_REQUEST_BODY_H_
#include <vector>
#include "base/basictypes.h"
#include "base/memory/ref_counted.h"
#include "base/supports_user_data.h"
+#include "googleurl/src/gurl.h"
#include "webkit/base/data_element.h"
-#include "webkit/glue/webkit_glue_export.h"
+#include "webkit/common/webkit_common_export.h"
namespace base {
class FilePath;
-class TaskRunner;
-}
-
-namespace fileapi {
-class FileSystemContext;
-}
-
-namespace net {
-class UploadDataStream;
-}
-
-namespace webkit_blob {
-class BlobStorageController;
}
namespace webkit_glue {
// A struct used to represent upload data. The data field is populated by
// WebURLLoader from the data given as WebHTTPBody.
-class WEBKIT_GLUE_EXPORT ResourceRequestBody
+class WEBKIT_COMMON_EXPORT ResourceRequestBody
: public base::RefCounted<ResourceRequestBody>,
public base::SupportsUserData {
public:
@@ -50,16 +38,6 @@ class WEBKIT_GLUE_EXPORT ResourceRequestBody
void AppendFileSystemFileRange(const GURL& url, uint64 offset, uint64 length,
const base::Time& expected_modification_time);
- // Creates a new UploadDataStream from this request body. This also resolves
- // any blob references using given |blob_controller|. |file_system_context| is
- // used to create FileStreamReader for files with filesystem URLs.
- // |file_task_runner| is used to perform file operations when the data gets
- // uploaded.
- net::UploadDataStream* ResolveElementsAndCreateUploadDataStream(
- webkit_blob::BlobStorageController* blob_controller,
- fileapi::FileSystemContext* file_system_context,
- base::TaskRunner* file_task_runner);
-
const std::vector<Element>* elements() const { return &elements_; }
std::vector<Element>* elements_mutable() { return &elements_; }
void swap_elements(std::vector<Element>* elements) {
@@ -76,12 +54,6 @@ class WEBKIT_GLUE_EXPORT ResourceRequestBody
friend class base::RefCounted<ResourceRequestBody>;
virtual ~ResourceRequestBody();
- // Resolves the |blob_url| using |blob_controller| and appends resolved
- // items to |resolved_elements|.
- void ResolveBlobReference(webkit_blob::BlobStorageController* blob_controller,
- const GURL& blob_url,
- std::vector<const Element*>* resolved_elements);
-
std::vector<Element> elements_;
int64 identifier_;
« no previous file with comments | « webkit/common/resource_devtools_info.cc ('k') | webkit/common/resource_request_body.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698